From 2ad38200fbad6a21403a1addaa65deab72741760 Mon Sep 17 00:00:00 2001 From: Stefan Weil Date: Fri, 19 Oct 2018 11:23:33 +0200 Subject: [PATCH] Add missing semicolons This fixes some recommendations from LGTM: Avoid automated semicolon insertion (90% of all statements in the enclosing function have an explicit semicolon). Signed-off-by: Stefan Weil --- apps/files/js/detailsview.js | 2 +- apps/files/js/filelist.js | 4 ++-- apps/files_sharing/js/sharedfilelist.js | 2 +- apps/user_ldap/js/wizard/wizardTabAbstractFilter.js | 2 +- core/js/sharedialoglinkshareview.js | 6 +++--- 5 files changed, 8 insertions(+), 8 deletions(-) diff --git a/apps/files/js/detailsview.js b/apps/files/js/detailsview.js index 2a5d589a73..675e380b52 100644 --- a/apps/files/js/detailsview.js +++ b/apps/files/js/detailsview.js @@ -102,7 +102,7 @@ this.$el.insertAfter($('#app-content')); } else { if ($appSidebar[0] !== this.el) { - $appSidebar.replaceWith(this.$el) + $appSidebar.replaceWith(this.$el); } } diff --git a/apps/files/js/filelist.js b/apps/files/js/filelist.js index 94052b10b3..771ec495a2 100644 --- a/apps/files/js/filelist.js +++ b/apps/files/js/filelist.js @@ -310,10 +310,10 @@ breadcrumbOptions.onDrop = _.bind(this._onDropOnBreadCrumb, this); breadcrumbOptions.onOver = function() { self.$el.find('td.filename.ui-droppable').droppable('disable'); - } + }; breadcrumbOptions.onOut = function() { self.$el.find('td.filename.ui-droppable').droppable('enable'); - } + }; } this.breadcrumb = new OCA.Files.BreadCrumb(breadcrumbOptions); diff --git a/apps/files_sharing/js/sharedfilelist.js b/apps/files_sharing/js/sharedfilelist.js index 3a6de0d501..19250815bd 100644 --- a/apps/files_sharing/js/sharedfilelist.js +++ b/apps/files_sharing/js/sharedfilelist.js @@ -232,7 +232,7 @@ promises.push($.ajax(remoteShares)); } if (this._isOverview) { - shares.data.shared_with_me = !shares.data.shared_with_me + shares.data.shared_with_me = !shares.data.shared_with_me; promises.push($.ajax(shares)); } } diff --git a/apps/user_ldap/js/wizard/wizardTabAbstractFilter.js b/apps/user_ldap/js/wizard/wizardTabAbstractFilter.js index c9c3aeb241..0131816e4e 100644 --- a/apps/user_ldap/js/wizard/wizardTabAbstractFilter.js +++ b/apps/user_ldap/js/wizard/wizardTabAbstractFilter.js @@ -302,7 +302,7 @@ OCA = OCA || {}; if( !view.getFilterItem().$element.val() && view.parsedFilterMode === view.configModel.FILTER_MODE_ASSISTED ) { - view.configModel.requestWizard(view.getFilterItem().keyName) + view.configModel.requestWizard(view.getFilterItem().keyName); } } else if (payload.feature === view.getGroupsItem().featureName) { if(view.manyGroupsSupport && payload.data.length > view._groupElementSwitchThreshold) { diff --git a/core/js/sharedialoglinkshareview.js b/core/js/sharedialoglinkshareview.js index 995ff480aa..aac4843c8e 100644 --- a/core/js/sharedialoglinkshareview.js +++ b/core/js/sharedialoglinkshareview.js @@ -114,7 +114,7 @@ }); clipboard.on('error', function (e) { var $menu = $(e.trigger); - var $linkTextMenu = $menu.parent().next('li.linkTextMenu') + var $linkTextMenu = $menu.parent().next('li.linkTextMenu'); var $input = $linkTextMenu.find('.linkText'); var actionMsg = ''; @@ -393,8 +393,8 @@ var isLinkShare = this.model.get('linkShare').isLinkShare; var isPasswordSet = !!this.model.get('linkShare').password; - var isPasswordEnforced = this.configModel.get('enforcePasswordForPublicLink') - var isPasswordEnabledByDefault = this.configModel.get('enableLinkPasswordByDefault') === true + var isPasswordEnforced = this.configModel.get('enforcePasswordForPublicLink'); + var isPasswordEnabledByDefault = this.configModel.get('enableLinkPasswordByDefault') === true; var showPasswordCheckBox = isLinkShare && ( !this.configModel.get('enforcePasswordForPublicLink') || !this.model.get('linkShare').password);