[FIX] API call users.setStatus does not trigger status update of clie…

…nts (#18961)
RocketChat · Sep 21, 2020 · 287d1dc · 287d1dc
1 parent 7a51a5e
commit 287d1dc
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/app/api/server/v1/users.js b/app/api/server/v1/users.js
@@ -22,6 +22,7 @@ import { setStatusText } from '../../../lib/server';
 import { findUsersToAutocomplete } from '../lib/users';
 import { getUserForCheck, emailCheck } from '../../../2fa/server/code';
 import { resetUserE2EEncriptionKey } from '../../../../server/lib/resetUserE2EKey';
+import { setUserStatus } from '../../../../imports/users-presence/server/activeUsers';
 
 API.v1.addRoute('users.create', { authRequired: true }, {
 	post() {
@@ -417,12 +418,14 @@ API.v1.addRoute('users.setStatus', { authRequired: true }, {
 			if (this.bodyParams.status) {
 				const validStatus = ['online', 'away', 'offline', 'busy'];
 				if (validStatus.includes(this.bodyParams.status)) {
+					const { status } = this.bodyParams;
 					Meteor.users.update(user._id, {
 						$set: {
-							status: this.bodyParams.status,
-							statusDefault: this.bodyParams.status,
+							status,
+							statusDefault: status,
 						},
 					});
+					setUserStatus(user, status);
 				} else {
 					throw new Meteor.Error('error-invalid-status', 'Valid status types include online, away, offline, and busy.', {
 						method: 'users.setStatus',

diff --git a/imports/users-presence/server/activeUsers.js b/imports/users-presence/server/activeUsers.js
@@ -11,7 +11,7 @@ export const STATUS_MAP = {
 	busy: 3,
 };
 
-const setUserStatus = (user, status/* , statusConnection*/) => {
+export const setUserStatus = (user, status/* , statusConnection*/) => {
 	const {
 		_id,
 		username,