From 3ac8910cb7ffd531ec7b8c766ac6e19599ae7394 Mon Sep 17 00:00:00 2001
From: Essem <smswessem@gmail.com>
Date: Sat, 30 Nov 2024 14:34:42 -0600
Subject: [PATCH] Show public limit comment on remote user profiles when their
 instance is limited

---
 .../flavours/glitch/api_types/accounts.ts     |  1 +
 .../components/limited_account_hint.tsx       | 39 ++++++++++++++-----
 .../features/account_timeline/index.jsx       |  5 ++-
 .../flavours/glitch/locales/en.json           |  2 +
 .../flavours/glitch/models/account.ts         |  1 +
 app/models/account.rb                         |  7 ++++
 app/serializers/rest/account_serializer.rb    |  5 +++
 7 files changed, 48 insertions(+), 12 deletions(-)

diff --git a/app/javascript/flavours/glitch/api_types/accounts.ts b/app/javascript/flavours/glitch/api_types/accounts.ts
index fdbd7523fc15cd..cee7e41e779356 100644
--- a/app/javascript/flavours/glitch/api_types/accounts.ts
+++ b/app/javascript/flavours/glitch/api_types/accounts.ts
@@ -42,6 +42,7 @@ export interface BaseApiAccountJSON {
   moved?: ApiAccountJSON;
   suspended?: boolean;
   limited?: boolean;
+  remote_limit_reason?: string;
   memorial?: boolean;
   hide_collections: boolean;
 }
diff --git a/app/javascript/flavours/glitch/features/account_timeline/components/limited_account_hint.tsx b/app/javascript/flavours/glitch/features/account_timeline/components/limited_account_hint.tsx
index 5613811d2f9c97..0ac5c8bd86c1e1 100644
--- a/app/javascript/flavours/glitch/features/account_timeline/components/limited_account_hint.tsx
+++ b/app/javascript/flavours/glitch/features/account_timeline/components/limited_account_hint.tsx
@@ -7,23 +7,42 @@ import { Button } from 'flavours/glitch/components/button';
 import { domain } from 'flavours/glitch/initial_state';
 import { useAppDispatch } from 'flavours/glitch/store';
 
