Misc cleanup

netbox/dcim/tables/devices.py

@@ -544,6 +544,11 @@ class Meta(CableTerminationTable.Meta, DeviceComponentTable.Meta):
 
 
 class BaseInterfaceTable(NetBoxTable):
+    name = tables.Column(
+        verbose_name=_('Name'),
+        linkify=True,
+        order_by=('_name',)
+    )
     enabled = columns.BooleanColumn(
         verbose_name=_('Enabled'),
     )
@@ -591,7 +596,7 @@ def value_tagged_vlans(self, value):
         return ",".join([str(obj) for obj in value.all()])
 
 
-class InterfaceTable(ModularDeviceComponentTable, BaseInterfaceTable, PathEndpointTable):
+class InterfaceTable(BaseInterfaceTable, ModularDeviceComponentTable, PathEndpointTable):
     device = tables.Column(
         verbose_name=_('Device'),
         linkify={

netbox/dcim/tables/devicetypes.py

@@ -218,6 +218,10 @@ class Meta(ComponentTemplateTable.Meta):
 
 
 class InterfaceTemplateTable(ComponentTemplateTable):
+    name = tables.Column(
+        verbose_name=_('Name'),
+        order_by=('_name',)
+    )
     enabled = columns.BooleanColumn(
         verbose_name=_('Enabled'),
     )

netbox/dcim/views.py

@@ -688,8 +688,7 @@ def get(self, request):
         sort = request.GET.get('sort', 'name')
         if sort not in ORDERING_CHOICES:
             sort = 'name'
-        sort_field = sort  # Use natural ordering
-        racks = racks.order_by(sort_field)
+        racks = racks.order_by(sort)
 
         # Pagination
         per_page = get_paginate_count(request)

netbox/utilities/fields.py

@@ -5,7 +5,6 @@
 from django.utils.safestring import mark_safe
 from django.utils.translation import gettext_lazy as _
 
-from utilities.ordering import naturalize_interface
 from .forms.widgets import ColorSelect
 from .validators import ColorValidator
 
@@ -40,7 +39,7 @@ class NaturalOrderingField(models.CharField):
     """
     description = "Stores a representation of its target field suitable for natural ordering"
 
-    def __init__(self, target_field, naturalize_function=naturalize_interface, *args, **kwargs):
+    def __init__(self, target_field, naturalize_function, *args, **kwargs):
         self.target_field = target_field
         self.naturalize_function = naturalize_function
         super().__init__(*args, **kwargs)

netbox/virtualization/models/virtualmachines.py

@@ -267,6 +267,11 @@ class ComponentModel(NetBoxModel):
         on_delete=models.CASCADE,
         related_name='%(class)ss'
     )
+    name = models.CharField(
+        verbose_name=_('name'),
+        max_length=64,
+        db_collation="natural_sort"
+    )
     description = models.CharField(
         verbose_name=_('description'),
         max_length=200,
@@ -311,12 +316,6 @@ class VMInterface(ComponentModel, BaseInterface, TrackingModelMixin):
         on_delete=models.CASCADE,
         related_name='interfaces'  # Override ComponentModel
     )
-    _name = NaturalOrderingField(
-        target_field='name',
-        naturalize_function=naturalize_interface,
-        max_length=100,
-        blank=True
-    )
     ip_addresses = GenericRelation(
         to='ipam.IPAddress',
         content_type_field='assigned_object_type',
@@ -405,11 +404,6 @@ def l2vpn_termination(self):
 
 
 class VirtualDisk(ComponentModel, TrackingModelMixin):
-    name = models.CharField(
-        verbose_name=_('name'),
-        max_length=64,
-        db_collation="natural_sort"
-    )
     size = models.PositiveIntegerField(
         verbose_name=_('size (MB)'),
     )