From d1a39f74460f43b1d0547005125eee65ffe815a8 Mon Sep 17 00:00:00 2001
From: rkuehn-uofl <randy.kuehn@louisville.edu>
Date: Mon, 4 Nov 2024 11:32:02 -0500
Subject: [PATCH] Updated user activity log table

---
 app/views/hyrax/users/_activity_log.html.erb | 24 ++++++++++++--------
 1 file changed, 15 insertions(+), 9 deletions(-)

diff --git a/app/views/hyrax/users/_activity_log.html.erb b/app/views/hyrax/users/_activity_log.html.erb
index 183bbf4685..6c3700bb4f 100644
--- a/app/views/hyrax/users/_activity_log.html.erb
+++ b/app/views/hyrax/users/_activity_log.html.erb
@@ -7,17 +7,23 @@
       </tr>
     </thead>
     <tbody>
-    <% events.each do |event| %>
-      <% next if event[:action].blank? or event[:timestamp].blank? %>
+    <% valid_events = events.reject { |event| event[:action].blank? || event[:timestamp].blank? } %>
+    <% if valid_events.empty? %>
       <tr>
-        <td class="ensure-wrapped"><%= sanitize event[:action] %></td>
-        <% time = Time.zone.at(event[:timestamp].to_i) %>
-        <td data-sort="<%= time.getutc.iso8601(5) %>">
-          <relative-time datetime="<%= time.getutc.iso8601 %>" title="<%= time.to_formatted_s(:standard) %>">
-            <%= time.to_formatted_s(:long_ordinal) %>
-          </relative-time>
-        </td>
+        <td colspan="2">No user activity</td>
       </tr>
+    <% else %>
+      <% valid_events.each do |event| %>
+        <tr>
+          <td class="ensure-wrapped"><%= sanitize event[:action] %></td>
+          <% time = Time.zone.at(event[:timestamp].to_i) %>
+          <td data-sort="<%= time.getutc.iso8601(5) %>">
+            <relative-time datetime="<%= time.getutc.iso8601 %>" title="<%= time.to_formatted_s(:standard) %>">
+              <%= time.to_formatted_s(:long_ordinal) %>
+            </relative-time>
+          </td>
+        </tr>
+      <% end %>
     <% end %>
     </tbody>
   </table>