]> git.proxmox.com Git - pve-manager.git/commitdiff
ui: eslint: enforce "no-extra-parens" rule
authorThomas Lamprecht <t.lamprecht@proxmox.com>
Tue, 19 Jan 2021 16:42:36 +0000 (17:42 +0100)
committerThomas Lamprecht <t.lamprecht@proxmox.com>
Tue, 19 Jan 2021 16:42:36 +0000 (17:42 +0100)
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
65 files changed:
www/manager6/Utils.js
www/manager6/Workspace.js
www/manager6/button/Split.js
www/manager6/ceph/FS.js
www/manager6/ceph/OSD.js
www/manager6/ceph/Services.js
www/manager6/ceph/Status.js
www/manager6/ceph/StatusDetail.js
www/manager6/data/ResourceStore.js
www/manager6/dc/Backup.js
www/manager6/dc/CorosyncLinkEdit.js
www/manager6/dc/Health.js
www/manager6/dc/NodeView.js
www/manager6/dc/OptionView.js
www/manager6/dc/SecurityGroups.js
www/manager6/dc/Summary.js
www/manager6/dc/TFAEdit.js
www/manager6/form/BridgeSelector.js
www/manager6/form/DiskStorageSelector.js
www/manager6/form/FileSelector.js
www/manager6/form/GlobalSearchField.js
www/manager6/form/MDevSelector.js
www/manager6/form/MemoryField.js
www/manager6/form/NodeSelector.js
www/manager6/form/PCISelector.js
www/manager6/form/StorageSelector.js
www/manager6/form/USBSelector.js
www/manager6/form/ViewSelector.js
www/manager6/grid/FirewallAliases.js
www/manager6/grid/FirewallRules.js
www/manager6/grid/PoolMembers.js
www/manager6/grid/Replication.js
www/manager6/ha/GroupEdit.js
www/manager6/ha/ResourceEdit.js
www/manager6/ha/Status.js
www/manager6/ha/StatusView.js
www/manager6/lxc/Config.js
www/manager6/lxc/FeaturesEdit.js
www/manager6/lxc/Network.js
www/manager6/lxc/Resources.js
www/manager6/node/LVM.js
www/manager6/node/ZFS.js
www/manager6/panel/ConfigPanel.js
www/manager6/panel/IPSet.js
www/manager6/panel/RunningChart.js
www/manager6/panel/StatusPanel.js
www/manager6/qemu/AgentIPView.js
www/manager6/qemu/BootOrderEdit.js
www/manager6/qemu/CDEdit.js
www/manager6/qemu/Config.js
www/manager6/qemu/HDEdit.js
www/manager6/qemu/HDMove.js
www/manager6/qemu/HardwareView.js
www/manager6/qemu/MemoryEdit.js
www/manager6/qemu/NetworkEdit.js
www/manager6/qemu/OSDefaults.js
www/manager6/sdn/Status.js
www/manager6/sdn/StatusView.js
www/manager6/storage/IScsiEdit.js
www/manager6/storage/ImageView.js
www/manager6/storage/TemplateView.js
www/manager6/tree/ResourceTree.js
www/manager6/window/Backup.js
www/manager6/window/BulkAction.js
www/manager6/window/Restore.js