-export const LimitedAccountHint: React.FC<{ accountId: string }> = ({
-  accountId,
-}) => {
+export const LimitedAccountHint: React.FC<{
+  accountId: string;
+  reason: string;
+}> = ({ accountId, reason }) => {
   const dispatch = useAppDispatch();
   const reveal = useCallback(() => {
     dispatch(revealAccount({ id: accountId }));
   }, [dispatch, accountId]);
 
+  const message = reason ? (
+    <p>
+      <FormattedMessage
+        id='limited_account_hint.instance_limit.title'
+        defaultMessage='The server this profile is hosted on has been limited by the moderators of {domain}.'
+        values={{ domain }}
+      />
+      <br />
+      <FormattedMessage
+        id='limited_account_hint.instance_limit.reason'
+        defaultMessage='Reason: {reason}'
+        values={{ reason }}
+      />
+    </p>
+  ) : (
+    <p>
+      <FormattedMessage
+        id='limited_account_hint.title'
+        defaultMessage='This profile has been hidden by the moderators of {domain}.'
+        values={{ domain }}
+      />
+    </p>
+  );
+
   return (
     <div className='limited-account-hint'>
-      <p>
-        <FormattedMessage
-          id='limited_account_hint.title'
-          defaultMessage='This profile has been hidden by the moderators of {domain}.'
-          values={{ domain }}
-        />
-      </p>
+      {message}
       <Button onClick={reveal}>
         <FormattedMessage
           id='limited_account_hint.action'
diff --git a/app/javascript/flavours/glitch/features/account_timeline/index.jsx b/app/javascript/flavours/glitch/features/account_timeline/index.jsx
index 813e0c7a311d92..62aa7487552d90 100644
--- a/app/javascript/flavours/glitch/features/account_timeline/index.jsx
+++ b/app/javascript/flavours/glitch/features/account_timeline/index.jsx
@@ -55,6 +55,7 @@ const mapStateToProps = (state, { params: { acct, id, tagged }, withReplies = fa
     hasMore:   state.getIn(['timelines', `account:${path}`, 'hasMore']),
     suspended: state.getIn(['accounts', accountId, 'suspended'], false),
     hidden: getAccountHidden(state, accountId),
+    instanceHideReason: state.getIn(['accounts', accountId, 'remote_limit_reason']),
   };
 };
 
@@ -164,7 +165,7 @@ class AccountTimeline extends ImmutablePureComponent {
   };
 
   render () {
-    const { accountId, statusIds, featuredStatusIds, isLoading, hasMore, suspended, isAccount, hidden, multiColumn, remote, remoteUrl } = this.props;
+    const { accountId, statusIds, featuredStatusIds, isLoading, hasMore, suspended, isAccount, hidden, multiColumn, remote, remoteUrl, instanceHideReason } = this.props;
 
     if (isLoading && statusIds.isEmpty()) {
       return (
@@ -185,7 +186,7 @@ class AccountTimeline extends ImmutablePureComponent {
     if (suspended) {
       emptyMessage = <FormattedMessage id='empty_column.account_suspended' defaultMessage='Account suspended' />;
     } else if (hidden) {
-      emptyMessage = <LimitedAccountHint accountId={accountId} />;
+      emptyMessage = <LimitedAccountHint accountId={accountId} reason={instanceHideReason} />;
     } else if (remote && statusIds.isEmpty()) {
       emptyMessage = <RemoteHint accountId={accountId} url={remoteUrl} />;
     } else {
diff --git a/app/javascript/flavours/glitch/locales/en.json b/app/javascript/flavours/glitch/locales/en.json
index 2fe5b715fa737c..263febc6adeabf 100644
--- a/app/javascript/flavours/glitch/locales/en.json
+++ b/app/javascript/flavours/glitch/locales/en.json
@@ -48,6 +48,8 @@
   "keyboard_shortcuts.bookmark": "to bookmark",
   "keyboard_shortcuts.secondary_toot": "to send toot using secondary privacy setting",
   "keyboard_shortcuts.toggle_collapse": "to collapse/uncollapse toots",
+  "limited_account_hint.instance_limit.title": "The server this profile is hosted on has been limited by the moderators of {domain}.",
+  "limited_account_hint.instance_limit.reason": "Reason: {reason}",
   "moved_to_warning": "This account is marked as moved to {moved_to_link}, and may thus not accept new follows.",
   "navigation_bar.app_settings": "App settings",
   "navigation_bar.keyboard_shortcuts": "Keyboard shortcuts",
diff --git a/app/javascript/flavours/glitch/models/account.ts b/app/javascript/flavours/glitch/models/account.ts
index 799265831ea4c5..ceae97f2ddc900 100644
--- a/app/javascript/flavours/glitch/models/account.ts
+++ b/app/javascript/flavours/glitch/models/account.ts
@@ -90,6 +90,7 @@ export const accountDefaultValues: AccountShape = {
   following_count: 0,
   statuses_count: 0,
   hidden: false,
+  remote_limit_reason: '',
   suspended: false,
   memorial: false,
   limited: false,
diff --git a/app/models/account.rb b/app/models/account.rb
index 3440a2b6ec22cf..03a3b6daed09f8 100644
--- a/app/models/account.rb
+++ b/app/models/account.rb
@@ -375,6 +375,13 @@ def synchronization_uri_prefix
     @synchronization_uri_prefix ||= "#{uri[URL_PREFIX_RE]}/"
   end
 
+  def remote_limit_reason
+    domain_block = DomainBlock.find_by(domain: domain)
+    return if domain_block.nil? || domain_block.public_comment.empty?
+
+    domain_block.public_comment
+  end
+
   class << self
     def readonly_attributes
       super - %w(statuses_count following_count followers_count)
diff --git a/app/serializers/rest/account_serializer.rb b/app/serializers/rest/account_serializer.rb
index a10d6988817370..d7b3f0ad4ed5d3 100644
--- a/app/serializers/rest/account_serializer.rb
+++ b/app/serializers/rest/account_serializer.rb
@@ -16,6 +16,7 @@ class REST::AccountSerializer < ActiveModel::Serializer
 
   attribute :suspended, if: :suspended?
   attribute :silenced, key: :limited, if: :silenced?
+  attribute :remote_limit_reason, if: :remote_limit_reason?
   attribute :noindex, if: :local?
 
   attribute :memorial, if: :memorial?
@@ -161,4 +162,8 @@ def noindex
   def moved_and_not_nested?
     object.moved?
   end
+
+  def remote_limit_reason?
+    object.silenced? && object.pretty_acct != object.username
+  end
 end