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.lampre...@proxmox.com>
---

changes v2 -> v3:
 * none

 www/manager6/form/ComboGrid.js | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/www/manager6/form/ComboGrid.js b/www/manager6/form/ComboGrid.js
index fbc318e5..18df7930 100644
--- a/www/manager6/form/ComboGrid.js
+++ b/www/manager6/form/ComboGrid.js
@@ -30,6 +30,8 @@ Ext.define('PVE.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('PVE.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


_______________________________________________
pve-devel mailing list
pve-devel@pve.proxmox.com
https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to