From 37cb296c2301b9a6dfda972ea3d7b0a931031699 Mon Sep 17 00:00:00 2001 From: dansheps Date: Thu, 9 Aug 2018 11:25:35 -0500 Subject: [PATCH] Fixes 2344 - VM not in a Site throws an AttributeError when assigning a VLAN to an interface --- netbox/dcim/forms.py | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/netbox/dcim/forms.py b/netbox/dcim/forms.py index 521c3c858..74dee89cd 100644 --- a/netbox/dcim/forms.py +++ b/netbox/dcim/forms.py @@ -1817,12 +1817,13 @@ class InterfaceAssignVLANsForm(BootstrapMixin, forms.ModelForm): vlan_choices.append((parent.site.name, [(vlan.pk, vlan) for vlan in site_vlans])) # Add grouped site VLANs - for group in VLANGroup.objects.filter(site=parent.site): - site_group_vlans = VLAN.objects.filter(group=group).exclude(pk__in=assigned_vlans) - vlan_choices.append(( - '{} / {}'.format(group.site.name, group.name), - [(vlan.pk, vlan) for vlan in site_group_vlans] - )) + if parent.site: + for group in VLANGroup.objects.filter(site=parent.site): + site_group_vlans = VLAN.objects.filter(group=group).exclude(pk__in=assigned_vlans) + vlan_choices.append(( + '{} / {}'.format(group.site.name, group.name), + [(vlan.pk, vlan) for vlan in site_group_vlans] + )) self.fields['vlans'].choices = vlan_choices