compose_typeahead: Don't insert current user with user group PM pills.

zulip · Jul 19, 2018 · 1406497 · 1406497
1 parent 0c5d3db
commit 1406497
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/frontend_tests/node_tests/composebox_typeahead.js b/frontend_tests/node_tests/composebox_typeahead.js
@@ -572,6 +572,9 @@ run_test('initialize', () => {
             var users = {100: hamlet, 104: lear};
             return users[user_id];
         };
+        people.my_current_email = function () {
+            return 'hamlet@zulip.com';
+        };
         compose_pm_pill.set_from_typeahead = function (item) {
             appended_names.push(item.full_name);
         };

diff --git a/static/js/composebox_typeahead.js b/static/js/composebox_typeahead.js
@@ -638,7 +638,10 @@ exports.initialize = function () {
             if (user_groups.is_user_group(item)) {
                 item.members.keys().forEach(function (user_id) {
                     var user = people.get_person_from_user_id(user_id);
-                    compose_pm_pill.set_from_typeahead(user);
+                    if (!people.is_current_user(user.email)) {
+                        // don't add pill for current user
+                        compose_pm_pill.set_from_typeahead(user);
+                    }
                 });
             } else {
                 compose_pm_pill.set_from_typeahead(item);