nautobot 2.4.10__py3-none-any.whl → 2.4.11__py3-none-any.whl

nautobot/dcim/views.py

@@ -16,6 +16,7 @@ from django.forms import (
     MultipleHiddenInput,
 )
 from django.shortcuts import get_object_or_404, HttpResponse, redirect, render
+from django.template.loader import render_to_string
 from django.urls import reverse
 from django.utils.encoding import iri_to_uri
 from django.utils.functional import cached_property
@@ -71,6 +72,7 @@ from nautobot.wireless.models import (
 from nautobot.wireless.tables import (
     ControllerManagedDeviceGroupRadioProfileAssignmentTable,
     ControllerManagedDeviceGroupWirelessNetworkAssignmentTable,
+    DeviceGroupWirelessNetworkTable,
     RadioProfileTable,
 )
 
@@ -106,6 +108,7 @@ from .models import (
     Module,
     ModuleBay,
     ModuleBayTemplate,
+    ModuleFamily,
     ModuleType,
     PathEndpoint,
     Platform,
@@ -251,21 +254,22 @@ class LocationTypeUIViewSet(NautobotUIViewSet):
 #
 
 
-class LocationListView(generic.ObjectListView):
-    queryset = Location.objects.all()
-    filterset = filters.LocationFilterSet
-    filterset_form = forms.LocationFilterForm
-    table = tables.LocationTable
-
-
-class LocationView(generic.ObjectView):
+class LocationUIViewSet(NautobotUIViewSet):
     # We aren't accessing tree fields anywhere so this is safe (note that `parent` itself is a normal foreign
     # key, not a tree field). If we ever do access tree fields, this will perform worse, because django will
     # automatically issue a second query (similar to behavior for
     # https://docs.djangoproject.com/en/3.2/ref/models/querysets/#django.db.models.query.QuerySet.only)
-    queryset = Location.objects.without_tree_fields().all()
+    queryset = Location.objects.without_tree_fields().select_related("location_type", "parent", "tenant")
+    filterset_class = filters.LocationFilterSet
+    filterset_form_class = forms.LocationFilterForm
+    table_class = tables.LocationTable
+    form_class = forms.LocationForm
+    bulk_update_form_class = forms.LocationBulkEditForm
+    serializer_class = serializers.LocationSerializer
 
     def get_extra_context(self, request, instance):
+        if instance is None:
+            return super().get_extra_context(request, instance)
         related_locations = (
             instance.descendants(include_self=True).restrict(request.user, "view").values_list("pk", flat=True)
         )
@@ -305,7 +309,6 @@ class LocationView(generic.ObjectView):
         )
 
         children_table = tables.LocationTable(children, hide_hierarchy_ui=True)
-
         paginate = {
             "paginator_class": EnhancedPaginator,
             "per_page": get_paginate_count(request),
@@ -323,34 +326,6 @@ class LocationView(generic.ObjectView):
         }
 
 
-class LocationEditView(generic.ObjectEditView):
-    queryset = Location.objects.all()
-    model_form = forms.LocationForm
-    template_name = "dcim/location_edit.html"
-
-
-class LocationDeleteView(generic.ObjectDeleteView):
-    queryset = Location.objects.all()
-
-
-class LocationBulkEditView(generic.BulkEditView):
-    queryset = Location.objects.select_related("location_type", "parent", "tenant")
-    filterset = filters.LocationFilterSet
-    table = tables.LocationTable
-    form = forms.LocationBulkEditForm
-
-
-class LocationBulkImportView(generic.BulkImportView):  # 3.0 TODO: remove, unused
-    queryset = Location.objects.all()
-    table = tables.LocationTable
-
-
-class LocationBulkDeleteView(generic.BulkDeleteView):
-    queryset = Location.objects.select_related("location_type", "parent", "tenant")
-    filterset = filters.LocationFilterSet
-    table = tables.LocationTable
-
-
 class MigrateLocationDataToContactView(generic.ObjectEditView):
     queryset = Location.objects.all()
     model_form = LocationMigrateDataToContactForm
@@ -1623,7 +1598,14 @@ class ModuleBayTemplateUIViewSet(
             return parent.display
         return ""
 
-    @action(detail=False, methods=["GET", "POST"], url_path="rename", url_name="bulk_rename")
+    @action(
+        detail=False,
+        methods=["GET", "POST"],
+        url_path="rename",
+        url_name="bulk_rename",
+        custom_view_base_action="change",
+        custom_view_additional_permissions=["dcim.change_modulebaytemplate"],
+    )
     def bulk_rename(self, request, *args, **kwargs):
         return self._bulk_rename(request, *args, **kwargs)
 
@@ -2393,35 +2375,6 @@ class ModuleUIViewSet(BulkComponentCreateUIViewSetMixin, NautobotUIViewSet):
     table_class = tables.ModuleTable
     component_model = None
 
-    def get_action(self):
-        if self.component_model:
-            method = self.request.method.lower()
-            if method == "get":
-                return "view"
-            else:
-                return "change"
-
-        return super().get_action()
-
-    def get_required_permission(self):
-        # TODO: standardize a pattern for permissions enforcement on custom actions
-        if self.component_model:
-            model = self.component_model
-            method = self.request.method.lower()
-            if method == "get":
-                component_action = "view"
-                permissions = [*self.get_permissions_for_model(model, [component_action]), "dcim.view_module"]
-            elif self.action.startswith("bulk_add"):
-                component_action = "add"
-                permissions = [*self.get_permissions_for_model(model, [component_action]), "dcim.change_module"]
-            else:
-                component_action = "change"
-                permissions = [*self.get_permissions_for_model(model, [component_action]), "dcim.change_module"]
-
-            return permissions
-
-        return super().get_required_permission()
-
     def get_extra_context(self, request, instance):
         context = super().get_extra_context(request, instance)
         if instance:
@@ -2448,7 +2401,13 @@ class ModuleUIViewSet(BulkComponentCreateUIViewSetMixin, NautobotUIViewSet):
 
         return active_parent_tab
 
-    @action(detail=True, url_path="console-ports", component_model=ConsolePort)
+    @action(
+        detail=True,
+        url_path="console-ports",
+        component_model=ConsolePort,
+        custom_view_base_action="view",
+        custom_view_additional_permissions=["dcim.view_consoleport"],
+    )
     def consoleports(self, request, *args, **kwargs):
         instance = self.get_object()
         consoleports = (
@@ -2467,7 +2426,13 @@ class ModuleUIViewSet(BulkComponentCreateUIViewSetMixin, NautobotUIViewSet):
             }
         )
 
-    @action(detail=True, url_path="console-server-ports", component_model=ConsoleServerPort)
+    @action(
+        detail=True,
+        url_path="console-server-ports",
+        component_model=ConsoleServerPort,
+        custom_view_base_action="view",
+        custom_view_additional_permissions=["dcim.view_consoleserverport"],
+    )
     def consoleserverports(self, request, *args, **kwargs):
         instance = self.get_object()
         consoleserverports = (
@@ -2490,7 +2455,13 @@ class ModuleUIViewSet(BulkComponentCreateUIViewSetMixin, NautobotUIViewSet):
             }
         )
 
