Fix some JSLint warnings

4 files changed, 8 insertions, 7 deletions

diff --git a/settings/js/apps.js b/settings/js/apps.js
index 0540d9b1c58..94300074652 100644
--- a/settings/js/apps.js
+++ b/settings/js/apps.js
@@ -54,7 +54,7 @@ OC.Settings.Apps = OC.Settings.Apps || {
 	enableApp:function(appid, active, element) {
 		console.log('enableApp:', appid, active, element);
 		var appitem=$('#leftcontent li[data-id="'+appid+'"]');
-		appData = appitem.data('app');
+		var appData = appitem.data('app');
 		appData.active = !active;
 		appitem.data('app', appData);
 		element.val(t('settings','Please wait....'));
diff --git a/settings/js/log.js b/settings/js/log.js
index 1ef9b419cdb..5832c698ad5 100644
--- a/settings/js/log.js
+++ b/settings/js/log.js
@@ -35,8 +35,9 @@ OC.Log={
 		$('#moreLog').show();
 		// remove all non-remaining items
 		$('#log tr').slice(OC.Log.loaded).remove();
-		if(OC.Log.loaded <= 3)
+		if(OC.Log.loaded <= 3) {
 			$('#lessLog').hide();
+		}
 	},
 	addEntries:function(entries){
 		for(var i=0;i<entries.length;i++){
diff --git a/settings/js/personal.js b/settings/js/personal.js
index 099c1426dc0..fc638c97455 100644
--- a/settings/js/personal.js
+++ b/settings/js/personal.js
@@ -8,7 +8,7 @@
  * Post the email address change to the server.
  */
 function changeEmailAddress(){
-    emailInfo = $('#email');
+    var emailInfo = $('#email');
     if (emailInfo.val() === emailInfo.defaultValue){
         return;
     }
diff --git a/settings/js/users.js b/settings/js/users.js
index 6a8afc4ca36..798acce8471 100644
--- a/settings/js/users.js
+++ b/settings/js/users.js
@@ -166,7 +166,7 @@ var UserList = {
 			var c = Number(aa[x]), d = Number(bb[x]);
 			if (c === aa[x] && d === bb[x]) {
 				return c - d;
-			} else return (aa[x] > bb[x]) ? 1 : -1;
+			} else { return (aa[x] > bb[x]) ? 1 : -1; }
 			}
 		}
 		return aa.length - bb.length;
@@ -262,7 +262,7 @@ var UserList = {
 					if ($(element).find('option[value="' + group + '"]').length === 0 && select.data('msid') !== $(element).data('msid')) {
 						$(element).append('<option value="' + escapeHTML(group) + '">' + escapeHTML(group) + '</option>');
 					}
-				})
+				});
 			};
 			var label;
 			if (isadmin) {
@@ -304,7 +304,7 @@ var UserList = {
 					if ($(element).find('option[value="' + group + '"]').length === 0) {
 						$(element).append('<option value="' + escapeHTML(group) + '">' + escapeHTML(group) + '</option>');
 					}
-				})
+				});
 			};
 			element.multiSelect({
 				createCallback: addSubAdmin,
@@ -469,7 +469,7 @@ $(document).ready(function () {
 		}
 		OC.Notification.hide();
 	});
-	UserList.useUndo = ('onbeforeunload' in window)
+	UserList.useUndo = ('onbeforeunload' in window);
 	$(window).bind('beforeunload', function () {
 		UserList.finishDelete(null);
 	});