ui: vm selector: don't add invalid not found items

Doing a simple numericity check and warn in the console so developers
can notice if there is something off.

Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
Reviewed-by: Dominik Csapak <d.csapak@proxmox.com>

diff --git a/www/manager6/form/VMSelector.js b/www/manager6/form/VMSelector.js
index 0c884aaead5530dd5663569603f9bde213611b80..22f7dd11d5b6619d6d372dbe46a61a8aa8b8a225 100644 (file)
--- a/www/manager6/form/VMSelector.js
+++ b/www/manager6/form/VMSelector.js
@@ -136,7 +136,11 @@ Ext.define('PVE.form.VMSelector', {
        let selection = value.map(item => {
            let found = store.findRecord('vmid', item, 0, false, true, true);
            if (!found) {
-               notFound.push(item);
+               if (Ext.isNumeric(item)) {
+                   notFound.push(item);
+               } else {
+                   console.warn(`invalid item in vm selection: ${item}`);
+               }
            }
            return found;
        }).filter(r => r);