Merge pull request #11926 from stweil/declarations

Add missing variable declarations

apps/federatedfilesharing/js/settings-personal.js

@@ -28,7 +28,7 @@ $(document).ready(function() {
 	// Clipboard!
 	var clipboard = new Clipboard('.clipboardButton');
 	clipboard.on('success', function(e) {
-		$input = $(e.trigger);
+		var $input = $(e.trigger);
 		$input.tooltip('hide')
 			.attr('data-original-title', t('core', 'Copied!'))
 			.tooltip('fixTitle')
@@ -41,7 +41,7 @@ $(document).ready(function() {
 		}, 3000);
 	});
 	clipboard.on('error', function (e) {
-		$input = $(e.trigger);
+		var $input = $(e.trigger);
 		var actionMsg = '';
 		if (/iPhone|iPad/i.test(navigator.userAgent)) {
 			actionMsg = t('core', 'Not supported!');

apps/federation/js/settings-admin.js

@@ -77,7 +77,7 @@
 
             console.log("typing away");
 
-            url = $(this).val();
+            var url = $(this).val();
 
             // toggle add-button visiblity based on input length
             if ( url.length > 0 )

apps/files/js/newfilemenu.js

@@ -171,7 +171,7 @@
 
 					/* Find the right actionHandler that should be called.
 					 * Actions is retrieved by using `actionSpec.id` */
-					action = _.filter(self._menuItems, function(item) {
+					var action = _.filter(self._menuItems, function(item) {
 						return item.id == $target.attr('data-action');
 					}).pop();
 					action.actionHandler(newname);

apps/files/js/sidebarpreviewmanager.js

@@ -9,7 +9,7 @@
  */
 
 (function () {
-	SidebarPreviewManager = function (fileList) {
+	var SidebarPreviewManager = function (fileList) {
 		this._fileList = fileList;
 		this._previewHandlers = {};
 		OC.Plugins.attach('OCA.Files.SidebarPreviewManager', this);

apps/files_sharing/js/sharedfilelist.js

@@ -90,7 +90,7 @@
 				var permission = fileData.permissions;
 				$tr.attr('data-share-permissions', permission);
 			}
-			
+
 			// add row with expiration date for link only shares - influenced by _createRow of filelist
 			if (this._linksOnly) {
 				var expirationTimestamp = 0;
@@ -107,6 +107,8 @@
 					modifiedColor = 160;
 				}
 
+				var formatted;
+				var text;
 				if (expirationTimestamp > 0) {
 					formatted = OC.Util.formatDate(expirationTimestamp);
 					text = OC.Util.relativeModifiedDate(expirationTimestamp);

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.

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';

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));