#4867: Misc cleanup

netbox/dcim/forms/common.py

@@ -23,7 +23,8 @@ class InterfaceCommonForm(forms.Form):
     primary_mac_address = DynamicModelChoiceField(
         queryset=MACAddress.objects.all(),
         label=_('Primary MAC address'),
-        required=False
+        required=False,
+        quick_add=True
     )
 
     def __init__(self, *args, **kwargs):

netbox/dcim/forms/model_forms.py

@@ -1758,11 +1758,17 @@ class MACAddressForm(NetBoxModelForm):
         label=_('Interface'),
         queryset=Interface.objects.all(),
         required=False,
+        context={
+            'parent': 'device',
+        },
     )
     vminterface = DynamicModelChoiceField(
         label=_('VM Interface'),
         queryset=VMInterface.objects.all(),
         required=False,
+        context={
+            'parent': 'virtual_machine',
+        },
     )
 
     fieldsets = (

netbox/netbox/navigation/menu.py

@@ -88,12 +88,6 @@ DEVICES_MENU = Menu(
                 get_model_item('dcim', 'manufacturer', _('Manufacturers')),
             ),
         ),
-        MenuGroup(
-            label=_('Addressing'),
-            items=(
-                get_model_item('dcim', 'macaddress', _('MAC Addresses')),
-            ),
-        ),
         MenuGroup(
             label=_('Device Components'),
             items=(
@@ -110,6 +104,12 @@ DEVICES_MENU = Menu(
                 get_model_item('dcim', 'inventoryitemrole', _('Inventory Item Roles')),
             ),
         ),
+        MenuGroup(
+            label=_('Addressing'),
+            items=(
+                get_model_item('dcim', 'macaddress', _('MAC Addresses')),
+            ),
+        ),
     ),
 )
 

netbox/templates/dcim/interface.html

@@ -124,8 +124,8 @@
           <tr>
             <th scope="row">{% trans "MAC Address" %}</th>
             <td>
-              {% if object.mac_address %}
+              {% if object.primary_mac_address %}
-                <span class="font-monospace">{{ object.mac_address }}</span>
+                <span class="font-monospace">{{ object.primary_mac_address|linkify }}</span>
                 <span class="badge text-bg-primary">{% trans "Primary" %}</span>
               {% else %}
                 {{ ''|placeholder }}