From 190daaa8a0bb79e750894b83e31b9d99f7d900a4 Mon Sep 17 00:00:00 2001
From: erdgeist <erdgeist@erdgeist.org>
Date: Thu, 16 Jul 2026 03:30:01 +0200
Subject: Add a reader for the action log at admin/log

NodeActionsController#index lists entries newest-first, filterable
by node_id or user_id -- the two zoom shapes the log was designed
around. Rendering goes through NodeActionsHelper.action_summary,
which builds one sentence per entry from metadata alone, so entries
referencing deleted users or nodes render from their snapshots;
live associations only upgrade names to links. Unknown verbs
degrade to a generic sentence rather than an error, since the log
outlives its vocabulary. The helper is the escaping boundary:
every metadata value passes through h() before assembly.

Actor names link to the log's own user zoom rather than the unused
users page -- inspecting a suspicious user's other actions is the
intended workflow. Publish entries with a translation_diff expose
a collapsed per-locale change table linking out to the revision
itself. Sentences live in en.yml/de.yml following the existing
widget-string convention. nodes#show links to its node's zoomed
log.
---
 app/views/node_actions/_change_details.html.erb | 27 +++++++++++++++++++++++++
 app/views/node_actions/index.html.erb           | 22 ++++++++++++++++++++
 app/views/nodes/show.html.erb                   |  1 +
 3 files changed, 50 insertions(+)
 create mode 100644 app/views/node_actions/_change_details.html.erb
 create mode 100644 app/views/node_actions/index.html.erb

(limited to 'app/views')

diff --git a/app/views/node_actions/_change_details.html.erb b/app/views/node_actions/_change_details.html.erb
new file mode 100644
index 0000000..383f53a
--- /dev/null
+++ b/app/views/node_actions/_change_details.html.erb
@@ -0,0 +1,27 @@
+<details class="node_action_details">
+  <summary><%= t("node_actions.show_changes") %></summary>
+  <table>
+    <% action_entry.metadata["translation_diff"].each do |locale, diff| %>
+      <tr>
+        <th><%= locale.upcase %></th>
+        <td>
+          <% case diff["status"] %>
+          <% when "added" %>
+            <%= t("node_actions.locale_added", :title => diff.dig("title", "to")) %>
+          <% when "removed" %>
+            <%= t("node_actions.locale_removed", :title => diff.dig("title", "from")) %>
+          <% else %>
+            <% if diff["title"] %>
+              <%= diff.dig("title", "from") %> → <%= diff.dig("title", "to") %>
+            <% end %>
+            <%= t("node_actions.abstract_changed") if diff["abstract_changed"] %>
+            <%= t("node_actions.body_changed") if diff["body_changed"] %>
+          <% end %>
+        </td>
+      </tr>
+    <% end %>
+  </table>
+  <% if action_entry.page && action_entry.node %>
+    <%= link_to t("node_actions.view_revision"), node_revision_path(action_entry.node, action_entry.page) %>
+  <% end %>
+</details>
diff --git a/app/views/node_actions/index.html.erb b/app/views/node_actions/index.html.erb
new file mode 100644
index 0000000..92740cd
--- /dev/null
+++ b/app/views/node_actions/index.html.erb
@@ -0,0 +1,22 @@
+<h1><%= t("node_actions.heading") %></h1>
+
+<% if params[:node_id].present? || params[:user_id].present? %>
+  <p><%= link_to t("node_actions.show_all"), admin_log_path %></p>
+<% end %>
+
+<%= will_paginate @actions %>
+
+<ul id="node_action_list">
+  <% @actions.each do |action| %>
+    <li class="node_action node_action--<%= action.action %>">
+      <span class="node_action_time"><%= l(action.occurred_at, :format => :ccc) %></span>
+      <% if action.inferred_from %>
+        <span class="node_action_inferred" title="<%= action.inferred_from %>"><%= t("node_actions.backfilled") %></span>
+      <% end %>
+      <span class="node_action_summary"><%= action_summary(action) %></span>
+      <%= render "change_details", :action_entry => action if action.metadata["translation_diff"].present? %>
+    </li>
+  <% end %>
+</ul>
+
+<%= will_paginate @actions %>
diff --git a/app/views/nodes/show.html.erb b/app/views/nodes/show.html.erb
index 66f04f9..d0eff60 100644
--- a/app/views/nodes/show.html.erb
+++ b/app/views/nodes/show.html.erb
@@ -183,6 +183,7 @@
         </ul>
       </details>
       <p class="revisions_full_history_link"><%= link_to 'Full history (diff / restore)', node_revisions_path(@node) %></p>
+      <p class)"revisions_full_history_link"><%= link_to t("node_actions.heading"), admin_log_path(:node_id => @node.id) %></p>
     </div>
 
 
-- 
cgit v1.3