From 7b908d7fa39cebeff0bbd93700ce07b6baee61ea Mon Sep 17 00:00:00 2001
From: Yaro Kasear <git@kasear.net>
Date: Thu, 17 Jul 2025 09:45:32 -0500
Subject: [PATCH] Refactor worklog export button to streamline functionality
 and remove redundant script

---
 inventory/templates/inventory.html | 27 +++++++++------------------
 1 file changed, 9 insertions(+), 18 deletions(-)

diff --git a/inventory/templates/inventory.html b/inventory/templates/inventory.html
index ff35738..d7e67f2 100644
--- a/inventory/templates/inventory.html
+++ b/inventory/templates/inventory.html
@@ -88,6 +88,15 @@
         }
     {% endset %}
     {% set buttonBar %}
+        {% if worklog %}
+        <button class="btn btn-primary" type="button" onclick="export_csv({ids: [{% for row in worklog %}{{ row['id'] }}, {% endfor %}]}, 'worklog', '{{ item.identifier
+                    | replace('Name: ', '')
+                    | replace('ID:', '')
+                    | replace('Serial: ', '')
+                    | replace('Barcode: ', '')
+                    | lower
+                }}_worklog_export');">Export WorkLog</button>
+        {% endif %}
         <div class="btn-group">
             {{ buttons.render_button(
                 id='save',
@@ -105,23 +114,6 @@
             {% endif %}
         </div>
     {% endset %}
-    {% set middleBar %}
-        {% if worklog %}
-        <button class="btn btn-primary" type="button" onclick="export_worklog_csv()">Export WorkLog</button>
-        <script>
-            function export_worklog_csv() {
-                const ids = [{% for row in worklog %}{{ row['id'] }}, {% endfor %}];
-                export_csv({ids: ids}, 'worklog', '{{ item.identifier
-                    | replace('Name: ', '')
-                    | replace('ID:', '')
-                    | replace('Serial: ', '')
-                    | replace('Barcode: ', '')
-                    | lower
-                }}_worklog_export');
-            }
-        </script>
-        {% endif %}
-    {% endset %}
     {{ toolbars.render_toolbar(
         id='inventory',
         left=breadcrumbs.breadcrumb_header(
@@ -130,7 +122,6 @@
             ],
             title=title
         ),
-        center=middleBar,
         right=buttonBar
     ) }}
     {% if item.condition in ["Removed", "Disposed"] %}