Merge pull request #37 from kpetremann/main

fix(api): safi disappearing for PATCH requests

Author: kpetremann

netbox_cmdb/netbox_cmdb/api/bgp/serializers.py

@@ -189,7 +189,7 @@ def update(self, instance, validated_data):
         bgp_peers["peer_b"] = DeviceBGPSession.objects.get(peer_b=instance)
         for peer, peer_instance in bgp_peers.items():
             # pop afi safi data for later creation/update
-            afi_safis_data[peer] = peers_data[peer].pop("afi_safis", {})
+            afi_safis_data[peer] = peers_data[peer].pop("afi_safis", None)
 
             # update peer instance with provided data
             peer_instance.device = peers_data[peer].get("device", peer_instance.device)
@@ -216,6 +216,11 @@ def update(self, instance, validated_data):
             )
             peer_instance.save()
 
+            # handle safi
+            if afi_safis_data[peer] is None:
+                # useful for PATCH requests: if the field is empty, we don't change anything
+                continue
+
             # get current list of afi safi
             current_afi_safis = list(AfiSafi.objects.filter(device_bgp_session=peer_instance))
             current_afi_safi_mapping = {