From 3d7fcaa2b28a298e58773f49ed72253929b08b2f Mon Sep 17 00:00:00 2001
From: Toby Zerner <toby.zerner@gmail.com>
Date: Sat, 24 Oct 2015 13:14:57 +1030
Subject: [PATCH] Truncate list of replies, pluralize notification text

ref flarum/core#597
---
 .../js/forum/src/addMentionedByList.js        | 47 ++++++++++++-------
 .../components/PostMentionedNotification.js   |  4 +-
 2 files changed, 32 insertions(+), 19 deletions(-)

diff --git a/extensions/mentions/js/forum/src/addMentionedByList.js b/extensions/mentions/js/forum/src/addMentionedByList.js
index e56cc478b..058d52db0 100644
--- a/extensions/mentions/js/forum/src/addMentionedByList.js
+++ b/extensions/mentions/js/forum/src/addMentionedByList.js
@@ -70,40 +70,53 @@ export default function addMentionedByList() {
         });
       };
 
-// NEEDS TO BE FIXED: The next two blocks of code. See https://github.com/flarum/core/issues/597 for details.
+      const users = [];
+      const repliers = replies
+        .sort(reply => reply.user() === app.session.user ? -1 : 0)
+        .filter(reply => {
+          const user = reply.user();
+          if (users.indexOf(user) === -1) {
+            users.push(user);
+            return true;
+          }
+        });
+
+      const limit = 4;
+      const overLimit = repliers.length > limit;
 
       // Create a list of unique users who have replied. So even if a user has
       // replied twice, they will only be in this array once.
-      const used = [];
-      const repliers = replies.filter(reply => {
-        const user = reply.user();
-        const id = user && user.id();
-        if (used.indexOf(id) === -1) {
-          used.push(id);
-          return true;
-        }
-      });
-
-      const names = repliers.sort(a => a === app.session.user ? -1 : 1)
+      const names = repliers
+        .slice(0, overLimit ? limit - 1 : limit)
         .map(reply => {
           const user = reply.user();
 
           return (
             <a href={app.route.post(reply)}
-              config={m.route}
-              onclick={hidePreview}
-              data-number={reply.number()}>
+               config={m.route}
+               onclick={hidePreview}
+               data-number={reply.number()}>
               {app.session.user === user ? app.translator.trans('flarum-mentions.forum.post.you_text') : username(user)}
             </a>
           );
         });
 
+      // If there are more users that we've run out of room to display, add a "x
+      // others" name to the end of the list. Clicking on it will display a modal
+      // with a full list of names.
+      if (overLimit) {
+        const count = repliers.length - names.length;
+
+        names.push(
+          app.translator.transChoice('flarum-mentions.forum.post.others_text', count, {count})
+        );
+      }
+
       items.add('replies',
         <div className="Post-mentionedBy" config={config}>
           <span className="Post-mentionedBy-summary">
             {icon('reply')}
-            // PLEASE CHECK: Using the syntax from "addLikesList.js" with "repliers[0]" in place of "likes[0]".
-            {app.translator.transChoice('flarum-mentions.forum.post.mentioned_by' + (repliers[0] === app.session.user ? '_self' : '') + '_text', names.length, {
+            {app.translator.transChoice('flarum-mentions.forum.post.mentioned_by' + (replies[0] === app.session.user ? '_self' : '') + '_text', names.length, {
               count: names.length,
               users: punctuateSeries(names)
             })}
diff --git a/extensions/mentions/js/forum/src/components/PostMentionedNotification.js b/extensions/mentions/js/forum/src/components/PostMentionedNotification.js
index 2693107bf..d18c8f333 100644
--- a/extensions/mentions/js/forum/src/components/PostMentionedNotification.js
+++ b/extensions/mentions/js/forum/src/components/PostMentionedNotification.js
@@ -21,11 +21,11 @@ export default class PostMentionedNotification extends Notification {
     const auc = notification.additionalUnreadCount();
     const user = notification.sender();
 
-    return app.translator.trans('flarum-mentions.forum.notifications.post_mentioned_text', {
+    return app.translator.transChoice('flarum-mentions.forum.notifications.post_mentioned_text', auc + 1, {
       user,
       username: auc ? punctuateSeries([
         username(user),
-        app.translator.trans('flarum-mentions.forum.notifications.others_text', {count: auc})
+        app.translator.transChoice('flarum-mentions.forum.notifications.others_text', auc, {count: auc})
       ]) : undefined
     });
   }