-    @action(detail=True, url_path="power-ports", component_model=PowerPort)
+    @action(
+        detail=True,
+        url_path="power-ports",
+        component_model=PowerPort,
+        custom_view_base_action="view",
+        custom_view_additional_permissions=["dcim.view_powerport"],
+    )
     def powerports(self, request, *args, **kwargs):
         instance = self.get_object()
         powerports = (
@@ -2511,7 +2482,13 @@ class ModuleUIViewSet(BulkComponentCreateUIViewSetMixin, NautobotUIViewSet):
             }
         )
 
-    @action(detail=True, url_path="power-outlets", component_model=PowerOutlet)
+    @action(
+        detail=True,
+        url_path="power-outlets",
+        component_model=PowerOutlet,
+        custom_view_base_action="view",
+        custom_view_additional_permissions=["dcim.view_poweroutlet"],
+    )
     def poweroutlets(self, request, *args, **kwargs):
         instance = self.get_object()
         poweroutlets = (
@@ -2532,7 +2509,12 @@ class ModuleUIViewSet(BulkComponentCreateUIViewSetMixin, NautobotUIViewSet):
             }
         )
 
-    @action(detail=True, component_model=Interface)
+    @action(
+        detail=True,
+        component_model=Interface,
+        custom_view_base_action="view",
+        custom_view_additional_permissions=["dcim.view_interface"],
+    )
     def interfaces(self, request, *args, **kwargs):
         instance = self.get_object()
         interfaces = (
@@ -2558,7 +2540,13 @@ class ModuleUIViewSet(BulkComponentCreateUIViewSetMixin, NautobotUIViewSet):
             }
         )
 
