Browse Source

Fixed bug when assigning Device to no rack race

Jeremy Stretch 9 years ago
parent
commit
cea0b2de33
1 changed files with 6 additions and 3 deletions
  1. 6 3
      netbox/dcim/forms.py

+ 6 - 3
netbox/dcim/forms.py

@@ -307,10 +307,13 @@ class DeviceForm(forms.ModelForm, BootstrapMixin):
             self.fields['rack'].choices = []
             self.fields['rack'].choices = []
 
 
         # Rack position
         # Rack position
+        face = self.data.get('face')
+        if face == '':
+            face = None
         try:
         try:
-            if self.is_bound and self.data.get('rack') and self.data.get('face') is not None:
-                position_choices = Rack.objects.get(pk=self.data['rack']).get_rack_units(face=self.data.get('face'))
-            elif self.initial.get('rack') and self.initial.get('face') is not None:
+            if self.is_bound and self.data.get('rack') and face is not None:
+                position_choices = Rack.objects.get(pk=self.data['rack']).get_rack_units(face=face)
+            elif self.initial.get('rack') and face is not None:
                 position_choices = Rack.objects.get(pk=self.initial['rack']).get_rack_units(face=self.initial.get('face'))
                 position_choices = Rack.objects.get(pk=self.initial['rack']).get_rack_units(face=self.initial.get('face'))
             else:
             else:
                 position_choices = []
                 position_choices = []