From fab90ecbe55b62a6ccc8da1199775c2355d663aa Mon Sep 17 00:00:00 2001
From: jeremystretch <jstretch@ns1.com>
Date: Wed, 23 Mar 2022 13:50:03 -0400
Subject: [PATCH] Closes #8803: Enable adding components from module view

---
 netbox/templates/dcim/device/base.html | 64 +++++---------------------
 netbox/templates/dcim/module.html      | 33 +++++++++++++
 2 files changed, 45 insertions(+), 52 deletions(-)

diff --git a/netbox/templates/dcim/device/base.html b/netbox/templates/dcim/device/base.html
index ff139ee4a..ea67154b1 100644
--- a/netbox/templates/dcim/device/base.html
+++ b/netbox/templates/dcim/device/base.html
@@ -16,79 +16,39 @@
 {% block extra_controls %}
     {% if perms.dcim.change_device %}
         <div class="dropdown">
-            <button id="add-device-components" type="button" class="btn btn-sm btn-primary dropdown-toggle" data-bs-toggle="dropdown" aria-expanded="false">
+            <button id="add-components" type="button" class="btn btn-sm btn-primary dropdown-toggle" data-bs-toggle="dropdown" aria-expanded="false">
                 <i class="mdi mdi-plus-thick" aria-hidden="true"></i> Add Components
             </button>
-            <ul class="dropdown-menu" aria-labeled-by="add-device-components">
+            <ul class="dropdown-menu" aria-labeled-by="add-components">
                 {% if perms.dcim.add_consoleport %}
-                    <li>
-                        <a class="dropdown-item" href="{% url 'dcim:consoleport_add' %}?device={{ object.pk }}&return_url={% url 'dcim:device_consoleports' pk=object.pk %}">
-                            Console Ports
-                        </a>
-                    </li>
+                    <li><a class="dropdown-item" href="{% url 'dcim:consoleport_add' %}?device={{ object.pk }}&return_url={% url 'dcim:device_consoleports' pk=object.pk %}">Console Ports</a></li>
                 {% endif %}
                 {% if perms.dcim.add_consoleserverport %}
-                    <li>
-                        <a class="dropdown-item" href="{% url 'dcim:consoleserverport_add' %}?device={{ object.pk }}&return_url={% url 'dcim:device_consoleserverports' pk=object.pk %}">
-                            Console Server Ports
-                        </a>
-                    </li>
+                    <li><a class="dropdown-item" href="{% url 'dcim:consoleserverport_add' %}?device={{ object.pk }}&return_url={% url 'dcim:device_consoleserverports' pk=object.pk %}">Console Server Ports</a></li>
                 {% endif %}
                 {% if perms.dcim.add_powerport %}
-                    <li>
-                        <a class="dropdown-item" href="{% url 'dcim:powerport_add' %}?device={{ object.pk }}&return_url={% url 'dcim:device_powerports' pk=object.pk %}">
-                            Power Ports
-                        </a>
-                    </li>
+                    <li><a class="dropdown-item" href="{% url 'dcim:powerport_add' %}?device={{ object.pk }}&return_url={% url 'dcim:device_powerports' pk=object.pk %}">Power Ports</a></li>
                 {% endif %}
                 {% if perms.dcim.add_poweroutlet %}
-                    <li>
-                        <a class="dropdown-item" href="{% url 'dcim:poweroutlet_add' %}?device={{ object.pk }}&return_url={% url 'dcim:device_poweroutlets' pk=object.pk %}">
-                            Power Outlets
-                        </a>
-                    </li>
+                    <li><a class="dropdown-item" href="{% url 'dcim:poweroutlet_add' %}?device={{ object.pk }}&return_url={% url 'dcim:device_poweroutlets' pk=object.pk %}">Power Outlets</a></li>
                 {% endif %}
                 {% if perms.dcim.add_interface %}
-                    <li>
-                        <a class="dropdown-item" href="{% url 'dcim:interface_add' %}?device={{ object.pk }}&return_url={% url 'dcim:device_interfaces' pk=object.pk %}">
-                            Interfaces
-                        </a>
-                    </li>
+                    <li><a class="dropdown-item" href="{% url 'dcim:interface_add' %}?device={{ object.pk }}&return_url={% url 'dcim:device_interfaces' pk=object.pk %}">Interfaces</a></li>
                 {% endif %}
                 {% if perms.dcim.add_frontport %}
-                    <li>
-                        <a class="dropdown-item" href="{% url 'dcim:frontport_add' %}?device={{ object.pk }}&return_url={% url 'dcim:device_frontports' pk=object.pk %}">
-                            Front Ports
-                        </a>
-                    </li>
+                    <li><a class="dropdown-item" href="{% url 'dcim:frontport_add' %}?device={{ object.pk }}&return_url={% url 'dcim:device_frontports' pk=object.pk %}">Front Ports</a></li>
                 {% endif %}
                 {% if perms.dcim.add_rearport %}
