From 06ae424b80f2fca5b6940f55c63084b0fede748c Mon Sep 17 00:00:00 2001
From: Jeremy Stretch <stretch@packetlife.net>
Date: Wed, 1 Jul 2020 15:12:05 -0400
Subject: [PATCH] #4416: Add bulk rename view for InventoryItem

---
 netbox/dcim/urls.py                          |   2 +-
 netbox/dcim/views.py                         |   4 +
 netbox/templates/dcim/device_inventory.html  | 108 +++++++++----------
 netbox/templates/dcim/inc/inventoryitem.html |  33 ++++--
 4 files changed, 86 insertions(+), 61 deletions(-)

diff --git a/netbox/dcim/urls.py b/netbox/dcim/urls.py
index 831d40402..7af91f0ae 100644
--- a/netbox/dcim/urls.py
+++ b/netbox/dcim/urls.py
@@ -320,7 +320,7 @@ urlpatterns = [
     path('inventory-items/add/', views.InventoryItemCreateView.as_view(), name='inventoryitem_add'),
     path('inventory-items/import/', views.InventoryItemBulkImportView.as_view(), name='inventoryitem_import'),
     path('inventory-items/edit/', views.InventoryItemBulkEditView.as_view(), name='inventoryitem_bulk_edit'),
-    # TODO: Bulk rename view for InventoryItems
+    path('inventory-items/rename/', views.InventoryItemBulkRenameView.as_view(), name='inventoryitem_bulk_rename'),
     path('inventory-items/delete/', views.InventoryItemBulkDeleteView.as_view(), name='inventoryitem_bulk_delete'),
     path('inventory-items/<int:pk>/', views.InventoryItemView.as_view(), name='inventoryitem'),
     path('inventory-items/<int:pk>/edit/', views.InventoryItemEditView.as_view(), name='inventoryitem_edit'),
diff --git a/netbox/dcim/views.py b/netbox/dcim/views.py
index 4456e408d..101c3ea6e 100644
--- a/netbox/dcim/views.py
+++ b/netbox/dcim/views.py
@@ -1761,6 +1761,10 @@ class InventoryItemBulkEditView(BulkEditView):
     form = forms.InventoryItemBulkEditForm
 
 
+class InventoryItemBulkRenameView(BulkRenameView):
+    queryset = InventoryItem.objects.all()
+
+
 class InventoryItemBulkDeleteView(BulkDeleteView):
     queryset = InventoryItem.objects.prefetch_related('device', 'manufacturer')
     table = tables.InventoryItemTable
diff --git a/netbox/templates/dcim/device_inventory.html b/netbox/templates/dcim/device_inventory.html
index 1a7e5d793..69afbb6a1 100644
--- a/netbox/templates/dcim/device_inventory.html
+++ b/netbox/templates/dcim/device_inventory.html
@@ -5,61 +5,61 @@
 
 {% block content %}
     <div class="row">
-        <div class="col-md-4">
-            <div class="panel panel-default">
-                <div class="panel-heading">
-                    <strong>Chassis</strong>
-                </div>
-                <table class="table table-hover panel-body attr-table">
-                    <tr>
-                        <td>Model</td>
-                        <td>{{ device.device_type.display_name }}</td>
-                    </tr>
-                    <tr>
-                        <td>Serial Number</td>
-                        <td><span>{{ device.serial|placeholder }}</span></td>
-                    </tr>
-                    <tr>
-                        <td>Asset Tag</td>
-                        <td><span>{{ device.asset_tag|placeholder }}</span></td>
-                    </tr>
-                </table>
-            </div>
-        </div>
-        <div class="col-md-8">
-            <div class="panel panel-default">
-                <div class="panel-heading">
-                    <strong>Hardware</strong>
-                </div>
-                <table class="table table-hover table-condensed panel-body" id="hardware">
-                    <thead>
-                        <tr>
-                            <th>Name</th>
-                            <th></th>
-                            <th>Manufacturer</th>
-                            <th>Part ID</th>
-                            <th>Serial Number</th>
-                            <th>Asset Tag</th>
-                            <th>Description</th>
-                            <th></th>
-                        </tr>
-                    </thead>
-                    <tbody>
-                        {% for item in inventory_items %}
-                            {% with template_name='dcim/inc/inventoryitem.html' indent=0 %}
-                                {% include template_name %}
-                            {% endwith %}
-                        {% endfor %}
-                    </tbody>
-                </table>
-                {% if perms.dcim.add_inventoryitem %}
-                    <div class="panel-footer text-right noprint">
-                        <a href="{% url 'dcim:inventoryitem_add' %}?device={{ device.pk }}&return_url={% url 'dcim:device_inventory' pk=device.pk %}" class="btn btn-primary btn-xs">
-                            <span class="fa fa-plus" aria-hidden="true"></span> Add Inventory Item
-                        </a>
+        <div class="col-md-12">
+            <form method="post">
+                {% csrf_token %}
+                <div class="panel panel-default">
+                    <div class="panel-heading">
+                        <strong>Inventory Items</strong>
                     </div>
-                {% endif %}
-            </div>
+                    <table class="table table-hover table-condensed panel-body" id="hardware">
+                        <thead>
+                            <tr>
+                                {% if perms.dcim.change_inventoryitem or perms.dcim.delete_inventoryitem %}
+                                    <th class="pk"><input type="checkbox" class="toggle" title="Toggle all" /></th>
+                                {% endif %}
+                                <th>Name</th>
+                                <th>Manufacturer</th>
+                                <th>Part ID</th>
+                                <th>Serial Number</th>
+                                <th>Asset Tag</th>
+                                <th>Discovered</th>
+                                <th>Description</th>
+                                <th></th>
+                            </tr>
+                        </thead>
+                        <tbody>
+                            {% for item in inventory_items %}
+                                {% with template_name='dcim/inc/inventoryitem.html' indent=0 %}
+                                    {% include template_name %}
+                                {% endwith %}
+                            {% endfor %}
+                        </tbody>
+                    </table>
+                    <div class="panel-footer noprint">
+                        {% if inventory_items and perms.dcim.change_inventoryitem %}
+                            <button type="submit" name="_rename" formaction="{% url 'dcim:inventoryitem_bulk_rename' %}?return_url={% url 'dcim:device_inventory' pk=device.pk %}" class="btn btn-warning btn-xs">
+                                <span class="glyphicon glyphicon-pencil" aria-hidden="true"></span> Rename
+                            </button>
+                            <button type="submit" name="_edit" formaction="{% url 'dcim:inventoryitem_bulk_edit' %}?device={{ device.pk }}&return_url={% url 'dcim:device_inventory' pk=device.pk %}" class="btn btn-warning btn-xs">
+                                <span class="glyphicon glyphicon-pencil" aria-hidden="true"></span> Edit
+                            </button>
+                        {% endif %}
+                        {% if inventory_items and perms.dcim.delete_inventoryitem %}
+                            <button type="submit" name="_delete" formaction="{% url 'dcim:inventoryitem_bulk_delete' %}?return_url={% url 'dcim:device_inventory' pk=device.pk %}" class="btn btn-danger btn-xs">
+                                <span class="glyphicon glyphicon-trash" aria-hidden="true"></span> Delete
+                            </button>
+                        {% endif %}
+                        {% if perms.dcim.add_inventoryitem %}
+                            <div class="pull-right">
+                                <a href="{% url 'dcim:inventoryitem_add' %}?device={{ device.pk }}&return_url={% url 'dcim:device_inventory' pk=device.pk %}" class="btn btn-primary btn-xs">
+                                    <span class="fa fa-plus" aria-hidden="true"></span> Add Inventory Item
+                                </a>
+                            </div>
+                        {% endif %}
+                    </div>
+                </div>
+            </form>
         </div>
     </div>
 {% endblock %}
diff --git a/netbox/templates/dcim/inc/inventoryitem.html b/netbox/templates/dcim/inc/inventoryitem.html
index d3a78afd4..1b103893f 100644
--- a/netbox/templates/dcim/inc/inventoryitem.html
+++ b/netbox/templates/dcim/inc/inventoryitem.html
@@ -1,13 +1,34 @@
+{% load helpers %}
 <tr>
+
+    {# Checkbox #}
+    {% if perms.dcim.change_inventoryitem or perms.dcim.delete_inventoryitem %}
+        <td class="pk">
+            <input name="pk" type="checkbox" value="{{ item.pk }}" />
+        </td>
+    {% endif %}
+
     <td style="padding-left: {{ indent|add:5 }}px">
         <a href="{{ item.get_absolute_url }}">{{ item }}</a>
     </td>
-    <td>{% if not item.discovered %}<i class="fa fa-asterisk" title="Manually created"></i>{% endif %}</td>
-    <td>{{ item.manufacturer|default:"" }}</td>
-    <td>{{ item.part_id }}</td>
-    <td>{{ item.serial }}</td>
-    <td>{{ item.asset_tag|default:"" }}</td>
-    <td>{{ item.description }}</td>
+    <td>
+        {% if item.manufacturer %}
+            <a href="{{ item.manufacturer.get_absolute_url }}">{{ item.manufacturer }}</a>
+        {% else %}
+            <span class="text-muted">&mdash;</span>
+        {% endif %}
+    </td>
+    <td>{{ item.part_id|placeholder }}</td>
+    <td>{{ item.serial|placeholder }}</td>
+    <td>{{ item.asset_tag|placeholder }}</td>
+    <td>
+        {% if item.discovered %}
+            <span class="text-success"><i class="fa fa-check"></i></span>
+        {% else %}
+            <span class="text-muted">&mdash;</span>
+        {% endif %}
+    </td>
+    <td>{{ item.description|placeholder }}</td>
     <td class="text-right noprint">
         {% if perms.dcim.change_inventoryitem %}
             <a href="{% url 'dcim:inventoryitem_edit' pk=item.pk %}" class="btn btn-xs btn-warning"><span class="glyphicon glyphicon-pencil" aria-hidden="true"></span></a>