-
Notifications
You must be signed in to change notification settings - Fork 65
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Skip parameters which are not there instead of failing out #257
Conversation
@@ -45,8 +45,12 @@ | |||
|
|||
_.each(matches, function(parameter) { | |||
parameter = parameter.substring(1, parameter.length - 1); | |||
var parsed = self.parseParameter(parameters[parameter]); | |||
if (!parameters.hasOwnProperty(parameter) || !parameters[parameter]) { | |||
// Malformed translation? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should this be reported instead of failing silently?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well only thing would be console.log, but that is disabled for non-debug anyway, so I don't think it's important.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
so, wouldn't it be helpful for debugging? for the app dev to to find out where an issues occurs?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
whatever I was thinking, yes of course, amended
travis is unhapy |
Travis was fixed with https://github.com/nextcloud/activity/pull/259/files But I will rebase |
6de54e7
to
92b9084
Compare
Signed-off-by: Joas Schilling <coding@schilljs.com>
92b9084
to
3da098f
Compare
@nickvergessen Next RC is coming - time for backport |
@nickvergessen To backport or not to backport? |
ws already backported see #265 |
Fix #256
Fix #77
Testing today:
Testing tomorrow:
See above + revert #256 (comment)
Expected
Actual