Show remote reports in admin UI as coming from domain rather than user (#7347)

Fix #6994

diff --git a/app/views/admin/reports/_report.html.haml b/app/views/admin/reports/_report.html.haml
index d6c881955c9596b48b36e09ea40e2be27d40d957..7b25c924bc9a18bff51ff52399ff15704c5501e9 100644 (file)
--- a/app/views/admin/reports/_report.html.haml
+++ b/app/views/admin/reports/_report.html.haml
@@ -4,7 +4,10 @@
   %td.target
     = admin_account_link_to report.target_account
   %td.reporter
-    = admin_account_link_to report.account
+    - if report.account.local?
+      = admin_account_link_to report.account
+    - else
+      = report.account.domain
   %td
     %div{ title: report.comment }
       = truncate(report.comment, length: 30, separator: ' ')

diff --git a/app/views/admin/reports/show.html.haml b/app/views/admin/reports/show.html.haml
index 2bba3079ed9a8dca5d70a21c1b67e35d227ab569..c9ebc041583e2a60b087ec74b4232a83a0d7f3cd 100644 (file)
--- a/app/views/admin/reports/show.html.haml
+++ b/app/views/admin/reports/show.html.haml
@@ -26,9 +26,12 @@
         %td= table_link_to 'file', pluralize(@report.target_account.targeted_moderation_notes.count, t('admin.reports.account.note')), admin_reports_path(target_account_id: @report.target_account.id)
       %tr
         %th= t('admin.reports.reported_by')
-        %td= admin_account_link_to @report.account
-        %td= table_link_to 'flag', pluralize(@report.account.targeted_reports.count, t('admin.reports.account.report')), admin_reports_path(target_account_id: @report.account.id)
-        %td= table_link_to 'file', pluralize(@report.account.targeted_moderation_notes.count, t('admin.reports.account.note')), admin_reports_path(target_account_id: @report.account.id)
+        - if @report.account.local?
+          %td= admin_account_link_to @report.account
+          %td= table_link_to 'flag', pluralize(@report.account.targeted_reports.count, t('admin.reports.account.report')), admin_reports_path(target_account_id: @report.account.id)
+          %td= table_link_to 'file', pluralize(@report.account.targeted_moderation_notes.count, t('admin.reports.account.note')), admin_reports_path(target_account_id: @report.account.id)
+        - else
+          %td{ colspan: 3 }= @report.account.domain
       %tr
         %th= t('admin.reports.created_at')
         %td{ colspan: 3 }

diff --git a/app/views/admin_mailer/new_report.text.erb b/app/views/admin_mailer/new_report.text.erb
index 671ae5ca7061c097038d0e0c5e95a85af6e13e20..d6c7d6bab378019a637b800f99ae7f6ea67a7d19 100644 (file)
--- a/app/views/admin_mailer/new_report.text.erb
+++ b/app/views/admin_mailer/new_report.text.erb
@@ -1,5 +1,5 @@
 <%= raw t('application_mailer.salutation', name: display_name(@me)) %>
 
-<%= raw t('admin_mailer.new_report.body', target: @report.target_account.acct, reporter: @report.account.acct) %>
+<%= raw(@report.account.local? ? t('admin_mailer.new_report.body', target: @report.target_account.acct, reporter: @report.account.acct) : t('admin_mailer.new_report.body_remote', target: @report.target_account.acct, domain: @report.account.domain)) %>
 
 <%= raw t('application_mailer.view')%> <%= admin_report_url(@report) %>

diff --git a/config/locales/en.yml b/config/locales/en.yml
index 1468d85598ec7b8ca2c65119ab22cf99f6174f4f..f7127f7948fcba3fd73997714578713ecc7512ef 100644 (file)
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -376,6 +376,7 @@ en:
   admin_mailer:
     new_report:
       body: "%{reporter} has reported %{target}"
+      body_remote: Someone from %{domain} has reported %{target}
       subject: New report for %{instance} (#%{id})
   application_mailer:
     notification_preferences: Change e-mail preferences