fix regression from bigbluebutton#7606, handle roles correctly

symut · Jun 28, 2019 · 0de1d1d · 0de1d1d
1 parent f19ba4d
commit 0de1d1d
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 4 deletions.
diff --git a/bigbluebutton-html5/imports/api/users/server/handlers/userJoined.js b/bigbluebutton-html5/imports/api/users/server/handlers/userJoined.js
@@ -5,7 +5,6 @@ import addUser from '../modifiers/addUser';
 export default function handleUserJoined({ body }, meetingId) {
   const user = body;
 
-  user.moderator = user.moderator || false;
   check(user, Object);
 
   return addUser(meetingId, user);

diff --git a/bigbluebutton-html5/imports/api/users/server/modifiers/addUser.js b/bigbluebutton-html5/imports/api/users/server/modifiers/addUser.js
@@ -30,7 +30,6 @@ export default function addUser(meetingId, user) {
     guestStatus: String,
     emoji: String,
     presenter: Boolean,
-    moderator: Boolean,
     locked: Boolean,
     avatar: String,
     clientType: String,

diff --git a/bigbluebutton-html5/imports/ui/components/video-provider/many-users-notify/container.jsx b/bigbluebutton-html5/imports/ui/components/video-provider/many-users-notify/container.jsx
@@ -5,14 +5,18 @@ import Users from '/imports/api/users/';
 import LockViewersService from '/imports/ui/components/lock-viewers/service';
 import ManyUsersComponent from './component';
 
+const USER_CONFIG = Meteor.settings.public.user;
+const ROLE_MODERATOR = USER_CONFIG.role_moderator;
+const ROLE_VIEWER = USER_CONFIG.role_viewer;
+
 export default withTracker(() => ({
   viewersInWebcam: Users.find({
     meetingId: Auth.meetingID,
     hasStream: true,
-    moderator: false,
+    role: ROLE_VIEWER,
     presenter: false,
   }).count(),
-  currentUserIsModerator: Users.findOne({ userId: Auth.userID }).moderator,
+  currentUserIsModerator: Users.findOne({ userId: Auth.userID }).role === ROLE_MODERATOR,
   lockSettings: Meetings.findOne({ meetingId: Auth.meetingID }).lockSettingsProps,
   webcamOnlyForModerator: Meetings.findOne({
     meetingId: Auth.meetingID,