on multiSelect we have to check the values that is in the values
array, but we get the 'displaystring' in the validator so we
have to get the 'real' value (aka the underlying array of values)
before checking if they are in the store

Signed-off-by: Dominik Csapak <d.csa...@proxmox.com>
---
 form/ComboGrid.js | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/form/ComboGrid.js b/form/ComboGrid.js
index 6dac534..3cc64f9 100644
--- a/form/ComboGrid.js
+++ b/form/ComboGrid.js
@@ -362,7 +362,11 @@ Ext.define('Proxmox.form.ComboGrid', {
        // we normally get here the displayField as value, but if a valueField
        // is configured we need to get the "actual" value, to ensure it is in
        // the store. Below check is copied from ExtJS 6.0.2 ComboBox source
-       if (me.valueField && me.valueField !== me.displayField) {
+       //
+       // we also have to get the 'real' value if the we have a mulitSelect
+       // Field but got a non array value
+       if ((me.valueField && me.valueField !== me.displayField) ||
+           (me.multiSelect && !Ext.isArray(value))) {
            value = me.getValue();
        }
 
-- 
2.20.1


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

Reply via email to