On 2/10/22 15:08, Oguz Bektas wrote:

thanks for the test and review!

instead of a validator function i guess we could just set minValue and
maxValue as well? (since allowBlank is set to true)

seemed to work fine in my short testing just now :)

diff --git a/www/manager6/qemu/NetworkEdit.js b/www/manager6/qemu/NetworkEdit.js
index 1e34ad1c..a3fa5724 100644
--- a/www/manager6/qemu/NetworkEdit.js
+++ b/www/manager6/qemu/NetworkEdit.js
@@ -184,6 +184,8 @@ Ext.define('PVE.qemu.NetworkInputPanel', {
                bind: {
                    disabled: '{!isVirtio}',
                },
+               minValue: 1,
+               maxValue: 65520,
                allowBlank: true,
            },
        ];

Duh of course...
That does what we want and is much nicer :)


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

Reply via email to