renamed audit_log to audit_logs to match naming convention (see error_reports)

Author: Oaphi

app/controllers/admin_controller.rb

@@ -100,7 +100,7 @@ def send_all_email
     redirect_to admin_path
   end
 
-  def audit_log
+  def audit_logs
     @page = helpers.safe_page(params)
     @per_page = helpers.safe_per_page(params)
 

app/views/admin/audit_log.html.erb app/views/admin/audit_logs.html.erbapp/views/admin/audit_log.html.erb renamed to app/views/admin/audit_logs.html.erb

@@ -1,8 +1,8 @@
-<h1><%= t 'admin.tools.audit_log' %></h1>
+<h1><%= t 'admin.tools.audit_logs' %></h1>
 
 <div>
   <h3>Filters</h3>
-  <%= form_tag audit_log_path, method: :get, class: 'form-inline audit-log-filters' do %>
+  <%= form_tag audit_logs_path, method: :get, class: 'form-inline audit-log-filters' do %>
     <div class="form-group-horizontal">
       <div class="form-group">
         <%= label_tag :log_type, 'Log category', class: 'form-element' %>

app/views/admin/index.html.erb

@@ -115,7 +115,7 @@
     <div class="widget">
       <div class="widget--body">
         <i class="fas fa-user-secret"></i>
-        <%= link_to t('admin.tools.audit_log'), audit_log_path, 'data-ckb-item-link' => '' %>
+        <%= link_to t('admin.tools.audit_logs'), audit_logs_path, 'data-ckb-item-link' => '' %>
       </div>
     </div>
   </div>

config/locales/strings/en.admin.yml

@@ -33,7 +33,7 @@ en:
       privileges: 'Privileges'
       close_reasons: 'Close Reasons'
       licenses: 'Licenses'
-      audit_log: 'Audit Log'
+      audit_logs: 'Audit Log'
       post_types: 'Post Types'
       email_query: 'User Lookup by Email'
     user_fed_stat: 'User fed to STAT.'

config/locales/strings/es.admin.yml

@@ -30,6 +30,6 @@ es:
       privileges: 'Privilegios'
       close_reasons: 'Razones para Cerrar'
       licenses: 'Licencias'
-      audit_log: 'Registro de Auditoría'
+      audit_logs: 'Registro de Auditoría'
       post_types: 'Tipos de Publicación'
     user_fed_stat: 'STAT alimentado con usuario/a.'

config/routes.rb

@@ -45,7 +45,7 @@
     get    'email-all',                    to: 'admin#all_email', as: :email_all
     post   'email-all',                    to: 'admin#send_all_email', as: :send_all_email
 
-    get    'audits',                       to: 'admin#audit_log', as: :audit_log
+    get    'audits',                       to: 'admin#audit_logs', as: :audit_logs
 
     get    'new-site',                     to: 'admin#new_site', as: :new_site
     post   'new-site',                     to: 'admin#create_site', as: :create_site

test/controllers/admin_controller_test.rb

@@ -3,7 +3,7 @@
 class AdminControllerTest < ActionController::TestCase
   include Devise::Test::ControllerHelpers
 
-  PARAM_LESS_ACTIONS = [:index, :error_reports, :privileges, :audit_log, :email_query, :admin_email, :all_email].freeze
+  PARAM_LESS_ACTIONS = [:index, :error_reports, :privileges, :audit_logs, :email_query, :admin_email, :all_email].freeze
 
   test 'should get index' do
     sign_in users(:admin)
@@ -127,7 +127,7 @@ class AdminControllerTest < ActionController::TestCase
 
   test 'should get audit log' do
     sign_in users(:admin)
-    get :audit_log
+    get :audit_logs
     assert_response(:success)
     assert_not_nil assigns(:logs)
   end
@@ -174,7 +174,7 @@ class AdminControllerTest < ActionController::TestCase
 
   test 'audit log should work with filter params' do
     sign_in users(:admin)
-    get :audit_log, params: { log_type: 'admin_audit', event_type: 'setting_update', from: '2025-04-13',
+    get :audit_logs, params: { log_type: 'admin_audit', event_type: 'setting_update', from: '2025-04-13',
                               to: '2025-04-13' }
     assert_response(:success)
     assert_not_nil assigns(:logs)