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 BC314729BF for ; Wed, 26 May 2021 10:58:41 +0200 (CEST) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id BA48CE2FB for ; Wed, 26 May 2021 10:58:41 +0200 (CEST) Received: from proxmox-new.maurer-it.com (proxmox-new.maurer-it.com [94.136.29.106]) (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 8ECD8E2BF for ; Wed, 26 May 2021 10:58:40 +0200 (CEST) Received: from proxmox-new.maurer-it.com (localhost.localdomain [127.0.0.1]) by proxmox-new.maurer-it.com (Proxmox) with ESMTP id 5FF804667D for ; Wed, 26 May 2021 10:58:40 +0200 (CEST) From: Dominik Csapak To: pve-devel@lists.proxmox.com Date: Wed, 26 May 2021 10:58:31 +0200 Message-Id: <20210526085839.9808-4-d.csapak@proxmox.com> X-Mailer: git-send-email 2.20.1 In-Reply-To: <20210526085839.9808-1-d.csapak@proxmox.com> References: <20210526085839.9808-1-d.csapak@proxmox.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-SPAM-LEVEL: Spam detection results: 1 AWL -1.573 Adjusted score from AWL reputation of From: address ENA_SUBJ_ODD_CASE 3.2 Subject has odd case KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_PASS -0.001 SPF: sender matches SPF record Subject: [pve-devel] [PATCH widget-toolkit 03/11] Toolkit: set clearPropertiesOnDestroy to false by default 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: Wed, 26 May 2021 08:58:41 -0000 instead of the upstream default of 'async' we do this since it creates some problems with our callbacks which can happen during component destruction. The upstream reasoning does not really makes sense for us normally, since we do not keep any references around for most things, and thus the garbage collector can claim it. note that this is only for components, Ext.Base sets it to 'true' by default and we do not change this since we normally do not extend from non-components Signed-off-by: Dominik Csapak --- src/Toolkit.js | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/src/Toolkit.js b/src/Toolkit.js index 6787173..a39f64d 100644 --- a/src/Toolkit.js +++ b/src/Toolkit.js @@ -662,6 +662,12 @@ Ext.define('Proxmox.selection.CheckboxModel', { // override the download server url globally, for privacy reasons Ext.draw.Container.prototype.defaultDownloadServerUrl = "-"; +// stop nulling of properties +Ext.define('Proxmox.Component', { + override: 'Ext.Component', + clearPropertiesOnDestroy: false, +}); + // force alert boxes to be rendered with an Error Icon // since Ext.Msg is an object and not a prototype, we need to override it // after the framework has been initiated -- 2.20.1