-    @action(detail=True, url_path="front-ports", component_model=FrontPort)
+    @action(
+        detail=True,
+        url_path="front-ports",
+        component_model=FrontPort,
+        custom_view_base_action="view",
+        custom_view_additional_permissions=["dcim.view_frontport"],
+    )
     def frontports(self, request, *args, **kwargs):
         instance = self.get_object()
         frontports = instance.front_ports.restrict(request.user, "view").select_related("cable", "rear_port")
@@ -2575,7 +2563,13 @@ class ModuleUIViewSet(BulkComponentCreateUIViewSetMixin, NautobotUIViewSet):
             },
         )
 
-    @action(detail=True, url_path="rear-ports", component_model=RearPort)
+    @action(
+        detail=True,
+        url_path="rear-ports",
+        component_model=RearPort,
+        custom_view_base_action="view",
+        custom_view_additional_permissions=["dcim.view_rearport"],
+    )
     def rearports(self, request, *args, **kwargs):
         instance = self.get_object()
         rearports = instance.rear_ports.restrict(request.user, "view").select_related("cable")
@@ -2592,7 +2586,13 @@ class ModuleUIViewSet(BulkComponentCreateUIViewSetMixin, NautobotUIViewSet):
             }
         )
 
-    @action(detail=True, url_path="module-bays", component_model=ModuleBay)
+    @action(
+        detail=True,
+        url_path="module-bays",
+        component_model=ModuleBay,
+        custom_view_base_action="view",
+        custom_view_additional_permissions=["dcim.view_modulebay"],
+    )
     def modulebays(self, request, *args, **kwargs):
         instance = self.get_object()
         modulebays = instance.module_bays.restrict(request.user, "view").prefetch_related(
@@ -2615,6 +2615,8 @@ class ModuleUIViewSet(BulkComponentCreateUIViewSetMixin, NautobotUIViewSet):
         url_path="console-ports/add",
         url_name="bulk_add_consoleport",
         component_model=ConsolePort,
+        custom_view_base_action="change",
+        custom_view_additional_permissions=["dcim.add_consoleport"],
     )
     def bulk_add_consoleport(self, request, *args, **kwargs):
         return self._bulk_component_create(
@@ -2629,6 +2631,8 @@ class ModuleUIViewSet(BulkComponentCreateUIViewSetMixin, NautobotUIViewSet):
         url_path="console-server-ports/add",
         url_name="bulk_add_consoleserverport",
         component_model=ConsoleServerPort,
+        custom_view_base_action="change",
+        custom_view_additional_permissions=["dcim.add_consoleserverport"],
     )
     def bulk_add_consoleserverport(self, request, *args, **kwargs):
         return self._bulk_component_create(
@@ -2643,6 +2647,8 @@ class ModuleUIViewSet(BulkComponentCreateUIViewSetMixin, NautobotUIViewSet):
         url_path="power-ports/add",
         url_name="bulk_add_powerport",
         component_model=PowerPort,
+        custom_view_base_action="change",
+        custom_view_additional_permissions=["dcim.add_powerport"],
     )
     def bulk_add_powerport(self, request, *args, **kwargs):
         return self._bulk_component_create(
@@ -2657,6 +2663,8 @@ class ModuleUIViewSet(BulkComponentCreateUIViewSetMixin, NautobotUIViewSet):
         url_path="power-outlets/add",
         url_name="bulk_add_poweroutlet",
         component_model=PowerOutlet,
+        custom_view_base_action="change",
+        custom_view_additional_permissions=["dcim.add_poweroutlet"],
     )
     def bulk_add_poweroutlet(self, request, *args, **kwargs):
         return self._bulk_component_create(
@@ -2671,6 +2679,8 @@ class ModuleUIViewSet(BulkComponentCreateUIViewSetMixin, NautobotUIViewSet):
         url_path="interfaces/add",
         url_name="bulk_add_interface",
         component_model=Interface,
+        custom_view_base_action="change",
+        custom_view_additional_permissions=["dcim.add_interface"],
     )
     def bulk_add_interface(self, request, *args, **kwargs):
         return self._bulk_component_create(
@@ -2685,6 +2695,8 @@ class ModuleUIViewSet(BulkComponentCreateUIViewSetMixin, NautobotUIViewSet):
         url_path="rear-ports/add",
         url_name="bulk_add_rearport",
         component_model=RearPort,
+        custom_view_base_action="change",
+        custom_view_additional_permissions=["dcim.add_rearport"],
     )
     def bulk_add_rearport(self, request, *args, **kwargs):
         return self._bulk_component_create(
@@ -2699,6 +2711,8 @@ class ModuleUIViewSet(BulkComponentCreateUIViewSetMixin, NautobotUIViewSet):
         url_path="module-bays/add",
         url_name="bulk_add_modulebay",
         component_model=ModuleBay,
+        custom_view_base_action="change",
+        custom_view_additional_permissions=["dcim.add_modulebay"],
     )
     def bulk_add_modulebay(self, request, *args, **kwargs):
         return self._bulk_component_create(
@@ -3394,7 +3408,6 @@ class ModuleBayUIViewSet(ModuleBayCommonViewSetMixin, NautobotUIViewSet):
     serializer_class = serializers.ModuleBaySerializer
     table_class = tables.ModuleBayTable
     create_template_name = "dcim/device_component_add.html"
-
     object_detail_content = object_detail.ObjectDetailContent(
         panels=(
             object_detail.ObjectFieldsPanel(
@@ -3439,7 +3452,14 @@ class ModuleBayUIViewSet(ModuleBayCommonViewSetMixin, NautobotUIViewSet):
             return parent.display
         return ""
 
-    @action(detail=False, methods=["GET", "POST"], url_path="rename", url_name="bulk_rename")
+    @action(
+        detail=False,
+        methods=["GET", "POST"],
+        url_path="rename",
+        url_name="bulk_rename",
+        custom_view_base_action="change",
+        custom_view_additional_permissions=["dcim.change_modulebay"],
+    )
     def bulk_rename(self, request, *args, **kwargs):
         return self._bulk_rename(request, *args, **kwargs)
 
@@ -4159,6 +4179,28 @@ class PowerPanelUIViewSet(NautobotUIViewSet):
 #
 # Power feeds
 #
+
+
+class CustomPowerFeedKeyValueTablePanel(object_detail.KeyValueTablePanel):
+    """Custom panel to render PowerFeed utilization graph cleanly."""
+
+    def render_value(self, key, value, context):
+        if key == "Utilization (Allocated)":
+            if not value or not isinstance(value, tuple) or len(value) != 2:
+                return helpers.placeholder(None)
+            allocated, available = value
+            if available <= 0:
+                return f"{allocated}VA / {available}VA"
+            graph_html = render_to_string(
+                "utilities/templatetags/utilization_graph.html",
+                helpers.utilization_graph_raw_data(allocated, available),
+            )
+            return format_html("{}VA / {}VA {}", allocated, available, graph_html)
+
+        # Fall back to default behavior for everything else
+        return super().render_value(key, value, context)
+
+
 class PowerFeedUIViewSet(NautobotUIViewSet):
     bulk_update_form_class = forms.PowerFeedBulkEditForm
     filterset_class = filters.PowerFeedFilterSet
@@ -4168,6 +4210,137 @@ class PowerFeedUIViewSet(NautobotUIViewSet):
     serializer_class = serializers.PowerFeedSerializer
     table_class = tables.PowerFeedTable
 
+    object_detail_content = object_detail.ObjectDetailContent(
+        panels=(
+            CustomPowerFeedKeyValueTablePanel(
+                section=SectionChoices.LEFT_HALF,
+                weight=100,
+                label="Power Feed",
+                context_data_key="powerfeed_data",
+            ),
+            object_detail.ObjectFieldsPanel(
+                section=SectionChoices.LEFT_HALF,
+                weight=200,
+                label="Electrical Characteristics",
+                fields=["supply", "voltage", "amperage", "phase", "max_utilization"],
+                value_transforms={
+                    "voltage": [lambda v: f"{v}V" if v is not None else helpers.placeholder(v)],
+                    "amperage": [lambda a: f"{a}A" if a is not None else helpers.placeholder(a)],
+                    "max_utilization": [lambda p: f"{p}%" if p is not None else helpers.placeholder(p)],
+                },
+            ),
+            object_detail.KeyValueTablePanel(
+                section=SectionChoices.RIGHT_HALF,
+                weight=300,
+                label="Connection",
+                context_data_key="connection_data",
+            ),
+        )
+    )
+
+    def get_extra_context(self, request, instance):
+        context = super().get_extra_context(request, instance)
+        if not instance or self.action != "retrieve":
+            return context
+
+        context["powerfeed_data"] = {
+            "Power Panel": instance.power_panel,
+            "Rack": instance.rack,
+            "Type": self._get_type_html(instance),  # Render Type with HTML label
+            "Status": instance.status,
+            "Connected Device": self._get_connected_device_html(instance),
+            "Utilization (Allocated)": self._get_utilization_data(instance),
+        }
+
+        context["connection_data"] = self._get_connection_data(request, instance)
+        return context
+
+    def _get_type_html(self, instance):
+        """
+        Render the PowerFeed type as a label with the appropriate CSS class.
+        """
+
+        type_class = instance.get_type_class()
+        return format_html('<span class="label label-{}">{}</span>', type_class, instance.get_type_display())
+
+    def _get_connected_device_html(self, instance):
+        endpoint = getattr(instance, "connected_endpoint", None)
+        if endpoint and endpoint.parent:
+            parent = helpers.hyperlinked_object(endpoint.parent)
+            return format_html("{} ({})", parent, endpoint)
+        return None
+
+    def _get_utilization_data(self, instance):
+        endpoint = getattr(instance, "connected_endpoint", None)
+        if not endpoint or not hasattr(endpoint, "get_power_draw"):
+            return None
+        utilization = endpoint.get_power_draw()
+        if not utilization or "allocated" not in utilization:
+            return None
+        allocated = utilization["allocated"]
+        available = instance.available_power or 0
+        return (allocated, available)
+
+    def _get_connection_data(self, request, instance):
+        if not instance:
+            return {}
+
+        if instance.cable:
+            trace_url = reverse("dcim:powerfeed_trace", kwargs={"pk": instance.pk})
+            cable_html = format_html(
+                '{} <a href="{}" class="btn btn-primary btn-xs" title="Trace">'
+                '<i class="mdi mdi-transit-connection-variant"></i></a>',
+                helpers.hyperlinked_object(instance.cable),
+                trace_url,
+            )
+
+            endpoint = getattr(instance, "connected_endpoint", None)
+            endpoint_data = {}
+
+            if endpoint:
+                endpoint_obj = getattr(endpoint, "device", None) or getattr(endpoint, "module", None)
+                # Removed the unused 'path' variable
+                endpoint_data = {
+                    "Device" if getattr(endpoint, "device", None) else "Module": endpoint_obj,
+                    "Power Port": endpoint,
+                    "Type": endpoint.get_type_display() if hasattr(endpoint, "get_type_display") else None,
+                    "Description": endpoint.description,
+                    "Path Status": self._get_path_status_html(instance),  # Render Path Status dynamically
+                }
+
+            return {
+                "Cable": cable_html,
+                **endpoint_data,
+            }
+
+        if request.user.has_perm("dcim.add_cable"):
+            connect_url = (
+                reverse(
+                    "dcim:powerfeed_connect",
+                    kwargs={"termination_a_id": instance.pk, "termination_b_type": "power-port"},
+                )
+                + f"?return_url={instance.get_absolute_url()}"
+            )
+            connect_link = format_html(
+                '<a href="{}" class="btn btn-primary btn-sm pull-right">'
+                '<span class="mdi mdi-ethernet-cable" aria-hidden="true"></span> Connect</a>',
+                connect_url,
+            )
+            return {"Connection": format_html("Not connected {}", connect_link)}
+
+        return {"Connection": "Not connected"}
+
+    def _get_path_status_html(self, instance):
+        """
+        Render the Path Status as a label based on the path status (active or not).
+        """
+        path_status = (
+            '<span class="label label-success">Reachable</span>'
+            if getattr(instance, "path", None) and instance.path.is_active
+            else '<span class="label label-danger">Not Reachable</span>'
+        )
+        return format_html(path_status)  # Safely render HTML
+
 
 class DeviceRedundancyGroupUIViewSet(NautobotUIViewSet):
     bulk_update_form_class = forms.DeviceRedundancyGroupBulkEditForm
@@ -4218,38 +4391,35 @@ class InterfaceRedundancyGroupUIViewSet(NautobotUIViewSet):
     table_class = tables.InterfaceRedundancyGroupTable
     lookup_field = "pk"
 
-    def get_extra_context(self, request, instance):
-        """Return additional panels for display."""
-        context = super().get_extra_context(request, instance)
-        if instance and self.action == "retrieve":
-            interface_table = self._get_interface_redundancy_groups_table(request, instance)
-            context["interface_table"] = interface_table
-        return context
-
-    def _get_interface_redundancy_groups_table(self, request, instance):
-        """Return a table of assigned Interfaces."""
-        queryset = instance.interface_redundancy_group_associations.restrict(request.user)
-        queryset = queryset.prefetch_related("interface")
-        queryset = queryset.order_by("priority")
-        column_sequence = (
-            "interface__device",
-            "interface",
-            "priority",
-            "interface__status",
-            "interface__enabled",
-            "interface__ip_addresses",
-            "interface__type",
-            "interface__description",
-            "interface__label",
-        )
-        table = tables.InterfaceRedundancyGroupAssociationTable(
-            data=queryset,
-            sequence=column_sequence,
-            orderable=False,
-        )
-        for column_name in column_sequence:
-            table.columns.show(column_name)
-        return table
+    object_detail_content = object_detail.ObjectDetailContent(
+        panels=(
+            object_detail.ObjectFieldsPanel(
+                weight=100,
+                section=SectionChoices.LEFT_HALF,
+                fields="__all__",
+            ),
+            object_detail.ObjectsTablePanel(
+                weight=200,
+                section=SectionChoices.FULL_WIDTH,
+                table_class=tables.InterfaceRedundancyGroupAssociationTable,
+                table_attribute="interface_redundancy_group_associations",
+                prefetch_related_fields=["interface"],
+                order_by_fields=["priority"],
+                table_title="Interfaces",
+                related_field_name="interface_redundancy_group",
+                include_columns=[
+                    "interface__device",
+                    "interface",
+                    "interface__status",
+                    "interface__enabled",
+                    "interface__ip_addresses",
+                    "interface__type",
+                    "interface__description",
+                    "interface__label",
+                ],
+            ),
+        ),
+    )
 
 
 class InterfaceRedundancyGroupAssociationUIViewSet(ObjectEditViewMixin, ObjectDestroyViewMixin):
@@ -4259,6 +4429,51 @@ class InterfaceRedundancyGroupAssociationUIViewSet(ObjectEditViewMixin, ObjectDe
     lookup_field = "pk"
 
 
+class ModuleFamilyUIViewSet(NautobotUIViewSet):
+    """ViewSet for the ModuleFamily model."""
+
+    filterset_class = filters.ModuleFamilyFilterSet
+    filterset_form_class = forms.ModuleFamilyFilterForm
+    form_class = forms.ModuleFamilyForm
+    bulk_update_form_class = forms.ModuleFamilyBulkEditForm
+    queryset = ModuleFamily.objects.all()
+    serializer_class = serializers.ModuleFamilySerializer
+    table_class = tables.ModuleFamilyTable
+    lookup_field = "pk"
+
+    def get_extra_context(self, request, instance):
+        context = super().get_extra_context(request, instance)
+        if not instance:
+            return context
+
+        if self.action == "retrieve":
+            module_types = (
+                ModuleType.objects.restrict(request.user, "view")
+                .filter(module_family=instance)
+                .select_related("manufacturer")
+            )
+            module_type_table = tables.ModuleTypeTable(module_types, orderable=False)
+
+            module_bays = (
+                ModuleBay.objects.restrict(request.user, "view")
+                .filter(module_family=instance)
+                .select_related("parent_device", "parent_module")
+            )
+            module_bay_table = tables.ModuleBayTable(module_bays, orderable=False)
+
+            paginate = {
+                "paginator_class": EnhancedPaginator,
+                "per_page": get_paginate_count(request),
+            }
+            RequestConfig(request, paginate).configure(module_type_table)
+            RequestConfig(request, paginate).configure(module_bay_table)
+
+            context["module_type_table"] = module_type_table
+            context["module_bay_table"] = module_bay_table
+
+        return context
+
+
 class DeviceFamilyUIViewSet(NautobotUIViewSet):
     filterset_class = filters.DeviceFamilyFilterSet
     filterset_form_class = forms.DeviceFamilyFilterForm
@@ -4308,7 +4523,6 @@ class SoftwareImageFileUIViewSet(NautobotUIViewSet):
     queryset = SoftwareImageFile.objects.all()
     serializer_class = serializers.SoftwareImageFileSerializer
     table_class = tables.SoftwareImageFileTable
-
     object_detail_content = object_detail.ObjectDetailContent(
         panels=(
             object_detail.ObjectFieldsPanel(
@@ -4392,19 +4606,42 @@ class SoftwareImageFileUIViewSet(NautobotUIViewSet):
         ),
     )
 
-    @action(detail=True, url_path="device-types", url_name="device_types")
+    @action(
+        detail=True,
+        url_path="device-types",
+        url_name="device_types",
+        custom_view_base_action="view",
+        custom_view_additional_permissions=["dcim.view_devicetype"],
+    )
     def device_types(self, request, *args, **kwargs):
         return Response({})
 
-    @action(detail=True, url_path="devices")
+    @action(
+        detail=True,
+        url_path="devices",
+        custom_view_base_action="view",
+        custom_view_additional_permissions=["dcim.view_device"],
+    )
     def devices(self, request, *args, **kwargs):
         return Response({})
 
-    @action(detail=True, url_path="inventory-items", url_name="inventory_items")
+    @action(
+        detail=True,
+        url_path="inventory-items",
+        url_name="inventory_items",
+        custom_view_base_action="view",
+        custom_view_additional_permissions=["dcim.view_inventoryitem"],
+    )
     def inventory_items(self, request, *args, **kwargs):
         return Response({})
 
-    @action(detail=True, url_path="virtual-machines", url_name="virtual_machines")
+    @action(
+        detail=True,
+        url_path="virtual-machines",
+        url_name="virtual_machines",
+        custom_view_base_action="view",
+        custom_view_additional_permissions=["virtualization.view_virtualmachine"],
+    )
     def virtual_machines(self, request, *args, **kwargs):
         return Response({})
 
@@ -4470,7 +4707,14 @@ class ControllerUIViewSet(NautobotUIViewSet):
 
         return context
 
-    @action(detail=True, url_path="wireless-networks", url_name="wirelessnetworks", methods=["get"])
+    @action(
+        detail=True,
+        url_path="wireless-networks",
+        url_name="wirelessnetworks",
+        methods=["get"],
+        custom_view_base_action="view",
+        custom_view_additional_permissions=["wireless.view_controllermanageddevicegroupwirelessnetworkassignment"],
+    )
     def wirelessnetworks(self, request, *args, **kwargs):
         instance = self.get_object()
         controller_managed_device_groups = instance.controller_managed_device_groups.restrict(
@@ -4495,10 +4739,6 @@ class ControllerUIViewSet(NautobotUIViewSet):
             }
         )
 
-    def get_action(self):
-        "Treat Wireless Networks as the same detail view for permission purposes."
-        return "view" if self.action == "wirelessnetworks" else super().get_action()
-
 
 class ControllerManagedDeviceGroupUIViewSet(NautobotUIViewSet):
     filterset_class = filters.ControllerManagedDeviceGroupFilterSet
@@ -4509,42 +4749,88 @@ class ControllerManagedDeviceGroupUIViewSet(NautobotUIViewSet):
     serializer_class = serializers.ControllerManagedDeviceGroupSerializer
     table_class = tables.ControllerManagedDeviceGroupTable
     template_name = "dcim/controllermanageddevicegroup_create.html"
+    object_detail_content = object_detail.ObjectDetailContent(
+        panels=(
+            object_detail.ObjectFieldsPanel(
+                section=SectionChoices.LEFT_HALF,
+                weight=100,
+                fields="__all__",
+                value_transforms={
+                    "capabilities": [helpers.label_list],
+                },
+            ),
+            object_detail.ObjectsTablePanel(
+                section=SectionChoices.FULL_WIDTH,
+                weight=100,
+                table_class=tables.DeviceTable,
+                table_filter="controller_managed_device_group",
+                add_button_route=None,
+            ),
+        ),
+        extra_tabs=(
+            object_detail.DistinctViewTab(
+                weight=800,
+                tab_id="wireless_networks",
+                label="Wireless Networks",
+                url_name="dcim:controllermanageddevicegroup_wireless_networks",
+                related_object_attribute="wireless_network_assignments",
+                panels=(
+                    object_detail.ObjectsTablePanel(
+                        section=SectionChoices.FULL_WIDTH,
+                        weight=100,
+                        table_title="Wireless Networks",
+                        table_class=DeviceGroupWirelessNetworkTable,
+                        table_filter="controller_managed_device_group",
+                        related_field_name="controller_managed_device_groups",
+                        tab_id="wireless_networks",
+                        add_button_route=None,
+                        exclude_columns=["controller_managed_device_group", "controller"],
+                    ),
+                ),
+            ),
+            object_detail.DistinctViewTab(
+                weight=900,
+                tab_id="radio_profiles",
+                label="Radio Profiles",
+                url_name="dcim:controllermanageddevicegroup_radio_profiles",
+                related_object_attribute="radio_profiles",
+                panels=(
+                    object_detail.ObjectsTablePanel(
+                        section=SectionChoices.FULL_WIDTH,
+                        weight=100,
+                        table_title="Radio Profiles",
+                        table_class=RadioProfileTable,
+                        table_filter="controller_managed_device_groups",
+                        tab_id="radio_profiles",
+                        add_button_route=None,
+                    ),
+                ),
+            ),
+        ),
+    )
 
-    def get_extra_context(self, request, instance):
-        context = super().get_extra_context(request, instance)
-
-        if self.action == "retrieve" and instance:
-            devices = instance.devices.restrict(request.user)
-            devices_table = tables.DeviceTable(devices)
-
-            paginate = {
-                "paginator_class": EnhancedPaginator,
-                "per_page": get_paginate_count(request),
-            }
-            RequestConfig(request, paginate).configure(devices_table)
-
-            context["devices_table"] = devices_table
+    @action(
+        detail=True,
+        url_path="wireless-networks",
+        url_name="wireless_networks",
+        custom_view_base_action="view",
+        custom_view_additional_permissions=["wireless.view_controllermanageddevicegroupwirelessnetworkassignment"],
+    )
+    def wireless_networks(self, request, *args, **kwargs):
+        return Response({})
 
-            # Wireless Networks
-            wireless_networks = instance.wireless_network_assignments.restrict(request.user, "view")
-            wireless_networks_table = ControllerManagedDeviceGroupWirelessNetworkAssignmentTable(wireless_networks)
-            wireless_networks_table.columns.hide("controller_managed_device_group")
-            wireless_networks_table.columns.hide("controller")
-            RequestConfig(
-                request, paginate={"paginator_class": EnhancedPaginator, "per_page": get_paginate_count(request)}
-            ).configure(wireless_networks_table)
-            context["wireless_networks_table"] = wireless_networks_table
-            context["wireless_networks_count"] = wireless_networks.count()
-
-            # Radio Profiles
-            radio_profiles = instance.radio_profiles.restrict(request.user, "view")
-            radio_profiles_table = RadioProfileTable(radio_profiles)
-            RequestConfig(
-                request, paginate={"paginator_class": EnhancedPaginator, "per_page": get_paginate_count(request)}
-            ).configure(radio_profiles_table)
-            context["radio_profiles_table"] = radio_profiles_table
-            context["radio_profiles_count"] = radio_profiles.count()
+    @action(
+        detail=True,
+        url_path="radio-profiles",
+        url_name="radio_profiles",
+        custom_view_base_action="view",
+        custom_view_additional_permissions=["wireless.view_radioprofile"],
+    )
+    def radio_profiles(self, request, *args, **kwargs):
+        return Response({})
 
+    def get_extra_context(self, request, instance):
+        context = super().get_extra_context(request, instance)
         if self.action in ["create", "update"]:
             context["wireless_networks"] = ControllerManagedDeviceGroupWirelessNetworkFormSet(
                 instance=instance,