From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from firstgate.proxmox.com (firstgate.proxmox.com [212.224.123.68]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by lists.proxmox.com (Postfix) with ESMTPS id D710A6CBD5 for ; Tue, 2 Feb 2021 14:07:45 +0100 (CET) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id D4DB0F24C for ; Tue, 2 Feb 2021 14:07:45 +0100 (CET) Received: from proxmox-new.maurer-it.com (proxmox-new.maurer-it.com [212.186.127.180]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by firstgate.proxmox.com (Proxmox) with ESMTPS id 636B3F23F for ; Tue, 2 Feb 2021 14:07:45 +0100 (CET) Received: from proxmox-new.maurer-it.com (localhost.localdomain [127.0.0.1]) by proxmox-new.maurer-it.com (Proxmox) with ESMTP id 2F6134612E for ; Tue, 2 Feb 2021 14:07:45 +0100 (CET) To: pve-devel@lists.proxmox.com References: <20210201142131.30024-1-a.lauterer@proxmox.com> <20210201142131.30024-5-a.lauterer@proxmox.com> From: Dominik Csapak Message-ID: <4f0cb5f8-b49d-8f68-c4e9-ccf5c1158299@proxmox.com> Date: Tue, 2 Feb 2021 14:07:44 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.6.1 MIME-Version: 1.0 In-Reply-To: <20210201142131.30024-5-a.lauterer@proxmox.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit X-SPAM-LEVEL: Spam detection results: 0 AWL 0.237 Adjusted score from AWL reputation of From: address KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment NICE_REPLY_A -0.001 Looks like a legit reply (A) RCVD_IN_DNSWL_MED -2.3 Sender listed at https://www.dnswl.org/, medium trust SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_PASS -0.001 SPF: sender matches SPF record Subject: Re: [pve-devel] [PATCH manager 4/7] ui: qemu/HardwareView: eslint: enforce "no-shadow" rule X-BeenThere: pve-devel@lists.proxmox.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Proxmox VE development discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 02 Feb 2021 13:07:45 -0000 the global confid variable is only there so we could reuse the name so i'd prefer we remove that and use 'let confid' in the relevant blocks (with var this would have failed) On 2/1/21 3:21 PM, Aaron Lauterer wrote: > `confid` is overwritten in each step anyways, so it should be no > problem to use it in the outer scope. > > Let's play it safe for `sm` and rename it in the function. > > Signed-off-by: Aaron Lauterer > --- > > Not sure if my solutions are okay or if another approach is preferable. > Didn't find any patches in the manager repo that deal with the > `no-shadow` rule as a guide though. > > www/manager6/qemu/HardwareView.js | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/www/manager6/qemu/HardwareView.js b/www/manager6/qemu/HardwareView.js > index cc707a2a..213a946f 100644 > --- a/www/manager6/qemu/HardwareView.js > +++ b/www/manager6/qemu/HardwareView.js > @@ -220,7 +220,7 @@ Ext.define('PVE.qemu.HardwareView', { > }; > > PVE.Utils.forEachBus(undefined, function(type, id) { > - var confid = type + id; > + confid = type + id; > rows[confid] = { > group: 10, > iconCls: 'hdd-o', > @@ -531,8 +531,8 @@ Ext.define('PVE.qemu.HardwareView', { > let isAtLimit = (type) => counts[type] >= PVE.Utils.hardware_counts[type]; > > var set_button_status = function() { > - var sm = me.getSelectionModel(); > - var rec = sm.getSelection()[0]; > + var selection_model = me.getSelectionModel(); > + var rec = selection_model.getSelection()[0]; > > // en/disable hardwarebuttons > counts = {}; >