jslint: empty block and || syntax

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

diff --git a/js/QuarantineView.js b/js/QuarantineView.js
index 8767c34f8fcaf1ff4e915b846d4a5265dbb1b6cc..e5957bb441ba8fa5f773a6f4bc5f8d7ae54d3085 100644 (file)
--- a/js/QuarantineView.js
+++ b/js/QuarantineView.js
@@ -79,12 +79,11 @@ Ext.define('PMG.QuarantineView', {
                subpath = subpath || 0;
                if (lastpanel.getActiveTab) {
                    // we assume lastpanel is a tabpanel
-                   if (lastpanel.getActiveTab().getItemId() === subpath) {
-                       // we are already there
-                   } else {
+                   if (lastpanel.getActiveTab().getItemId() !== subpath) {
                        // set the active tab
                        lastpanel.setActiveTab(subpath);
                    }
+                   // else we are already there
                }
                action.stop();
                return;

diff --git a/js/SpamDetectorOptions.js b/js/SpamDetectorOptions.js
index 60aeb3fefd765ab12c1d76ec6368f74ea759fc65..e53f5b4379051454a95921601ca7aa36dfbb0778 100644 (file)
--- a/js/SpamDetectorOptions.js
+++ b/js/SpamDetectorOptions.js
@@ -28,7 +28,7 @@ Ext.define('PMG.SpamDetectorOptions', {
            header: gettext('Languages'),
            editor: 'PMG.SpamDetectorLanguages',
            renderer: function(value) {
-               return value ? value : 'all';
+               return value || 'all';
            }
        };