-                    <li>
-                        <a class="dropdown-item" href="{% url 'dcim:rearport_add' %}?device={{ object.pk }}&return_url={% url 'dcim:device_rearports' pk=object.pk %}">
-                            Rear Ports
-                        </a>
-                    </li>
+                    <li><a class="dropdown-item" href="{% url 'dcim:rearport_add' %}?device={{ object.pk }}&return_url={% url 'dcim:device_rearports' pk=object.pk %}">Rear Ports</a></li>
                 {% endif %}
                 {% if perms.dcim.add_devicebay %}
-                    <li>
-                        <a class="dropdown-item" href="{% url 'dcim:modulebay_add' %}?device={{ object.pk }}&return_url={% url 'dcim:device_modulebays' pk=object.pk %}">
-                            Module Bays
-                        </a>
-                    </li>
+                    <li><a class="dropdown-item" href="{% url 'dcim:modulebay_add' %}?device={{ object.pk }}&return_url={% url 'dcim:device_modulebays' pk=object.pk %}">Module Bays</a></li>
                 {% endif %}
                 {% if perms.dcim.add_devicebay %}
-                    <li>
-                        <a class="dropdown-item" href="{% url 'dcim:devicebay_add' %}?device={{ object.pk }}&return_url={% url 'dcim:device_devicebays' pk=object.pk %}">
-                            Device Bays
-                        </a>
-                    </li>
+                    <li><a class="dropdown-item" href="{% url 'dcim:devicebay_add' %}?device={{ object.pk }}&return_url={% url 'dcim:device_devicebays' pk=object.pk %}">Device Bays</a></li>
                 {% endif %}
                 {% if perms.dcim.add_inventoryitem %}
-                    <li>
-                        <a class="dropdown-item" href="{% url 'dcim:inventoryitem_add' %}?device={{ object.pk }}&return_url={% url 'dcim:device_inventory' pk=object.pk %}">
-                            Inventory Items
-                        </a>
-                    </li>
+                    <li><a class="dropdown-item" href="{% url 'dcim:inventoryitem_add' %}?device={{ object.pk }}&return_url={% url 'dcim:device_inventory' pk=object.pk %}">Inventory Items</a></li>
                 {% endif %}
             </ul>
         </div>
diff --git a/netbox/templates/dcim/module.html b/netbox/templates/dcim/module.html
index 632e37c2c..525d39534 100644
--- a/netbox/templates/dcim/module.html
+++ b/netbox/templates/dcim/module.html
@@ -10,6 +10,39 @@
   </li>
 {% endblock %}
 
+{% block extra_controls %}
+    {% if perms.dcim.change_module %}
+        <div class="dropdown">
+            <button id="add-components" type="button" class="btn btn-sm btn-primary dropdown-toggle" data-bs-toggle="dropdown" aria-expanded="false">
+                <i class="mdi mdi-plus-thick" aria-hidden="true"></i> Add Components
+            </button>
+            <ul class="dropdown-menu" aria-labeled-by="add-components">
+                {% if perms.dcim.add_consoleport %}
+                    <li><a class="dropdown-item" href="{% url 'dcim:consoleport_add' %}?device={{ object.device.pk }}&module={{ object.pk }}&return_url={% url 'dcim:device_consoleports' pk=object.pk %}">Console Ports</a></li>
+                {% endif %}
+                {% if perms.dcim.add_consoleserverport %}
+                    <li><a class="dropdown-item" href="{% url 'dcim:consoleserverport_add' %}?device={{ object.device.pk }}&module={{ object.pk }}&return_url={% url 'dcim:device_consoleserverports' pk=object.pk %}">Console Server Ports</a></li>
+                {% endif %}
+                {% if perms.dcim.add_powerport %}
+                    <li><a class="dropdown-item" href="{% url 'dcim:powerport_add' %}?device={{ object.device.pk }}&module={{ object.pk }}&return_url={% url 'dcim:device_powerports' pk=object.pk %}">Power Ports</a></li>
+                {% endif %}
+                {% if perms.dcim.add_poweroutlet %}
+                    <li><a class="dropdown-item" href="{% url 'dcim:poweroutlet_add' %}?device={{ object.device.pk }}&module={{ object.pk }}&return_url={% url 'dcim:device_poweroutlets' pk=object.pk %}">Power Outlets</a></li>
+                {% endif %}
+                {% if perms.dcim.add_interface %}
+                    <li><a class="dropdown-item" href="{% url 'dcim:interface_add' %}?device={{ object.device.pk }}&module={{ object.pk }}&return_url={% url 'dcim:device_interfaces' pk=object.pk %}">Interfaces</a></li>
+                {% endif %}
+                {% if perms.dcim.add_frontport %}
+                    <li><a class="dropdown-item" href="{% url 'dcim:frontport_add' %}?device={{ object.device.pk }}&module={{ object.pk }}&return_url={% url 'dcim:device_frontports' pk=object.pk %}">Front Ports</a></li>
+                {% endif %}
+                {% if perms.dcim.add_rearport %}
+                    <li><a class="dropdown-item" href="{% url 'dcim:rearport_add' %}?device={{ object.device.pk }}&module={{ object.pk }}&return_url={% url 'dcim:device_rearports' pk=object.pk %}">Rear Ports</a></li>
+                {% endif %}
+            </ul>
+        </div>
+    {% endif %}
+{% endblock %}
+
 {% block content %}
 <div class="row">
 	<div class="col col-md-6">