[pve-devel] [PATCH widget-toolkit 09/15] ComboGrid: improve setting 'editable' default value

Dominik Csapak d.csapak at proxmox.com
Mon Jan 15 15:17:42 CET 2018


port commit
bff876ecb5f3388b7d65c491680b9a40574e4744
from pve-manager

commit message:

    This allows child classes to use another default, the initialConfig
    didn't allowed that.
    This is also the ExtJS like way to set defaults.

    Signed-off-by: Thomas Lamprecht <t.lamprecht at proxmox.com>

Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
---
 form/ComboGrid.js | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/form/ComboGrid.js b/form/ComboGrid.js
index 9c88a5d..b9fea19 100644
--- a/form/ComboGrid.js
+++ b/form/ComboGrid.js
@@ -30,6 +30,8 @@ Ext.define('Proxmox.form.ComboGrid', {
     // needed to trigger onKeyUp etc.
     enableKeyEvents: true,
 
+    editable: false,
+
     // override ExtJS method
     // if the field has multiSelect enabled, the store is not loaded, and
     // the displayfield == valuefield, it saves the rawvalue as an array
@@ -254,10 +256,6 @@ Ext.define('Proxmox.form.ComboGrid', {
     initComponent: function() {
 	var me = this;
 
-	if (me.initialConfig.editable === undefined) {
-	    me.editable = false;
-	}
-
 	Ext.apply(me, {
 	    queryMode: 'local',
 	    matchFieldWidth: false
-- 
2.11.0





More information about the pve-devel mailing list