From 300faa67445770d598efe5e0a957d0dd6b367f3d Mon Sep 17 00:00:00 2001
From: Robin Ward <robin.ward@gmail.com>
Date: Tue, 7 Jul 2015 15:50:14 -0400
Subject: [PATCH] UX: Put links back in flags summaries in advance of refactor.

---
 .../components/actions-summary.js.es6         | 36 +++++++++++++++++--
 1 file changed, 33 insertions(+), 3 deletions(-)

diff --git a/app/assets/javascripts/discourse/components/actions-summary.js.es6 b/app/assets/javascripts/discourse/components/actions-summary.js.es6
index e78e7dde3..31c145952 100644
--- a/app/assets/javascripts/discourse/components/actions-summary.js.es6
+++ b/app/assets/javascripts/discourse/components/actions-summary.js.es6
@@ -7,7 +7,8 @@ export default Ember.Component.extend(StringBuffer, {
   actionsSummary: Em.computed.alias('post.actionsWithoutLikes'),
   emptySummary: Em.computed.empty('actionsSummary'),
   hidden: Em.computed.and('emptySummary', 'post.notDeleted'),
-  rerenderTriggers: ['actionsSummary.@each', 'post.deleted'],
+
+  rerenderTriggers: ['actionsSummary.@each', 'actionsSummary.users.length', 'post.deleted'],
 
   // This was creating way too many bound ifs and subviews in the handlebars version.
   renderString(buffer) {
@@ -21,15 +22,38 @@ export default Ember.Component.extend(StringBuffer, {
         };
 
         // TODO multi line expansion for flags
-        buffer.push(c.get('description') + '.');
+        let iconsHtml = "";
+        if (c.get('usersExpanded')) {
+          let postUrl;
+          c.get('users').forEach(function(u) {
+            iconsHtml += "<a href=\"" + Discourse.getURL("/users/") + u.get('username_lower') + "\" data-user-card=\"" + u.get('username_lower') + "\">";
+            if (u.post_url) {
+              postUrl = postUrl || u.post_url;
+            }
+            iconsHtml += Discourse.Utilities.avatarImg({
+              size: 'small',
+              avatarTemplate: u.get('avatarTemplate'),
+              title: u.get('username')
+            });
+            iconsHtml += "</a>";
+          });
+
+          let key = 'post.actions.people.' + c.get('actionType.name_key');
+          if (postUrl) { key = key + "_with_url"; }
+
+          // TODO postUrl might be uninitialized? pick a good default
+          buffer.push(" " + I18n.t(key, { icons: iconsHtml, postUrl: postUrl}) + ".");
+        }
+        renderActionIf('usersCollapsed', 'who-acted', c.get('description'));
         renderActionIf('can_undo', 'undo', I18n.t("post.actions.undo." + c.get('actionType.name_key')));
         renderActionIf('can_defer_flags', 'defer-flags', I18n.t("post.actions.defer_flags", { count: c.count }));
+
         buffer.push("</div>");
       });
     }
 
     const post = this.get('post');
-    if (!post.get('deleted')) {
+    if (post.get('deleted')) {
       buffer.push("<div class='post-action'>" +
                   iconHTML('fa-trash-o') + '&nbsp;' +
                   Discourse.Utilities.tinyAvatar(post.get('postDeletedBy.avatar_template'), {title: post.get('postDeletedBy.username')}) +
@@ -53,6 +77,12 @@ export default Ember.Component.extend(StringBuffer, {
       return false;
     }
 
+    // User wants to know who actioned it
+    if (actionTypeId = $target.data('who-acted')) {
+      this.actionTypeById(actionTypeId).loadUsers(post);
+      return false;
+    }
+
     if (actionTypeId = $target.data('undo')) {
       this.get('actionsSummary').findProperty('id', actionTypeId).undo(post);
       return false;