Pārlūkot izejas kodu

Fixes #6006: Fix VLAN group/site association for bulk prefix import

Jeremy Stretch 4 gadi atpakaļ
vecāks
revīzija
e4ecd2dae9
2 mainītis faili ar 9 papildinājumiem un 6 dzēšanām
  1. 1 0
      docs/release-notes/version-2.10.md
  2. 8 6
      netbox/ipam/forms.py

+ 1 - 0
docs/release-notes/version-2.10.md

@@ -9,6 +9,7 @@
 * [#5962](https://github.com/netbox-community/netbox/issues/5962) - Ensure consistent display of change log action labels
 * [#5966](https://github.com/netbox-community/netbox/issues/5966) - Skip Markdown reference link when tabbing through form fields
 * [#5977](https://github.com/netbox-community/netbox/issues/5977) - Correct validation of `RELEASE_CHECK_URL` config parameter
+* [#6006](https://github.com/netbox-community/netbox/issues/6006) - Fix VLAN group/site association for bulk prefix import
 
 ---
 

+ 8 - 6
netbox/ipam/forms.py

@@ -465,12 +465,14 @@ class PrefixCSVForm(CustomFieldModelCSVForm):
 
         if data:
 
-            # Limit vlan queryset by assigned site and group
-            params = {
-                f"site__{self.fields['site'].to_field_name}": data.get('site'),
-                f"group__{self.fields['vlan_group'].to_field_name}": data.get('vlan_group'),
-            }
-            self.fields['vlan'].queryset = self.fields['vlan'].queryset.filter(**params)
+            # Limit VLAN queryset by assigned site and/or group (if specified)
+            params = {}
+            if data.get('site'):
+                params[f"site__{self.fields['site'].to_field_name}"] = data.get('site')
+            if data.get('vlan_group'):
+                params[f"group__{self.fields['vlan_group'].to_field_name}"] = data.get('vlan_group')
+            if params:
+                self.fields['vlan'].queryset = self.fields['vlan'].queryset.filter(**params)
 
 
 class PrefixBulkEditForm(BootstrapMixin, AddRemoveTagsForm, CustomFieldBulkEditForm):