Use real name instead of username for messages and direct messages list

packages/rocketchat-cas/cas_server.js

@@ -188,7 +188,7 @@ Accounts.registerLoginHandler(function(options) {
 			logger.debug('Syncing user attributes');
 			// Update name
 			if (int_attrs.name) {
-				Meteor.users.update(user, { $set: { name: int_attrs.name }});
+				RocketChat._setRealName(user._id, int_attrs.name);
 			}
 
 			// Update email

packages/rocketchat-crowd/server/crowd.js

@@ -85,7 +85,6 @@ const CROWD = class CROWD {
 
 	syncDataToUser(crowdUser, id) {
 		const user = {
-			name: crowdUser.displayname,
 			username: crowdUser.username,
 			emails: [{
 				address : crowdUser.email,
@@ -95,6 +94,10 @@ const CROWD = class CROWD {
 			active: crowdUser.active
 		};
 
+		if (crowdUser.displayname) {
+			RocketChat._setRealName(user._id, crowdUser.displayname);
+		}
+
 		Meteor.users.update(id, {
 			$set: user
 		});

packages/rocketchat-i18n/i18n/en.i18n.json

@@ -1505,6 +1505,7 @@
   "UI_DisplayRoles": "Display Roles",
   "UI_Merge_Channels_Groups": "Merge private groups with channels",
   "UI_Use_Name_Avatar": "Use full name initials to generate default avatar",
+  "UI_Use_Real_Name" : "Use Real Name",
   "Unarchive": "Unarchive",
   "Unblock_User": "Unblock User",
   "Unmute_someone_in_room": "Unmute someone in the room",

packages/rocketchat-importer-hipchat/server.coffee

@@ -221,7 +221,7 @@ Importer.HipChat = class Importer.HipChat extends Importer.Base
 
 	getRocketUser: (hipchatId) =>
 		for user in @users.users when user.user_id is hipchatId
-			return RocketChat.models.Users.findOneById user.rocketId, { fields: { username: 1 }}
+			return RocketChat.models.Users.findOneById user.rocketId, { fields: { username: 1, name: 1 }}
 
 	convertHipChatMessageToRocketChat: (message) =>
 		if message?

packages/rocketchat-importer-slack/server.coffee

@@ -342,7 +342,7 @@ Importer.Slack = class Importer.Slack extends Importer.Base
 
 	getRocketUser: (slackId) =>
 		for user in @users.users when user.id is slackId
-			return RocketChat.models.Users.findOneById user.rocketId, { fields: { username: 1 }}
+			return RocketChat.models.Users.findOneById user.rocketId, { fields: { username: 1, name: 1 }}
 
 	convertSlackMessageToRocketChat: (message) =>
 		if message?

packages/rocketchat-ldap/server/sync.js

@@ -122,9 +122,13 @@ syncUserData = function syncUserData(user, ldapUser) {
 
 	const userData = getDataToSyncUserData(ldapUser, user);
 	if (user && user._id && userData) {
+		logger.debug('setting', JSON.stringify(userData, null, 2));
+		if (userData.name) {
+			RocketChat._setRealName(user._id, userData.name);
+			delete userData.name;
+		}
 		Meteor.users.update(user._id, { $set: userData });
 		user = Meteor.users.findOne({_id: user._id});
-		logger.debug('setting', JSON.stringify(userData, null, 2));
 	}
 
 	if (RocketChat.settings.get('LDAP_Username_Field') !== '') {

packages/rocketchat-lib/package.js

@@ -87,6 +87,7 @@ Package.onUse(function(api) {
 	api.addFiles('server/functions/settings.coffee', 'server');
 	api.addFiles('server/functions/setUserAvatar.js', 'server');
 	api.addFiles('server/functions/setUsername.coffee', 'server');
+	api.addFiles('server/functions/setRealName.coffee', 'server');
 	api.addFiles('server/functions/setEmail.js', 'server');
 	api.addFiles('server/functions/unarchiveRoom.js', 'server');
 	api.addFiles('server/functions/updateMessage.js', 'server');

packages/rocketchat-lib/server/functions/addUserToRoom.js

@@ -27,7 +27,8 @@ RocketChat.addUserToRoom = function(rid, user, inviter, silenced) {
 				ts: now,
 				u: {
 					_id: inviter._id,
-					username: inviter.username
+					username: inviter.username,
+					name: inviter.name
 				}
 			});
 		} else {

packages/rocketchat-lib/server/functions/saveUser.js

@@ -140,6 +140,10 @@ RocketChat.saveUser = function(userId, userData) {
 			RocketChat.setUsername(userData._id, userData.username);
 		}
 
+		if (userData.name) {
+			RocketChat.setRealName(userData._id, userData.name);
+		}
+
 		if (userData.email) {
 			RocketChat.setEmail(userData._id, userData.email);
 		}
@@ -152,10 +156,6 @@ RocketChat.saveUser = function(userId, userData) {
 			$set: {}
 		};
 
-		if (userData.name) {
-			updateUser.$set.name = userData.name;
-		}
-
 		if (userData.roles) {
 			updateUser.$set.roles = userData.roles;
 		}

packages/rocketchat-lib/server/functions/sendMessage.coffee

@@ -5,7 +5,7 @@ RocketChat.sendMessage = (user, message, room, upsert = false) ->
 	unless message.ts?
 		message.ts = new Date()
 
-	message.u = _.pick user, ['_id','username']
+	message.u = _.pick user, ['_id','username', 'name']
 
 	if not Match.test(message.msg, String)
 		message.msg = ''

packages/rocketchat-lib/server/functions/setRealName.coffee

@@ -0,0 +1,24 @@
+RocketChat._setRealName = (userId, name) ->
+	name = s.trim name
+	if not userId or not name
+		return false
+
+	user = RocketChat.models.Users.findOneById userId
+
+	# User already has desired name, return
+	if user.name is name
+		return user
+
+	previousName = user.name
+
+	if previousName
+		RocketChat.models.Messages.updateAllNamesByUserId user._id, name
+		RocketChat.models.Subscriptions.setRealNameForDirectRoomsWithUsername user.username, name
+
+	# Set new name
+	RocketChat.models.Users.setName user._id, name
+	user.name = name
+	return user
+
+RocketChat.setRealName = RocketChat.RateLimiter.limitFunction RocketChat._setRealName, 1, 60000,
+	0: () -> return not Meteor.userId() or not RocketChat.authz.hasPermission(Meteor.userId(), 'edit-other-user-info') # Administrators have permission to change others names, so don't limit those

packages/rocketchat-lib/server/lib/sendNotificationsOnMessage.js

@@ -128,7 +128,7 @@ RocketChat.callbacks.add('afterSaveMessage', function(message, room) {
 		}
 		if ((userOfMention != null) && canBeNotified(userOfMentionId, 'mobile')) {
 			RocketChat.Notifications.notifyUser(userOfMention._id, 'notification', {
-				title: `@${ user.username }`,
+				title: RocketChat.settings.get('UI_Use_Real_Name') ? user.name : `@${ user.username }`,
 				text: message.msg,
 				duration: settings.desktopNotificationDurations[userOfMention._id],
 				payload: {

packages/rocketchat-lib/server/methods/setRealName.js

@@ -17,10 +17,14 @@ Meteor.methods({
 			name = _.trim(name);
 		}
 
-		if (!RocketChat.models.Users.setName(Meteor.userId(), name)) {
+		if (!RocketChat.setRealName(Meteor.userId(), name)) {
 			throw new Meteor.Error('error-could-not-change-name', 'Could not change name', { method: 'setRealName' });
 		}
 
 		return name;
 	}
 });
+
+RocketChat.RateLimiter.limitMethod('setRealName', 1, 1000, {
+	userId: () => true
+});

packages/rocketchat-lib/server/models/Messages.coffee

@@ -320,6 +320,16 @@ RocketChat.models.Messages = new class extends RocketChat.models._Base
 
 		return @update query, update
 
+	updateAllNamesByUserId: (userId, name) ->
+		query =
+			'u._id': userId
+
+		update =
+			$set:
+				"u.name": name
+
+		return @update query, update, { multi: true }
+
 	updateUserStarById: (_id, userId, starred) ->
 		query =
 			_id: _id
@@ -380,6 +390,7 @@ RocketChat.models.Messages = new class extends RocketChat.models._Base
 			u:
 				_id: user._id
 				username: user.username
+				name: user.name
 			groupable: false
 
 		_.extend record, extraData

packages/rocketchat-lib/server/models/Subscriptions.coffee

@@ -242,6 +242,17 @@ class ModelSubscriptions extends RocketChat.models._Base
 
 		return @update query, update, { multi: true }
 
+	setRealNameForDirectRoomsWithUsername: (username, name) ->
+		query =
+			name: username
+			t: "d"
+
+		update =
+			$set:
+				fname: name
+
+		return @update query, update, { multi: true }
+
 	setNameForDirectRoomsWithOldName: (oldName, name) ->
 		query =
 			name: oldName

packages/rocketchat-lib/server/models/Users.coffee

@@ -109,7 +109,7 @@ class ModelUsers extends RocketChat.models._Base
 
 		return @find query, options
 
-	findByActiveUsersUsernameExcept: (searchTerm, exceptions = [], options = {}) ->
+	findByActiveUsersExcept: (searchTerm, exceptions = [], options = {}) ->
 		if not _.isArray exceptions
 			exceptions = [ exceptions ]
 
@@ -118,7 +118,14 @@ class ModelUsers extends RocketChat.models._Base
 			$and: [
 				{
 					active: true
-					username: termRegex
+					$or: [
+						{
+							username: termRegex
+						}
+						{
+							name: termRegex
+						}
+					]
 				}
 				{
 					username: { $nin: exceptions }

packages/rocketchat-lib/server/startup/settings.coffee

@@ -263,6 +263,7 @@ RocketChat.settings.addGroup 'Layout', ->
 		@add 'UI_DisplayRoles', true, { type: 'boolean', public: true }
 		@add 'UI_Merge_Channels_Groups', true, { type: 'boolean', public: true }
 		@add 'UI_Use_Name_Avatar', false, { type: 'boolean', public: true }
+		@add 'UI_Use_Real_Name', false, { type: 'boolean', public: true }
 
 
 RocketChat.settings.addGroup 'Logs', ->

packages/rocketchat-lib/startup/defaultRoomTypes.js

@@ -25,7 +25,7 @@ RocketChat.roomTypes.add('c', 10, {
 	},
 
 	roomName(roomData) {
-		return roomData.name;
+		return { name: roomData.name };
 	},
 
 	condition() {
@@ -64,8 +64,7 @@ RocketChat.roomTypes.add('d', 20, {
 	},
 
 	roomName(roomData) {
-		const room = ChatSubscription.findOne({ rid: roomData._id }, { fields: { name: 1 } });
-		return room && room.name;
+		return ChatSubscription.findOne({ rid: roomData._id }, { fields: { name: 1, fname: 1 } });
 	},
 
 	condition() {
@@ -100,7 +99,7 @@ RocketChat.roomTypes.add('p', 30, {
 	},
 
 	roomName(roomData) {
-		return roomData.name;
+		return { name: roomData.name };
 	},
 
 	condition() {

packages/rocketchat-livechat/server/lib/Livechat.js

@@ -116,7 +116,6 @@ RocketChat.Livechat = {
 
 				userId = existingUser._id;
 			} else {
-				updateUser.$set.name = name;
 
 				const userData = {
 					username,
@@ -156,6 +155,10 @@ RocketChat.Livechat = {
 			];
 		}
 
+		if (name) {
+			RocketChat._setRealName(userId, name);
+		}
+
 		Meteor.users.update(userId, updateUser);
 
 		return userId;

packages/rocketchat-theme/client/imports/base.less

@@ -2495,6 +2495,11 @@ label.required::after {
 			display: inline-block;
 		}
 	}
+
+	.secondary-name {
+		color: #666666;
+		font-size: 15px;
+	}
 }
 
 .message-popup-position {