From: Stefan Hanreich <s.hanreich@proxmox.com>
Date: Wed, 22 Nov 2023 12:28:08 +0000 (+0100)
Subject: validation: add support for arrays to change tracking
X-Git-Url: https://git.puffer.fish/?a=commitdiff_plain;h=fb97ed300a1eb8ed646d327bca874517621b614f;p=matthieu%2Fpve-network.git

validation: add support for arrays to change tracking

This is needed so dhcp-ranges are properly displayed as changed in the
web UI.

Also took the chance to properly indent the encode_value function with
our indentation scheme.

Signed-off-by: Stefan Hanreich <s.hanreich@proxmox.com>
Tested-by: Hannes Duerr <h.duerr@proxmox.com>
---

diff --git a/src/PVE/Network/SDN.pm b/src/PVE/Network/SDN.pm
index c306527..3af09b5 100644
--- a/src/PVE/Network/SDN.pm
+++ b/src/PVE/Network/SDN.pm
@@ -241,12 +241,14 @@ sub generate_dhcp_config {
 sub encode_value {
     my ($type, $key, $value) = @_;
 
-    if ($key eq 'nodes' || $key eq 'exitnodes') {
-        if(ref($value) eq 'HASH') {
-            return join(',', sort keys(%$value));
-        } else {
-            return $value;
-        }
+    if ($key eq 'nodes' || $key eq 'exitnodes' || $key eq 'dhcp-range') {
+	if (ref($value) eq 'HASH') {
+	    return join(',', sort keys(%$value));
+	} elsif (ref($value) eq 'ARRAY') {
+	    return join(',', sort @$value);
+	} else {
+	    return $value;
+	}
     }
 
     return $value;