[PATCH manager v3 2/2] ui: fabrics: skip unsupported IP properties on submit

[Lukas Sichert <l.sichert@proxmox.com>]

The interface selector store is initialized from the node network API,
which includes existing IPv4 and IPv6 addresses as ip and ip6. Hiding an
IP column does not remove those fields from the selected record.

getValue() serializes the selected record data, so BGP unnumbered
interfaces could still submit stale ip and ip6 properties even though
BGP supports neither per-interface address. The backend then rejects the
property string with errors such as 'interfaces[0].ip: property is not
defined in schema'.

Skip ip and ip6 during serialization when the corresponding support flag
is disabled.

Signed-off-by: Lukas Sichert <l.sichert@proxmox.com>
Tested-by: Hannes Laimer <h.laimer@proxmox.com>
Reviewed-by: Hannes Laimer <h.laimer@proxmox.com>
---
 www/manager6/sdn/fabrics/InterfacePanel.js | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/www/manager6/sdn/fabrics/InterfacePanel.js b/www/manager6/sdn/fabrics/InterfacePanel.js
index 95424bc0..08a53a34 100644
--- a/www/manager6/sdn/fabrics/InterfacePanel.js
+++ b/www/manager6/sdn/fabrics/InterfacePanel.js
@@ -178,7 +178,11 @@ Ext.define('PVE.sdn.Fabric.InterfacePanel', {
                     continue;
                 }
 
-                if (['type', 'isDisabled'].includes(key)) {
+                if (
+                    ['type', 'isDisabled'].includes(key) ||
+                    (key === 'ip' && !me.hasIpv4Support) ||
+                    (key === 'ip6' && !me.hasIpv6Support)
+                ) {
                     continue;
                 }
 
-- 
2.47.3