From 26ea30cb162684990cacdfd17aeb6bfcc5e10e03 Mon Sep 17 00:00:00 2001
From: hmhealey <harrisonmhealey@gmail.com>
Date: Tue, 1 Mar 2016 16:28:47 -0500
Subject: Changed information displayed in user list to obey display name
 settings

---
 web/react/components/user_list_row.jsx | 34 ++++++++++------------------------
 web/react/utils/constants.jsx          |  2 ++
 web/react/utils/utils.jsx              |  4 ++--
 3 files changed, 14 insertions(+), 26 deletions(-)

(limited to 'web')

diff --git a/web/react/components/user_list_row.jsx b/web/react/components/user_list_row.jsx
index cf2e2668c..2aeca7d47 100644
--- a/web/react/components/user_list_row.jsx
+++ b/web/react/components/user_list_row.jsx
@@ -1,32 +1,18 @@
 // Copyright (c) 2015 Mattermost, Inc. All Rights Reserved.
 // See License.txt for license information.
 
+import Constants from '../utils/constants.jsx';
+import PreferenceStore from '../stores/preference_store.jsx';
 import * as Utils from '../utils/utils.jsx';
 
 export default function UserListRow({user, actions}) {
-    const details = [];
+    const nameFormat = PreferenceStore.get(Constants.Preferences.CATEGORY_DISPLAY_SETTINGS, 'name_format', '');
 
-    const fullName = Utils.getFullName(user);
-    if (fullName) {
-        details.push(
-            <span
-                key={`${user.id}__full-name`}
-                className='full-name'
-            >
-                {fullName}
-            </span>
-        );
-    }
-
-    if (user.nickname) {
-        const separator = fullName ? ' - ' : '';
-        details.push(
-            <span
-                key={`${user.nickname}__nickname`}
-            >
-                {separator + user.nickname}
-            </span>
-        );
+    let name = user.username;
+    if (user.nickname && nameFormat === Constants.Preferences.DISPLAY_PREFER_NICKNAME) {
+        name = `${user.nickname} (${user.username})`;
+    } else if ((user.first_name || user.last_name) && (nameFormat === Constants.Preferences.DISPLAY_PREFER_NICKNAME || nameFormat === Constants.Preferences.DISPLAY_PREFER_FULL_NAME)) {
+        name = `${Utils.getFullName(user)} (${user.username})`;
     }
 
     const buttons = actions.map((Action, index) => {
@@ -53,10 +39,10 @@ export default function UserListRow({user, actions}) {
                     className='user-list-item__details'
                 >
                     <div className='more-name'>
-                        {user.username}
+                        {name}
                     </div>
                     <div className='more-description'>
-                        {details}
+                        {user.email}
                     </div>
                 </div>
                 <div
diff --git a/web/react/utils/constants.jsx b/web/react/utils/constants.jsx
index 88f01a475..daea9f43e 100644
--- a/web/react/utils/constants.jsx
+++ b/web/react/utils/constants.jsx
@@ -437,6 +437,8 @@ export default {
     Preferences: {
         CATEGORY_DIRECT_CHANNEL_SHOW: 'direct_channel_show',
         CATEGORY_DISPLAY_SETTINGS: 'display_settings',
+        DISPLAY_PREFER_NICKNAME: 'nickname_full_name',
+        DISPLAY_PREFER_FULL_NAME: 'full_name',
         CATEGORY_ADVANCED_SETTINGS: 'advanced_settings',
         TUTORIAL_STEP: 'tutorial_step'
     },
diff --git a/web/react/utils/utils.jsx b/web/react/utils/utils.jsx
index 3e531c821..6ab2f64d4 100644
--- a/web/react/utils/utils.jsx
+++ b/web/react/utils/utils.jsx
@@ -1082,9 +1082,9 @@ export function displayUsername(userId) {
 
     let username = '';
     if (user) {
-        if (nameFormat === 'nickname_full_name') {
+        if (nameFormat === Constants.Preferences.DISPLAY_PREFER_NICKNAME) {
             username = user.nickname || getFullName(user);
-        } else if (nameFormat === 'full_name') {
+        } else if (nameFormat === Constants.Preferences.DISPLAY_PREFER_FULL_NAME) {
             username = getFullName(user);
         }
         if (!username.trim().length) {
-- 
cgit v1.2.3-1-g7c22