index 5d476600b458bfa5caf44c192158ae908310cd19..074823036aca70bd47707f2354169610615ea5f6 100644 (file)
@@ -308,7 +308,7 @@ Ext.define('PVE.Utils', {
        selected.forEach(function(item) {
            cur++;
            if (item > 2) {
-               days.push(Ext.Date.dayNames[(cur+1)] + '-' + Ext.Date.dayNames[(cur+item)%7]);
+               days.push(Ext.Date.dayNames[cur+1] + '-' + Ext.Date.dayNames[(cur+item)%7]);
                cur += item-1;
            } else if (item == 2) {
                days.push(Ext.Date.dayNames[cur+1]);
@@ -376,7 +376,7 @@ Ext.define('PVE.Utils', {
     render_hotplug_features: function(value) {
        var fa = [];
 
-       if (!value || (value === '0')) {
+       if (!value || value === '0') {
            return gettext('Disabled');
        }
 
@@ -445,7 +445,7 @@ Ext.define('PVE.Utils', {
     },
 
     render_qemu_machine: function(value) {
-       return value || (Proxmox.Utils.defaultText + ' (i440fx)');
+       return value || Proxmox.Utils.defaultText + ' (i440fx)';
     },
 
     render_qemu_bios: function(value) {
@@ -468,7 +468,7 @@ Ext.define('PVE.Utils', {
        }
     },
     render_as_property_string: function(value) {
-       return (!value) ? Proxmox.Utils.defaultText
+       return !value ? Proxmox.Utils.defaultText
            : PVE.Parser.printPropertyString(value);
     },
 
@@ -879,9 +879,9 @@ Ext.define('PVE.Utils', {
 
     format_storage_type: function(value, md, record) {
        if (value === 'rbd') {
-           value = (!record || record.get('monhost') ? 'rbd' : 'pveceph');
+           value = !record || record.get('monhost') ? 'rbd' : 'pveceph';
        } else if (value === 'cephfs') {
-           value = (!record || record.get('monhost') ? 'cephfs' : 'pvecephfs');
+           value = !record || record.get('monhost') ? 'cephfs' : 'pvecephfs';
        }
 
        var schema = PVE.Utils.storageSchema[value];
@@ -933,7 +933,7 @@ Ext.define('PVE.Utils', {
 
        var maxcpu = record.data.maxcpu || 1;
 
-       if (!Ext.isNumeric(maxcpu) && (maxcpu >= 1)) {
+       if (!Ext.isNumeric(maxcpu) && maxcpu >= 1) {
            return '';
        }
 
@@ -969,7 +969,7 @@ Ext.define('PVE.Utils', {
            return -1;
        }
 
-       return (data.mem / data.maxmem);
+       return data.mem / data.maxmem;
     },
 
     render_mem_usage_percent: function(value, metaData, record, rowIndex, colIndex, store) {
@@ -986,7 +986,7 @@ Ext.define('PVE.Utils', {
                return '';
            }
 
-           return ((mem*100)/maxmem).toFixed(1) + " %";
+           return (mem*100/maxmem).toFixed(1) + " %";
        }
        return (value*100).toFixed(1) + " %";
     },
@@ -1009,12 +1009,12 @@ Ext.define('PVE.Utils', {
     calculate_disk_usage: function(data) {
        if (!Ext.isNumeric(data.disk) ||
            data.type === 'qemu' ||
-           (data.type === 'lxc' && data.uptime === 0) ||
+           data.type === 'lxc' && data.uptime === 0 ||
            data.maxdisk === 0) {
            return -1;
        }
 
-       return (data.disk / data.maxdisk);
+       return data.disk / data.maxdisk;
     },
 
     render_disk_usage_percent: function(value, metaData, record, rowIndex, colIndex, store) {
@@ -1033,7 +1033,7 @@ Ext.define('PVE.Utils', {
        if (!Ext.isNumeric(disk) ||
            type === 'qemu' ||
            maxdisk === 0 ||
-           (type === 'lxc' && record.data.uptime === 0)) {
+           type === 'lxc' && record.data.uptime === 0) {
            return '';
        }
 
@@ -1211,9 +1211,9 @@ Ext.define('PVE.Utils', {
        }
        var dv = PVE.VersionInfo.console || 'xtermjs';
        if (dv === 'vv' && !allowSpice) {
-           dv = (allowXtermjs) ? 'xtermjs' : 'html5';
+           dv = allowXtermjs ? 'xtermjs' : 'html5';
        } else if (dv === 'xtermjs' && !allowXtermjs) {
-           dv = (allowSpice) ? 'vv' : 'html5';
+           dv = allowSpice ? 'vv' : 'html5';
        }
 
        return dv;
index c9c4808dbf2e281cc2bdbd0c4352c58ce1007ce9..0c1b9e0c42e0e02d7869c2daf988899b27574163 100644 (file)
@@ -231,7 +231,7 @@ Ext.define('PVE.StdWorkspace', {
                            var comp = {
                                xtype: tlckup[n.data.type || 'root'] ||
                                    'pvePanelConfig',
-                               showSearch: (n.data.id === 'root') ||
+                               showSearch: n.data.id === 'root' ||
                                    Ext.isDefined(n.data.groupbyid),
                                pveSelNode: n,
                                workspace: me,
@@ -455,7 +455,7 @@ Ext.define('PVE.StdWorkspace', {
                    listeners: {
                        resize: function(panel, width, height) {
                            var viewHeight = me.getSize().height;
-                           if (height > (viewHeight - 150)) {
+                           if (height > viewHeight - 150) {
                                panel.setHeight(viewHeight - 150);
                            }
                        },
index 93d60835be20574d16afd49aa1b7e6b2dc929c60..ea001dbbc9e15f066750930a3bff9357298cd4b5 100644 (file)
@@ -25,7 +25,7 @@ Ext.define('PVE.button.Split', {
        var rec, msg;
        if (me.selModel) {
            rec = me.selModel.getSelection()[0];
-           if (!rec || (me.enableFn(rec) === false)) {
+           if (!rec || me.enableFn(rec) === false) {
                return;
            }
        }
@@ -72,7 +72,7 @@ Ext.define('PVE.button.Split', {
                if (item.selModel) {
                    me.mon(item.selModel, "selectionchange", function() {
                        var rec = item.selModel.getSelection()[0];
-                       if (!rec || (item.enableFn(rec) === false)) {
+                       if (!rec || item.enableFn(rec) === false) {
                            item.setDisabled(true);
                        } else {
                            item.setDisabled(false);
@@ -87,7 +87,7 @@ Ext.define('PVE.button.Split', {
        if (me.selModel) {
            me.mon(me.selModel, "selectionchange", function() {
                var rec = me.selModel.getSelection()[0];
-               if (!rec || (me.enableFn(rec) === false)) {
+               if (!rec || me.enableFn(rec) === false) {
                    me.setDisabled(true);
                } else {
                    me.setDisabled(false);
index 397e90f9b84a3e032799ecf94c85ae64cbf50791..31f0b582e75fa80d5d3b7b60b0c8c3793a8573d9 100644 (file)
@@ -91,7 +91,7 @@ Ext.define('PVE.NodeCephFSPanel', {
        },
        formulas: {
            canCreateFS: function(get) {
-               return (!get('cephfsConfigured') && get('mdsCount') > 0);
+               return !get('cephfsConfigured') && get('mdsCount') > 0;
            },
        },
     },
index a4bf358ac1bb832d6ec94029add80d0d42449349..421502dc4380c00dacaec2e3f8dc8f0976a2dac6 100644 (file)
@@ -460,7 +460,7 @@ Ext.define('PVE.node.CephOsdTree', {
            let rec = selection[0];
            let vm = this.getViewModel();
 
-           let isOsd = rec.data.host && (rec.data.type === 'osd') && (rec.data.id >= 0);
+           let isOsd = rec.data.host && rec.data.type === 'osd' && rec.data.id >= 0;
 
            vm.set('isOsd', isOsd);
            vm.set('downOsd', isOsd && rec.data.status === 'down');
index 3248546d228674e6abb33932deed45f45200ebda..d8cdbaf8dc61083388570cca2277d2a84c7914ab 100644 (file)
@@ -65,7 +65,7 @@ Ext.define('PVE.ceph.Services', {
            }
        });
        var i;
-       var quorummap = (status && status.quorum_names) ? status.quorum_names : [];
+       var quorummap = status && status.quorum_names ? status.quorum_names : [];
        var monmessages = {};
        var mgrmessages = {};
        var mdsmessages = {};
index 9c77d4edb9fd3ffe2d00e9ea5cd256c8d1c538e2..96b190c0759a9fdc55bca09e4217b0364f76cd58 100644 (file)
@@ -335,7 +335,7 @@ Ext.define('PVE.node.CephStatus', {
        if (pgmap.recovering_objects_per_sec !== undefined || unhealthy > 0) {
            let total = pgmap.misplaced_total || pgmap.unfound_total || pgmap.degraded_total || 0;
            if (total === 0) return;
-           let recovered = (total - unhealthy) || 0;
+           let recovered = total - unhealthy || 0;
            let speed = pgmap.recovering_bytes_per_sec || 0;
            let speedTxt = PVE.Utils.render_bandwidth(speed);
            let obj_per_sec = speed / (4*1024*1024); // 4MiB per Object
@@ -392,7 +392,7 @@ Ext.define('PVE.node.CephStatus', {
        var regex = new RegExp("not (installed|initialized)", "i");
        PVE.Utils.handleStoreErrorOrMask(me, me.store, regex, function(me, error) {
            me.store.stopUpdate();
-           PVE.Utils.showCephInstallOrMask(me, error.statusText, (nodename || 'localhost'),
+           PVE.Utils.showCephInstallOrMask(me, error.statusText, nodename || 'localhost',
                function(win) {
                    me.mon(win, 'cephInstallWindowClosed', function() {
                        me.store.startUpdate();
index 716b6a019e26fc3465ef582be534aa01e120b0f6..dfd6d103f360f64b7a19fd9e81ad220310ad597e 100644 (file)
@@ -216,7 +216,7 @@ Ext.define('PVE.ceph.StatusDetail', {
        var pgmap = status.pgmap || {};
        var pgs_by_state = pgmap.pgs_by_state || [];
        pgs_by_state.sort(function(a, b) {
-           return (a.state_name < b.state_name)?-1:(a.state_name === b.state_name)?0:1;
+           return a.state_name < b.state_name?-1:a.state_name === b.state_name?0:1;
        });
 
        me.statecategories.forEach(function(cat) {
index 3b466f8544cd0c24ac52ac6a5efa65d49657e571..e94cdd43ab611315994e3f4ef3e51b1ab12ceb3a 100644 (file)
@@ -5,7 +5,7 @@ Ext.define('PVE.data.ResourceStore', {
     findVMID: function(vmid) {
        var me = this, i;
 
-       return (me.findExact('vmid', parseInt(vmid, 10)) >= 0);
+       return me.findExact('vmid', parseInt(vmid, 10)) >= 0;
     },
 
     // returns the cached data from all nodes
@@ -81,7 +81,7 @@ Ext.define('PVE.data.ResourceStore', {
                hidden: true,
                convert: function(value, record) {
                    var info = record.data;
-                   return (Ext.isNumeric(info.uptime) && (info.uptime > 0));
+                   return Ext.isNumeric(info.uptime) && info.uptime > 0;
                },
            },
            text: {
index 83f5beafcf0ff76e08f43a5b9dd3e7818c508578..0efc9d4998b56f356050c6fd685e2d500113c463 100644 (file)
@@ -165,7 +165,7 @@ Ext.define('PVE.dc.BackupEdit', {
                    var mode = selModeField.getValue();
                    store.clearFilter();
                    store.filterBy(function(rec) {
-                       return (!value || rec.get('node') === value);
+                       return !value || rec.get('node') === value;
                    });
                    if (mode === 'all') {
                        sm.selectAll(true);
@@ -330,7 +330,7 @@ Ext.define('PVE.dc.BackupEdit', {
                    var node = nodesel.getValue();
                    store.clearFilter();
                    store.filterBy(function(rec) {
-                       return (!node || node.length === 0 || rec.get('node') === node);
+                       return !node || node.length === 0 || rec.get('node') === node;
                    });
                    var list = vmidField.getValue();
                    var mode = selModeField.getValue();
index a6e1ecc59942dd4c8bfa95e57d9efb476c416426..a5213b81860d07ef60c6cfba140cbb5b3de7f6e9 100644 (file)
@@ -262,7 +262,7 @@ Ext.define('PVE.form.CorosyncLinkSelector', {
                return true;
            }
 
-           if (val === undefined || (val instanceof String && val.length === 0)) {
+           if (val === undefined || val instanceof String && val.length === 0) {
                // let this be caught by allowBlank, if at all
                return true;
            }
index ec7d3497e163092951cf262622d7244589064a8a..8f4a387cd59e9cec43a7c5c1871804b70800e1ee 100644 (file)
@@ -59,7 +59,7 @@ Ext.define('PVE.dc.Health', {
            }
        }
 
-       if (numNodes !== (nodes.online + nodes.offline)) {
+       if (numNodes !== nodes.online + nodes.offline) {
            nodes.offline = numNodes - nodes.online;
        }
 
index f1d6ed154616fab5f349cdbb758c9902df1dd4b4..838d347cdf9bf778fbdaf8681ba130cf2727c8f3 100644 (file)
@@ -25,7 +25,7 @@ Ext.define('PVE.dc.NodeView', {
            sortable: true,
            dataIndex: 'online',
            renderer: function(value) {
-               var cls = (value)?'good':'critical';
+               var cls = value?'good':'critical';
                return '<i class="fa ' + PVE.Utils.get_health_icon(cls) + '"><i/>';
            },
        },
index a5232567eb4d3eb1eb0ed1a2710c4e0c84e12cdf..0e783cf497a9658d438e45780bc6df96dc3fb4bd 100644 (file)
@@ -12,7 +12,7 @@ Ext.define('PVE.dc.OptionView', {
        opts = opts || {};
        me.rows = me.rows || {};
 
-       let canEdit = (!opts.hasOwnProperty('caps') || opts.caps);
+       let canEdit = !opts.hasOwnProperty('caps') || opts.caps;
        me.rows[name] = {
            required: true,
            defaultValue: opts.defaultValue,
index 289c92c91989bf660c17853d0fd3ac56b18e7c32..2aecaeeb945e272d1c52802d55b8c3eab22cd566 100644 (file)
@@ -15,7 +15,7 @@ Ext.define('PVE.SecurityGroupEdit', {
     initComponent: function() {
        var me = this;
 
-       me.isCreate = (me.group_name === undefined);
+       me.isCreate = me.group_name === undefined;
 
        var subject;
 
@@ -152,7 +152,7 @@ Ext.define('PVE.SecurityGroupList', {
            selModel: sm,
            baseurl: me.base_url + '/',
            enableFn: function(rec) {
-               return (rec && me.base_url);
+               return rec && me.base_url;
            },
            callback: function() {
                reload();
index a6acc964bb3a7e5d0ed8d652eb320999a8d168a5..5b09116b3d27805f9ff2cb81df0ba0f75c7c69b6 100644 (file)
@@ -157,7 +157,7 @@ Ext.define('PVE.dc.Summary', {
                var item = results[i];
                switch (item.data.type) {
                    case 'node':
-                       cpu += (item.data.cpu * item.data.maxcpu);
+                       cpu += item.data.cpu * item.data.maxcpu;
                        maxcpu += item.data.maxcpu || 0;
                        memory += item.data.mem || 0;
                        maxmem += item.data.maxmem || 0;
@@ -183,7 +183,7 @@ Ext.define('PVE.dc.Summary', {
                            break;
                        }
                        if (!countedStorages[item.data.storage] ||
-                           (!item.data.shared && !countedStorages[item.data.id])) {
+                           !item.data.shared && !countedStorages[item.data.id]) {
                            used += item.data.disk;
                            total += item.data.maxdisk;
 
@@ -207,13 +207,13 @@ Ext.define('PVE.dc.Summary', {
            }
 
            var text = Ext.String.format(gettext('of {0} CPU(s)'), maxcpu);
-           cpustat.updateValue((cpu/maxcpu), text);
+           cpustat.updateValue(cpu/maxcpu, text);
 
            text = Ext.String.format(gettext('{0} of {1}'), PVE.Utils.render_size(memory), PVE.Utils.render_size(maxmem));
-           memorystat.updateValue((memory/maxmem), text);
+           memorystat.updateValue(memory/maxmem, text);
 
            text = Ext.String.format(gettext('{0} of {1}'), PVE.Utils.render_size(used), PVE.Utils.render_size(total));
-           storagestat.updateValue((used/total), text);
+           storagestat.updateValue(used/total, text);
 
            gueststatus.updateValues(qemu, lxc, error);
 
index 39790823d32b3a1a32a4e12c7859e88062322601..0613ecf993e3adc2937e696ada9f5bd2915025ed 100644 (file)
@@ -112,15 +112,15 @@ Ext.define('PVE.window.TFAEdit', {
                return get('secret').length > 0 && get('canSetupTOTP');
            },
            canDeleteTFA: function(get) {
-               return (get('tfa_type') !== null && !get('tfa_required'));
+               return get('tfa_type') !== null && !get('tfa_required');
            },
            canSetupTOTP: function(get) {
                var tfa = get('tfa_type');
-               return (tfa === null || tfa === 'totp' || tfa === 1);
+               return tfa === null || tfa === 'totp' || tfa === 1;
            },
            canSetupU2F: function(get) {
                var tfa = get('tfa_type');
-               return (get('u2f_available') && (tfa === null || tfa === 'u2f' || tfa === 1));
+               return get('u2f_available') && (tfa === null || tfa === 'u2f' || tfa === 1);
            },
            secretEmpty: function(get) {
                return get('secret').length === 0;
index 6e08741e7d8a22dcd6848ee7bc04d2d15b018622..350588cdb87018c13e5d307c45e2456259c731ca 100644 (file)
@@ -42,7 +42,7 @@ Ext.define('PVE.form.BridgeSelector', {
     setNodename: function(nodename) {
        var me = this;
 
-       if (!nodename || (me.nodename === nodename)) {
+       if (!nodename || me.nodename === nodename) {
            return;
        }
 
index 453dc1992d952217164f8c2beb2adf13722ab9e7..cf73f2e204f286a0bbe59631b66b850383d3eb6b 100644 (file)
@@ -58,7 +58,7 @@ Ext.define('PVE.form.DiskStorageSelector', {
        if (rec.data.format) {
            var format = rec.data.format[0]; // 0 is the formats, 1 the default in the backend
            delete format.subvol; // we never need subvol in the gui
-           selectformat = (Ext.Object.getSize(format) > 1);
+           selectformat = Ext.Object.getSize(format) > 1;
        }
 
        var select = !!rec.data.select_existing && !me.hideSelection;
index 94e716d64284d197bd3b974fbd27f919f4ce6875..ef2bedf9921f670c5a398defaea2038a9a179422 100644 (file)
@@ -19,12 +19,12 @@ Ext.define('PVE.form.FileSelector', {
        var me = this;
 
        var change = false;
-       if (storage && (me.storage !== storage)) {
+       if (storage && me.storage !== storage) {
            me.storage = storage;
            change = true;
        }
 
-       if (nodename && (me.nodename !== nodename)) {
+       if (nodename && me.nodename !== nodename) {
            me.nodename = nodename;
            change = true;
        }
index f4225d9f6f085482de8849723c4c79cc4d9eb5ec..5158cce5e521dc34c9b573d07045b806c33edfd7 100644 (file)
@@ -141,7 +141,7 @@ Ext.define('PVE.form.GlobalSearchField', {
        }
        // give the row the 'relevance' value
        item.data.relevance = match;
-       return (match > 0);
+       return match > 0;
     },
 
     updateFilter: function(field, newValue, oldValue) {
index 399f7b379bbbcaf4e3fd4269d6a4ba7bb08fba52..8ee73c0c0874da69ea5cd5b05fbae7871acf9363 100644 (file)
@@ -45,7 +45,7 @@ Ext.define('PVE.form.MDevSelector', {
     setPciID: function(pciid, force) {
        var me = this;
 
-       if (!force && (!pciid || (me.pciid === pciid))) {
+       if (!force && (!pciid || me.pciid === pciid)) {
            return;
        }
 
@@ -57,7 +57,7 @@ Ext.define('PVE.form.MDevSelector', {
     setNodename: function(nodename) {
        var me = this;
 
-       if (!nodename || (me.nodename === nodename)) {
+       if (!nodename || me.nodename === nodename) {
            return;
        }
 
index c4d92643e4105636958f4e157ba2b08ddb151ae8..10b7ff36293f3a6fa18ef504dca8133934f8e148 100644 (file)
@@ -36,10 +36,10 @@ Ext.define('PVE.form.MemoryField', {
        var i, j;
        for (j = 0; j < 9; j++) {
            for (i = 0; i < 32; i++) {
-               if ((value >= current_size) && (value < (current_size + dimm_size))) {
+               if (value >= current_size && value < current_size + dimm_size) {
                    value_start = current_size;
                    value_up = current_size + dimm_size;
-                   value_down = current_size - ((i === 0) ? prev_dimm_size : dimm_size);
+                   value_down = current_size - (i === 0 ? prev_dimm_size : dimm_size);
                }
                current_size += dimm_size;
            }
index 2a89d1d2e4f3d1b52b755e0e67c59f7cd6f0e43c..4a0f7da7c3e9855f23370bbe52c7834089383235 100644 (file)
@@ -64,7 +64,7 @@ Ext.define('PVE.form.NodeSelector', {
 
     validator: function(value) {
        var me = this;
-       if (!me.onlineValidator || (me.allowBlank && !value)) {
+       if (!me.onlineValidator || me.allowBlank && !value) {
            return true;
        }
 
index fb9f84d4c34784f3b4898d92568c6e2675ab1e04..1c949a20ed98988551da6fc83422f024e073d61d 100644 (file)
@@ -58,7 +58,7 @@ Ext.define('PVE.form.PCISelector', {
     setNodename: function(nodename) {
        var me = this;
 
-       if (!nodename || (me.nodename === nodename)) {
+       if (!nodename || me.nodename === nodename) {
            return;
        }
 
index ea4a0744c87cffc2ab75644e827e7eee572bf2ed..60d7d7416c253a68afbfdacca1e06f2d5c588de1 100644 (file)
@@ -63,7 +63,7 @@ Ext.define('PVE.form.StorageSelector', {
     setTargetNode: function(targetNode) {
        var me = this;
 
-       if (!targetNode || (me.targetNode === targetNode)) {
+       if (!targetNode || me.targetNode === targetNode) {
            return;
        }
 
@@ -75,7 +75,7 @@ Ext.define('PVE.form.StorageSelector', {
     setNodename: function(nodename) {
        var me = this;
 
-       if (!nodename || (me.nodename === nodename)) {
+       if (!nodename || me.nodename === nodename) {
            return;
        }
 
index 4fef6d341cf58d06b90fe3ab27574cb1aa7e35f6..355bc88ccd002e6e16eea000347bd6ce47cce9e2 100644 (file)
@@ -62,7 +62,7 @@ Ext.define('PVE.form.USBSelector', {
                width: 520,
                columns: [
                    {
-                       header: (me.type === 'device')?gettext('Device'):gettext('Port'),
+                       header: me.type === 'device'?gettext('Device'):gettext('Port'),
                        sortable: true,
                        dataIndex: 'usbid',
                        width: 80,
index d744b319e94aa70372125ee657500b16f36c7541..96d2c6ed3be966473e04043064059c80ec90d766 100644 (file)
@@ -73,7 +73,7 @@ Ext.define('PVE.form.ViewSelector', {
 
            applyState: function(state, doSelect) {
                var view = me.getValue();
-               if (state && state.value && (view != state.value)) {
+               if (state && state.value && view != state.value) {
                    var record = store.findRecord('key', state.value, 0, false, true, true);
                    if (record) {
                        me.setValue(state.value, true);
index 0281e99d63ae04fb79ce42662afdbd0dd5f79f5a..9562abc2984c78db7c7fe9c4f2a6ba451f6906d7 100644 (file)
@@ -10,7 +10,7 @@ Ext.define('PVE.FirewallAliasEdit', {
     initComponent: function() {
        var me = this;
 
-       me.isCreate = (me.alias_name === undefined);
+       me.isCreate = me.alias_name === undefined;
 
        if (me.isCreate) {
            me.url = '/api2/extjs' + me.base_url;
index d0d804c94762c719e6b739385d047dd30a492da0..c3b8fa538c1faf670e9aeee95ef1b849b6adce1c 100644 (file)
@@ -245,7 +245,7 @@ Ext.define('PVE.FirewallRuleEdit', {
            throw "no list_refs_url specified";
        }
 
-       me.isCreate = (me.rule_pos === undefined);
+       me.isCreate = me.rule_pos === undefined;
 
        if (me.isCreate) {
             me.url = '/api2/extjs' + me.base_url;
@@ -301,7 +301,7 @@ Ext.define('PVE.FirewallGroupRuleEdit', {
     initComponent: function() {
        var me = this;
 
-       me.isCreate = (me.rule_pos === undefined);
+       me.isCreate = me.rule_pos === undefined;
 
        if (me.isCreate) {
             me.url = '/api2/extjs' + me.base_url;
@@ -568,7 +568,7 @@ Ext.define('PVE.FirewallRules', {
            text: gettext('Copy'),
            selModel: sm,
            enableFn: function(rec) {
-               return (rec.data.type === 'in' || rec.data.type === 'out');
+               return rec.data.type === 'in' || rec.data.type === 'out';
            },
            disabled: true,
            handler: run_copy_editor,
index 4be2267c54225754e9670be3cfda2868c434e224..a8ef4b80c517646472c38310d2c857158a333a9c 100644 (file)
@@ -30,7 +30,7 @@ Ext.define('PVE.pool.AddVM', {
            ],
            filters: [
                function(item) {
-                   return ((item.data.type === 'lxc' || item.data.type === 'qemu') && item.data.pool === '');
+                   return (item.data.type === 'lxc' || item.data.type === 'qemu') && item.data.pool === '';
                },
            ],
        });
index ddc80fdd36d355a2d97ba7b23493f1db7c7abf5d..a2b7cd95410ca719dced25e6db30bdbb05d0ee2e 100644 (file)
@@ -17,7 +17,7 @@ Ext.define('PVE.window.ReplicaEdit', {
        var items = [];
 
        items.push({
-           xtype: (me.isCreate && !vmid)?'pveGuestIDSelector':'displayfield',
+           xtype: me.isCreate && !vmid?'pveGuestIDSelector':'displayfield',
            name: 'guest',
            fieldLabel: 'CT/VM ID',
            value: vmid || '',
@@ -430,7 +430,7 @@ Ext.define('PVE.grid.ReplicaView', {
 
        me.rstore = Ext.create('Proxmox.data.UpdateStore', {
            storeid: 'pve-replica-' + me.nodename + me.vmid,
-           model: (mode === 'dc')? 'pve-replication' : 'pve-replication-state',
+           model: mode === 'dc'? 'pve-replication' : 'pve-replication-state',
            interval: 3000,
            proxy: {
                type: 'proxmox',
index f1a1cc88a9e982bc349b3694aba6eb9c60e4b64f..b061cfa05074ecd48c272d92c98554da4883f677 100644 (file)
@@ -104,7 +104,7 @@ Ext.define('PVE.ha.GroupInputPanel', {
            },
            isValid: function() {
                var value = nodefield.getValue();
-               return (value && 0 !== value.length);
+               return value && 0 !== value.length;
            },
        });
 
index 8aaf49dc6ead708e0d811403dcf35e12fd5f49e4..c1388ce5e7d0d868ad6aa81a41fbee5c083e23e4 100644 (file)
@@ -58,7 +58,7 @@ Ext.define('PVE.ha.VMResourceInputPanel', {
            },
        });
 
-       var vmidStore = (me.vmid) ? {} : {
+       var vmidStore = me.vmid ? {} : {
            model: 'PVEResources',
            autoLoad: true,
            sorters: 'vmid',
@@ -80,7 +80,7 @@ Ext.define('PVE.ha.VMResourceInputPanel', {
                xtype: me.vmid ? 'displayfield' : 'vmComboSelector',
                submitValue: me.isCreate,
                name: 'vmid',
-               fieldLabel: (me.vmid && me.guestType === 'ct') ? 'CT' : 'VM',
+               fieldLabel: me.vmid && me.guestType === 'ct' ? 'CT' : 'VM',
                value: me.vmid,
                store: vmidStore,
                validateExists: true,
index 08515710e54d0ab6aea1fb36bf9eaa274217da66..8870faf4f114cf217f8396cb9a03cac6c0570c26 100644 (file)
@@ -14,7 +14,7 @@ Ext.define('PVE.ha.Status', {
        me.rstore = Ext.create('Proxmox.data.ObjectStore', {
            interval: me.interval,
            model: 'pve-ha-status',
-           storeid: 'pve-store-' + (++Ext.idSeed),
+           storeid: 'pve-store-' + ++Ext.idSeed,
            groupField: 'type',
            proxy: {
                 type: 'proxmox',
index c9c52bc45c70e28b763ebc49b830dc159f470715..1ccc670fa636bec3c81af6fd0b6ed26748566e81 100644 (file)
@@ -27,7 +27,7 @@ Ext.define('PVE.ha.StatusView', {
                sorterFn: function(rec1, rec2) {
                    var p1 = me.sortPriority[rec1.data.type];
                    var p2 = me.sortPriority[rec2.data.type];
-                   return (p1 !== p2) ? ((p1 > p2) ? 1 : -1) : 0;
+                   return p1 !== p2 ? p1 > p2 ? 1 : -1 : 0;
                },
            }],
            filters: {
index 1d405cf21e656117f0c6f58e386eab2fe985058c..c0eb75be3c8352e261a8560bebf83a35c4aec39d 100644 (file)
@@ -141,7 +141,7 @@ Ext.define('PVE.lxc.Config', {
                        Ext.create('PVE.ha.VMResourceEdit', {
                            vmid: vmid,
                            guestType: 'ct',
-                           isCreate: (!ha || ha === 'unmanaged'),
+                           isCreate: !ha || ha === 'unmanaged',
                        }).show();
                    },
                },
index 042d32a6c7d2f12bdc44c90150476a15f0d275ae..1c2795b9c45590f4f9bdc391049271b723839965 100644 (file)
@@ -14,10 +14,10 @@ Ext.define('PVE.lxc.FeaturesInputPanel', {
        },
        formulas: {
            privilegedOnly: function(get) {
-               return (get('unprivileged') ? gettext('privileged only') : '');
+               return get('unprivileged') ? gettext('privileged only') : '';
            },
            unprivilegedOnly: function(get) {
-               return (!get('unprivileged') ? gettext('unprivileged only') : '');
+               return !get('unprivileged') ? gettext('unprivileged only') : '';
            },
        },
     },
index a52f3ec4d94dec0dddf38a224dbbe76923b65e52..81f9c07697937abbfd243848fb6957e14e6f587d 100644 (file)
@@ -9,7 +9,7 @@ Ext.define('PVE.lxc.NetworkInputPanel', {
     setNodename: function(nodename) {
        var me = this;
 
-       if (!nodename || (me.nodename === nodename)) {
+       if (!nodename || me.nodename === nodename) {
            return;
        }
 
@@ -56,7 +56,7 @@ Ext.define('PVE.lxc.NetworkInputPanel', {
            cdata.name = 'eth0';
            me.dataCache = {};
        }
-       cdata.firewall = (me.insideWizard || me.isCreate);
+       cdata.firewall = me.insideWizard || me.isCreate;
 
        if (!me.dataCache) {
            throw "no dataCache specified";
@@ -155,14 +155,14 @@ Ext.define('PVE.lxc.NetworkInputPanel', {
            },
        ];
 
-       var dhcp4 = (cdata.ip === 'dhcp');
+       var dhcp4 = cdata.ip === 'dhcp';
        if (dhcp4) {
            cdata.ip = '';
            cdata.gw = '';
        }
 
-       var auto6 = (cdata.ip6 === 'auto');
-       var dhcp6 = (cdata.ip6 === 'dhcp');
+       var auto6 = cdata.ip6 === 'auto';
+       var dhcp6 = cdata.ip6 === 'dhcp';
        if (auto6 || dhcp6) {
            cdata.ip6 = '';
            cdata.gw6 = '';
@@ -284,7 +284,7 @@ Ext.define('PVE.lxc.NetworkInputPanel', {
                value: cdata.ip6,
                emptyText: dhcp6 || auto6 ? '' : Proxmox.Utils.NoneText,
                vtype: 'IP6CIDRAddress',
-               disabled: (dhcp6 || auto6),
+               disabled: dhcp6 || auto6,
                fieldLabel: 'IPv6/CIDR', // do not localize
            },
            {
@@ -292,7 +292,7 @@ Ext.define('PVE.lxc.NetworkInputPanel', {
                name: 'gw6',
                vtype: 'IP6Address',
                value: cdata.gw6,
-               disabled: (dhcp6 || auto6),
+               disabled: dhcp6 || auto6,
                fieldLabel: gettext('Gateway') + ' (IPv6)',
            },
        ];
@@ -370,7 +370,7 @@ Ext.define('PVE.lxc.NetworkView', {
                    records.push(net);
                });
                me.store.loadData(records);
-               me.down('button[name=addButton]').setDisabled((records.length >= 32));
+               me.down('button[name=addButton]').setDisabled(records.length >= 32);
            },
        });
     },
index e58860a55b9ef1078bfa710a10775c0aba29932f..2135d21f9c1516cca9c4084c96e642c6b95d0efd 100644 (file)
@@ -231,7 +231,7 @@ Ext.define('PVE.lxc.RessourceView', {
            var rowdef = rows[key];
 
            var pending = rec.data['delete'] || me.hasPendingChanges(key);
-           var isDisk = (rowdef.tdCls == 'pve-itype-icon-storage');
+           var isDisk = rowdef.tdCls == 'pve-itype-icon-storage';
            var isUnusedDisk = key.match(/^unused\d+/);
 
            var noedit = rec.data['delete'] || !rowdef.editor;
@@ -257,11 +257,11 @@ Ext.define('PVE.lxc.RessourceView', {
            var order1 = rows[v1].order || 0;
            var order2 = rows[v2].order || 0;
 
-           if ((g1 - g2) !== 0) {
+           if (g1 - g2 !== 0) {
                return g1 - g2;
            }
 
-           if ((order1 - order2) !== 0) {
+           if (order1 - order2 !== 0) {
                return order1 - order2;
            }
 
index a778d135fa718d08372bff892df86f4e770d92f5..9e7732d3e65316ea1e9c5d41df694e0cc3d74146 100644 (file)
@@ -164,7 +164,7 @@ Ext.define('PVE.node.LVMList', {
                    name: 'iconCls',
                    calculate: function(data) {
                        var txt = 'fa x-fa-tree fa-';
-                       txt += (data.leaf) ? 'hdd-o' : 'object-group';
+                       txt += data.leaf ? 'hdd-o' : 'object-group';
                        return txt;
                    },
                },
@@ -172,7 +172,7 @@ Ext.define('PVE.node.LVMList', {
                    type: 'number',
                    name: 'usage',
                    calculate: function(data) {
-                       return ((data.size-data.free)/data.size);
+                       return (data.size-data.free)/data.size;
                    },
                },
            ],
index 8ca6776f7bbc6dc0fe7186bbdd556436faa5bca0..b54fb4e7392a34b9c1a6946466fe8e2824318237 100644 (file)
@@ -27,7 +27,7 @@ Ext.define('PVE.node.CreateZFS', {
            disks.sort(function(a, b) {
                var aorder = a.get('order') || 0;
                var border = b.get('order') || 0;
-               return (aorder - border);
+               return aorder - border;
            });
 
            disks.forEach(function(disk) {
index 9f1c373b213b0526bac2948a96f7c3dcbd9abea2..23a6ee581a8be435d31e1e9c15589223ca5684bb 100644 (file)
@@ -254,12 +254,12 @@ Ext.define('PVE.panel.Config', {
        // select the new item
        var statechange = function(sp, key, state) {
            // it the state change is for this panel
-           if (stateid && (key === stateid) && state) {
+           if (stateid && key === stateid && state) {
                // get active item
                var acard = me.getLayout().getActiveItem().itemId;
                // get the itemid of the new value
                var ncard = state.value || me.firstItem;
-               if (ncard && (acard != ncard)) {
+               if (ncard && acard != ncard) {
                    // select the chosen item
                    menu.setSelection(root.findChild('id', ncard, true) || root.firstChild);
                }
index 8cb387c01de27ecf8ce8bbb9dea801065a8535a3..303e518a565c63bc5e0e1d15c5ef7d665f5a8a90 100644 (file)
@@ -168,7 +168,7 @@ Ext.define('PVE.IPSetCidrEdit', {
     initComponent: function() {
        var me = this;
 
-       me.isCreate = (me.cidr === undefined);
+       me.isCreate = me.cidr === undefined;
 
 
        if (me.isCreate) {
index f6c9109d25746c1fd81e45f4922195a3eb0f7191..ab4c9039500293cb77cc5552a0b0b55bbb6e3f85 100644 (file)
@@ -103,7 +103,7 @@ Ext.define('PVE.widget.RunningChart', {
        let me = this.chart;
        let panel = me.up();
        let now = new Date().getTime();
-       let begin = new Date(now - (1000 * panel.timeFrame)).getTime();
+       let begin = new Date(now - 1000 * panel.timeFrame).getTime();
 
        me.store.add({
            time: time || now,
index 12a76042e1e9bfadac471f595c75514116217412..f7bb4a915188358a3ecaa3ad262ee0226ccc0710 100644 (file)
@@ -50,7 +50,7 @@ Ext.define('PVE.panel.StatusPanel', {
            if (key === stateid) {
                var atab = me.getActiveTab().itemId;
                var ntab = state.value;
-               if (state && ntab && (atab != ntab)) {
+               if (state && ntab && atab != ntab) {
                    me.setActiveTab(ntab);
                }
            }
index c3b1f89fd58ebd2aa959588e601c956aff63a908..fd75437fe2453f948d6a3eb5efd552cb1c571471 100644 (file)
@@ -125,8 +125,8 @@ Ext.define('PVE.qemu.AgentIPView', {
        let agentRec = store.getById('agent');
        let state = store.getById('status');
 
-       me.agent = (agentRec && agentRec.data.value === 1);
-       me.running = (state && state.data.value === 'running');
+       me.agent = agentRec && agentRec.data.value === 1;
+       me.running = state && state.data.value === 'running';
 
        var caps = Ext.state.Manager.get('GuiCap');
 
index de80c7e1eeabe28d1f4e6f69afed88867e5bc782..73c2894655b64fccb15bf87bdfb9181cbc2cc155 100644 (file)
@@ -28,10 +28,10 @@ Ext.define('PVE.qemu.BootOrderPanel', {
     },
 
     isBootdev: function(dev, value) {
-       return (this.isDisk(dev) && !this.isCloudinit(value)) ||
+       return this.isDisk(dev) && !this.isCloudinit(value) ||
            (/^net\d+/).test(dev) ||
            (/^hostpci\d+/).test(dev) ||
-           ((/^usb\d+/).test(dev) && !(/spice/).test(value));
+           (/^usb\d+/).test(dev) && !(/spice/).test(value);
     },
 
     setVMConfig: function(vmconfig) {
index 091fa50b32d24dae6d648fa967083caf4c8bb99d..72c010379f8accfd19589f5501f1085ab02d966d 100644 (file)
@@ -7,7 +7,7 @@ Ext.define('PVE.qemu.CDInputPanel', {
     onGetValues: function(values) {
        var me = this;
 
-       var confid = me.confid || (values.controller + values.deviceid);
+       var confid = me.confid || values.controller + values.deviceid;
 
        me.drive.media = 'cdrom';
        if (values.mediaType === 'iso') {
index 49189489ff4e7e271f93815b0e899e4cfca648a5..c9b7637b318dea7fafe9454e4ce5abe6e57d9152 100644 (file)
@@ -116,7 +116,7 @@ Ext.define('PVE.qemu.Config', {
                        var ha = vm.hastate;
                        Ext.create('PVE.ha.VMResourceEdit', {
                            vmid: vmid,
-                           isCreate: (!ha || ha === 'unmanaged'),
+                           isCreate: !ha || ha === 'unmanaged',
                        }).show();
                    },
                },
@@ -397,7 +397,7 @@ Ext.define('PVE.qemu.Config', {
                return;
            }
 
-           var resume = (['prelaunch', 'paused', 'suspended'].indexOf(qmpstatus) !== -1);
+           var resume = ['prelaunch', 'paused', 'suspended'].indexOf(qmpstatus) !== -1;
 
            if (resume || lock === 'suspended') {
                startBtn.setVisible(false);
index d0723c0567de5860a6cc36b7237a7ff8dcfb389d..e22111bf492c9934094fc5fd0b0120292ba9ba4a 100644 (file)
@@ -62,7 +62,7 @@ Ext.define('PVE.qemu.HDInputPanel', {
        var me = this;
 
        var params = {};
-       var confid = me.confid || (values.controller + values.deviceid);
+       var confid = me.confid || values.controller + values.deviceid;
 
        if (me.unused) {
            me.drive.file = me.vmconfig[values.unusedId];
@@ -133,7 +133,7 @@ Ext.define('PVE.qemu.HDInputPanel', {
        values.noreplicate = !PVE.Parser.parseBoolean(drive.replicate, 1);
        values.diskformat = drive.format || 'raw';
        values.cache = drive.cache || '__default__';
-       values.discard = (drive.discard === 'on');
+       values.discard = drive.discard === 'on';
        values.ssd = PVE.Parser.parseBoolean(drive.ssd);
        values.iothread = PVE.Parser.parseBoolean(drive.iothread);
 
index 1047ce5cd22ddd7596122612b30c3ae1279a2a20..6cb1552cd6367b129319ce048236156a52525b77 100644 (file)
@@ -6,7 +6,7 @@ Ext.define('PVE.window.HDMove', {
 
     move_disk: function(disk, storage, format, delete_disk) {
        var me = this;
-       var qemu = (me.type === 'qemu');
+       var qemu = me.type === 'qemu';
        var params = {};
        params.storage = storage;
        params[qemu ? 'disk':'volume'] = disk;
@@ -58,7 +58,7 @@ Ext.define('PVE.window.HDMove', {
            me.type = 'qemu';
        }
 
-       var qemu = (me.type === 'qemu');
+       var qemu = me.type === 'qemu';
 
         var items = [
             {
index 5440a04974341fd829ee2e85560cbe2ea9a6513d..a2e20840e5060e784904b9467b1496a3ab0726e6 100644 (file)
@@ -10,7 +10,7 @@ Ext.define('PVE.qemu.HardwareView', {
        var rowdef = rows[key] || {};
        var iconCls = rowdef.iconCls;
        var icon = '';
-       var txt = (rowdef.header || key);
+       var txt = rowdef.header || key;
 
        metaData.tdAttr = "valign=middle";
 
@@ -93,7 +93,7 @@ Ext.define('PVE.qemu.HardwareView', {
            sockets: {
                header: gettext('Processors'),
                never_delete: true,
-               editor: (caps.vms['VM.Config.CPU'] || caps.vms['VM.Config.HWType'])
+               editor: caps.vms['VM.Config.CPU'] || caps.vms['VM.Config.HWType']
                    ? 'PVE.qemu.ProcessorEdit' : undefined,
                tdCls: 'pve-itype-icon-processor',
                group: 3,
@@ -315,11 +315,11 @@ Ext.define('PVE.qemu.HardwareView', {
            var order1 = rows[v1].order || 0;
            var order2 = rows[v2].order || 0;
 
-           if ((g1 - g2) !== 0) {
+           if (g1 - g2 !== 0) {
                return g1 - g2;
            }
 
-           if ((order1 - order2) !== 0) {
+           if (order1 - order2 !== 0) {
                return order1 - order2;
            }
 
@@ -528,7 +528,7 @@ Ext.define('PVE.qemu.HardwareView', {
        });
 
        let counts = {};
-       let isAtLimit = (type) => (counts[type] >= PVE.Utils.hardware_counts[type]);
+       let isAtLimit = (type) => counts[type] >= PVE.Utils.hardware_counts[type];
 
        var set_button_status = function() {
            var sm = me.getSelectionModel();
@@ -585,19 +585,19 @@ Ext.define('PVE.qemu.HardwareView', {
            var rowdef = rows[key];
 
            var pending = rec.data['delete'] || me.hasPendingChanges(key);
-           var isCDRom = (value && !!value.toString().match(/media=cdrom/));
+           var isCDRom = value && !!value.toString().match(/media=cdrom/);
            var isUnusedDisk = key.match(/^unused\d+/);
            var isUsedDisk = !isUnusedDisk && rowdef.isOnStorageBus && !isCDRom;
 
-           var isCloudInit = (value && value.toString().match(/vm-.*-cloudinit/));
+           var isCloudInit = value && value.toString().match(/vm-.*-cloudinit/);
 
-           var isEfi = (key === 'efidisk0');
+           var isEfi = key === 'efidisk0';
 
-           remove_btn.setDisabled(rec.data['delete'] || (rowdef.never_delete === true) || (isUnusedDisk && !diskCap));
-           remove_btn.setText((isUsedDisk && !isCloudInit) ? remove_btn.altText : remove_btn.defaultText);
+           remove_btn.setDisabled(rec.data['delete'] || rowdef.never_delete === true || isUnusedDisk && !diskCap);
+           remove_btn.setText(isUsedDisk && !isCloudInit ? remove_btn.altText : remove_btn.defaultText);
            remove_btn.RESTMethod = isUnusedDisk ? 'POST':'PUT';
 
-           edit_btn.setDisabled(rec.data['delete'] || !rowdef.editor || isCloudInit || (!isCDRom && !diskCap));
+           edit_btn.setDisabled(rec.data['delete'] || !rowdef.editor || isCloudInit || !isCDRom && !diskCap);
 
            resize_btn.setDisabled(pending || !isUsedDisk || !diskCap);
 
index b4ea127b5960eefae26a11c9e8b78ad05d8b50f3..63c1294ff8d0ba4ecad3329163b375d8249bbbe1 100644 (file)
@@ -19,7 +19,7 @@ Ext.define('PVE.qemu.MemoryInputPanel', {
        } else if (values.memory === values.balloon) {
            delete res.balloon;
            res['delete'] = 'balloon,shares';
-       } else if (Ext.isDefined(values.shares) && (values.shares !== "")) {
+       } else if (Ext.isDefined(values.shares) && values.shares !== "") {
            res.shares = values.shares;
        } else {
            res['delete'] = "shares";
@@ -102,7 +102,7 @@ Ext.define('PVE.qemu.MemoryInputPanel', {
                        var shares = me.down('field[name=shares]');
                        var memory = me.down('field[name=memory]');
                        bf.setDisabled(!value);
-                       shares.setDisabled(!value || (bf.getValue() === memory.getValue()));
+                       shares.setDisabled(!value || bf.getValue() === memory.getValue());
                    },
                },
            },
@@ -156,7 +156,7 @@ Ext.define('PVE.qemu.MemoryEdit', {
                    ballooning: data.balloon === 0 ? '0' : '1',
                    shares: data.shares,
                    memory: data.memory || '512',
-                   balloon: data.balloon > 0 ? data.balloon : (data.memory || '512'),
+                   balloon: data.balloon > 0 ? data.balloon : data.memory || '512',
                };
 
                ipanel.setValues(values);
index 9500b53dcc2094e0340fa89801ec12f7222fe7c5..b39cffdc6fa62b3e979c84eda0eed903736d5bf7 100644 (file)
@@ -83,7 +83,7 @@ Ext.define('PVE.qemu.NetworkInputPanel', {
                xtype: 'proxmoxcheckbox',
                fieldLabel: gettext('Firewall'),
                name: 'firewall',
-               checked: (me.insideWizard || me.isCreate),
+               checked: me.insideWizard || me.isCreate,
            },
        ];
 
index 4af0f5c4305e8bd11c40198f8e0225a690403448..8e5856510268b7b4446d2de32819110ddf89a228 100644 (file)
@@ -28,7 +28,7 @@ Ext.define('PVE.qemu.OSDefaults', {
                    var child = Ext.clone(me[settings.parent]);
                    me[settings.pveOS] = Ext.apply(child, settings);
                } else {
-                   throw ("Could not find your genitor");
+                   throw "Could not find your genitor";
                }
            };
 
index 2ad02809bfddb44a4ba66ebb1a78c665a967ad8e..858d09bdec3b896f6e06d61b591850a2cc456160 100644 (file)
@@ -15,7 +15,7 @@ Ext.define('PVE.sdn.Status', {
        me.rstore = Ext.create('Proxmox.data.ObjectStore', {
            interval: me.interval,
            model: 'pve-sdn-status',
-           storeid: 'pve-store-' + (++Ext.idSeed),
+           storeid: 'pve-store-' + ++Ext.idSeed,
            groupField: 'type',
            proxy: {
                 type: 'proxmox',
index 58dc8d2b4f5142192dfa9209e5a9a3473998bbe3..970aa9195cc2666fc205b6a46af2a41986ae69c0 100644 (file)
@@ -24,7 +24,7 @@ Ext.define('PVE.sdn.StatusView', {
                sorterFn: function(rec1, rec2) {
                    var p1 = me.sortPriority[rec1.data.type];
                    var p2 = me.sortPriority[rec2.data.type];
-                   return (p1 !== p2) ? ((p1 > p2) ? 1 : -1) : 0;
+                   return p1 !== p2 ? p1 > p2 ? 1 : -1 : 0;
                },
            }],
            filters: {
index 4cb6bec57c81b723722a2e6e59c15c677aa284c7..c5afa53bccc8bf4534cdb277d11c3c02c8049663 100644 (file)
@@ -71,7 +71,7 @@ Ext.define('PVE.storage.IScsiInputPanel', {
     },
 
     setValues: function(values) {
-       values.luns = (values.content.indexOf('images') !== -1);
+       values.luns = values.content.indexOf('images') !== -1;
        this.callParent([values]);
     },
 
index e2a6b71538dd204f9280dca0582c0317e4e5b938..c828116d028a7607279b8ab60ffe1f2c8b163cea 100644 (file)
@@ -16,7 +16,7 @@ Ext.define('PVE.storage.ImageView', {
            throw "no storage ID specified";
        }
 
-       if (!me.content || (me.content !== 'images' && me.content !== 'rootdir')) {
+       if (!me.content || me.content !== 'images' && me.content !== 'rootdir') {
            throw "content needs to be either 'images' or 'rootdir'";
        }
 
index c5fda247a253ad012708e857d84a22110105be0b..8230774d06aa041ea356b96b2372211152cdb021 100644 (file)
@@ -53,8 +53,8 @@ Ext.define('PVE.grid.TemplateSelector', {
                            var value = field.getValue().toLowerCase();
                            store.clearFilter(true);
                            store.filterBy(function(rec) {
-                               return (rec.data['package'].toLowerCase().indexOf(value) !== -1) ||
-                               (rec.data.headline.toLowerCase().indexOf(value) !== -1);
+                               return rec.data['package'].toLowerCase().indexOf(value) !== -1 ||
+                               rec.data.headline.toLowerCase().indexOf(value) !== -1;
                            });
                        },
                    },
index cac8aae4b76c7ad880b29c51338db20872fb6d2b..0b76ae1a908a894114d5cfc3a06cb8c133ff462e 100644 (file)
@@ -45,14 +45,14 @@ Ext.define('PVE.tree.ResourceTree', {
        var n1 = node1.data;
        var n2 = node2.data;
 
-       if ((n1.groupbyid && n2.groupbyid) ||
+       if (n1.groupbyid && n2.groupbyid ||
            !(n1.groupbyid || n2.groupbyid)) {
            var tcmp;
 
            var v1 = n1.type;
            var v2 = n2.type;
 
-           if ((tcmp = v1 > v2 ? 1 : (v1 < v2 ? -1 : 0)) != 0) {
+           if ((tcmp = v1 > v2 ? 1 : v1 < v2 ? -1 : 0) != 0) {
                return tcmp;
            }
 
@@ -66,12 +66,12 @@ Ext.define('PVE.tree.ResourceTree', {
                }
                v1 = n1.vmid;
                v2 = n2.vmid;
-               if ((tcmp = v1 > v2 ? 1 : (v1 < v2 ? -1 : 0)) != 0) {
+               if ((tcmp = v1 > v2 ? 1 : v1 < v2 ? -1 : 0) != 0) {
                    return tcmp;
                }
            }
 
-           return n1.id > n2.id ? 1 : (n1.id < n2.id ? -1 : 0);
+           return n1.id > n2.id ? 1 : n1.id < n2.id ? -1 : 0;
        } else if (n1.groupbyid) {
            return -1;
        } else if (n2.groupbyid) {
@@ -422,7 +422,7 @@ Ext.define('PVE.tree.ResourceTree', {
                    var sm = me.getSelectionModel();
                    // disable selection when right clicking
                    // except the record is already selected
-                   me.allowSelection = (ev.button !== 2) || sm.isSelected(record);
+                   me.allowSelection = ev.button !== 2 || sm.isSelected(record);
                },
                beforeselect: function(tree, record, index, eopts) {
                    var allow = me.allowSelection;
index f39906331e0af55cb26f116150d36a27ef46dc89..615073f345259b9f95ff29ddfa0cc0c711a3514e 100644 (file)
@@ -128,7 +128,7 @@ Ext.define('PVE.window.Backup', {
        });
 
        var title = gettext('Backup') + " " +
-           ((me.vmtype === 'lxc') ? "CT" : "VM") +
+           (me.vmtype === 'lxc' ? "CT" : "VM") +
            " " + me.vmid;
 
        Ext.apply(me, {
index de8ef2558a1ac6e85841926673b67bb43de98a11..135f570f33d0ba8d883d2a957b3f9acefe85298d 100644 (file)
@@ -133,8 +133,8 @@ Ext.define('PVE.window.BulkAction', {
                selectionchange: function(vmselector, records) {
                    if (me.action == 'migrateall') {
                        var showWarning = records.some(function(item) {
-                           return (item.data.type == 'lxc' &&
-                               item.data.status == 'running');
+                           return item.data.type == 'lxc' &&
+                               item.data.status == 'running';
                        });
                        me.down('#lxcwarning').setVisible(showWarning);
                    }
index dac883f11403ea9ad8e5aef198956f70da64de87..d220c7bf67b069f85d6ae91b5d9de1425ae36b91 100644 (file)
@@ -23,11 +23,11 @@ Ext.define('PVE.window.Restore', {
            name: 'storage',
            value: '',
            fieldLabel: gettext('Storage'),
-           storageContent: (me.vmtype === 'lxc') ? 'rootdir' : 'images',
+           storageContent: me.vmtype === 'lxc' ? 'rootdir' : 'images',
            // when restoring a container without specifying a storage, the backend defaults
            // to 'local', which is unintuitive and 'rootdir' might not even be allowed on it
            allowBlank: me.vmtype !== 'lxc',
-           emptyText: (me.vmtype === 'lxc') ? '' : gettext('From backup configuration'),
+           emptyText: me.vmtype === 'lxc' ? '' : gettext('From backup configuration'),
            autoSelect: me.vmtype === 'lxc',
        });
 
@@ -36,7 +36,7 @@ Ext.define('PVE.window.Restore', {
            IDfield = Ext.create('Ext.form.field.Display', {
                name: 'vmid',
                value: me.vmid,
-               fieldLabel: (me.vmtype === 'lxc') ? 'CT' : 'VM',
+               fieldLabel: me.vmtype === 'lxc' ? 'CT' : 'VM',
            });
        } else {
            IDfield = Ext.create('PVE.form.GuestIDSelector', {
@@ -186,7 +186,7 @@ Ext.define('PVE.window.Restore', {
        });
 
        var title = gettext('Restore') + ": " + (
-           (me.vmtype === 'lxc') ? 'CT' : 'VM');
+           me.vmtype === 'lxc' ? 'CT' : 'VM');
 
        if (me.vmid) {
            title += " " + me.vmid;