diff options
| author | Stefan Weil <sw@weilnetz.de> | 2018-10-19 12:37:11 +0300 |
|---|---|---|
| committer | Roeland Jago Douma <roeland@famdouma.nl> | 2018-10-19 21:17:55 +0300 |
| commit | c5d3febf37f739b9ffbab636914f2ab368f68bbb (patch) | |
| tree | afaef37a4fc6fc46888cd7131ac5ebe8436efcfd /core | |
| parent | 543c8634340607c5dd088b9cbdcd85700222ee0b (diff) | |
Add missing variable declarations
This fixes errors from LGTM like the following one:
Variable i is used like a local variable,
but is missing a declaration.
Signed-off-by: Stefan Weil <sw@weilnetz.de>
Diffstat (limited to 'core')
| -rw-r--r-- | core/js/jquery-showpassword.js | 2 | ||||
| -rw-r--r-- | core/js/l10n.js | 2 | ||||
| -rw-r--r-- | core/js/placeholder.js | 2 |
3 files changed, 3 insertions, 3 deletions
diff --git a/core/js/jquery-showpassword.js b/core/js/jquery-showpassword.js index 5d518c78bcb..de18c8068be 100644 --- a/core/js/jquery-showpassword.js +++ b/core/js/jquery-showpassword.js @@ -25,7 +25,7 @@ var $element = $(element); - $clone = $("<input />"); + var $clone = $("<input />"); // Name added for JQuery Validation compatibility // Element name is required to avoid script warning. diff --git a/core/js/l10n.js b/core/js/l10n.js index 52dd136e8f5..6027d42dca0 100644 --- a/core/js/l10n.js +++ b/core/js/l10n.js @@ -167,7 +167,7 @@ OC.L10N = { * @private */ _getPlural: function(number) { - locale = OC.getLocale(); + var locale = OC.getLocale(); if ('pt_BR' === locale) { // temporary set a locale for brazilian locale = 'xbr'; diff --git a/core/js/placeholder.js b/core/js/placeholder.js index 7c1bd724550..04c2ca88401 100644 --- a/core/js/placeholder.js +++ b/core/js/placeholder.js @@ -91,7 +91,7 @@ var palette = new Array(); palette.push(color1); var step = stepCalc(steps, [color1, color2]) - for (i = 1; i < steps; i++) { + for (var i = 1; i < steps; i++) { var r = parseInt(color1.r + (step[0] * i)); var g = parseInt(color1.g + (step[1] * i)); var b = parseInt(color1.b + (step[2] * i)); |