From 125e90582c862db29951c2741dac524b18c1f3e0 Mon Sep 17 00:00:00 2001
From: Lucas Fialho Zawacki <lfzawacki@gmail.com>
Date: Wed, 18 Jul 2018 18:12:34 -0300
Subject: [PATCH 1/6] Add processing of userdata parameters and correct bug
 while processing metadata

---
 .../imports/startup/client/auth.js            | 29 +++++++++++++++++--
 1 file changed, 27 insertions(+), 2 deletions(-)

diff --git a/bigbluebutton-html5/imports/startup/client/auth.js b/bigbluebutton-html5/imports/startup/client/auth.js
index 896e6370fc..087a2657b9 100755
--- a/bigbluebutton-html5/imports/startup/client/auth.js
+++ b/bigbluebutton-html5/imports/startup/client/auth.js
@@ -7,6 +7,7 @@ import deviceInfo from '/imports/utils/deviceInfo';
 // disconnected and trying to open a new connection
 const STATUS_CONNECTING = 'connecting';
 const METADATA_KEY = 'metadata';
+const CUSTOM_DATA_KEY = 'customdata';
 
 export function joinRouteHandler(nextState, replace, callback) {
   const { sessionToken } = nextState.location.query;
@@ -23,7 +24,7 @@ export function joinRouteHandler(nextState, replace, callback) {
     .then(response => response.json())
     .then(({ response }) => {
       const {
-        returncode, meetingID, internalUserID, authToken, logoutUrl, customLogoURL, metadata,
+        returncode, meetingID, internalUserID, authToken, logoutUrl, customLogoURL, metadata, customdata
       } = response;
 
       if (returncode === 'FAILED') {
@@ -32,6 +33,7 @@ export function joinRouteHandler(nextState, replace, callback) {
       }
 
       setCustomLogoUrl(customLogoURL);
+
       const metakeys = metadata.length
         ? metadata.reduce((acc, meta) => {
           const key = Object.keys(meta).shift();
@@ -52,8 +54,31 @@ export function joinRouteHandler(nextState, replace, callback) {
             log('error', `Caught: ${e.message}`);
           }
           return { ...acc, [key]: value };
-        }) : {};
+        }, {}) : {};
+
+      const customData = customdata.length
+        ? customdata.reduce((acc, data) => {
+          const key = Object.keys(data).shift();
+
+          const handledHTML5Parameters = [
+            'html5recordingbot'
+          ];
+          if (handledHTML5Parameters.indexOf(key) === -1) {
+            return acc;
+          }
+
+          let value = data[key];
+          try {
+            value = JSON.parse(value);
+          } catch (e) {
+            log('error', `Caught: ${e.message}`);
+          }
+
+          return { ...acc, [key]: value};
+        }, {}) : {};
+
       SessionStorage.setItem(METADATA_KEY, metakeys);
+      SessionStorage.setItem(CUSTOM_DATA_KEY, customData);
 
       Auth.set(meetingID, internalUserID, authToken, logoutUrl, sessionToken);
 

From ffcf35b7daebbf3e758a17a7f88d312e7934d50c Mon Sep 17 00:00:00 2001
From: Lucas Fialho Zawacki <lfzawacki@gmail.com>
Date: Thu, 19 Jul 2018 14:42:17 -0300
Subject: [PATCH 2/6] react key for screenshare video area

---
 .../imports/ui/components/screenshare/component.jsx           | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/bigbluebutton-html5/imports/ui/components/screenshare/component.jsx b/bigbluebutton-html5/imports/ui/components/screenshare/component.jsx
index 433e6adeb0..906e66c6aa 100755
--- a/bigbluebutton-html5/imports/ui/components/screenshare/component.jsx
+++ b/bigbluebutton-html5/imports/ui/components/screenshare/component.jsx
@@ -29,8 +29,8 @@ export default class ScreenshareComponent extends React.Component {
 
   render() {
     return (
-      [!this.state.loaded ? (<div className={styles.connecting} />) : null,
-        (<video id="screenshareVideo" style={{ maxHeight: '100%', width: '100%' }} autoPlay playsInline onLoadedData={this.onVideoLoad} />)]
+      [!this.state.loaded ? (<div key="screenshareArea" className={styles.connecting} />) : null,
+        (<video key="screenshareVideo" id="screenshareVideo" style={{ maxHeight: '100%', width: '100%' }} autoPlay playsInline onLoadedData={this.onVideoLoad} />)]
     );
   }
 }

From 8e023d57c9e9710c56b3e18a21314963b3eb11d4 Mon Sep 17 00:00:00 2001
From: Lucas Fialho Zawacki <lfzawacki@gmail.com>
Date: Thu, 19 Jul 2018 15:01:26 -0300
Subject: [PATCH 3/6] Ids for share video and share screen buttons

---
 .../ui/components/actions-bar/actions-dropdown/component.jsx    | 1 +
 .../ui/components/video-provider/video-menu/component.jsx       | 1 +
 .../ui/components/video-provider/video-menu/container.jsx       | 2 ++
 3 files changed, 4 insertions(+)

diff --git a/bigbluebutton-html5/imports/ui/components/actions-bar/actions-dropdown/component.jsx b/bigbluebutton-html5/imports/ui/components/actions-bar/actions-dropdown/component.jsx
index f36e9a8bd8..5d4d2fa075 100644
--- a/bigbluebutton-html5/imports/ui/components/actions-bar/actions-dropdown/component.jsx
+++ b/bigbluebutton-html5/imports/ui/components/actions-bar/actions-dropdown/component.jsx
@@ -118,6 +118,7 @@ class ActionsDropdown extends Component {
             intlMessages.stopDesktopShareDesc : intlMessages.desktopShareDesc)}
             key={this.videoItemId}
             onClick={isVideoBroadcasting ? handleUnshareScreen : handleShareScreen}
+            id={isVideoBroadcasting ? 'unshare-screen-button' : 'share-screen-button' }
           />
         : null),
       (record && isUserModerator && allowStartStopRecording ?
diff --git a/bigbluebutton-html5/imports/ui/components/video-provider/video-menu/component.jsx b/bigbluebutton-html5/imports/ui/components/video-provider/video-menu/component.jsx
index 867ba8f7ba..80c48a5029 100755
--- a/bigbluebutton-html5/imports/ui/components/video-provider/video-menu/component.jsx
+++ b/bigbluebutton-html5/imports/ui/components/video-provider/video-menu/component.jsx
@@ -48,6 +48,7 @@ const JoinVideoOptions = ({
           description={item.description}
           onClick={item.click}
           tabIndex={-1}
+          id={item.id}
         >
           <img src={item.iconPath} className={styles.imageSize} alt="video menu icon" />
           <span className={styles.label}>{item.label}</span>
diff --git a/bigbluebutton-html5/imports/ui/components/video-provider/video-menu/container.jsx b/bigbluebutton-html5/imports/ui/components/video-provider/video-menu/container.jsx
index 0648d047a0..2994d93fae 100755
--- a/bigbluebutton-html5/imports/ui/components/video-provider/video-menu/container.jsx
+++ b/bigbluebutton-html5/imports/ui/components/video-provider/video-menu/container.jsx
@@ -42,6 +42,7 @@ const JoinVideoOptionsContainer = (props) => {
       label: intl.formatMessage(intlMessages.swapCam),
       disabled: !swapLayoutAllowed,
       click: toggleSwapLayout,
+      id: 'swap-button',
     },
     {
       iconPath: `${baseName}/resources/images/video-menu/icon-webcam-off.svg`,
@@ -49,6 +50,7 @@ const JoinVideoOptionsContainer = (props) => {
       label: intl.formatMessage(intlMessages[isSharingVideo ? 'leaveVideo' : 'joinVideo']),
       disabled: isDisabled && !isSharingVideo,
       click: isSharingVideo ? handleCloseVideo : handleJoinVideo,
+      id: isSharingVideo ? 'leave-video-button' : 'join-video-button',
     },
   ];
 

From 75e72e9e72dc3f478f8cbeb403100f52ca335100 Mon Sep 17 00:00:00 2001
From: Anton Georgiev <anto.georgiev@gmail.com>
Date: Tue, 24 Jul 2018 21:09:17 +0000
Subject: [PATCH 4/6] resolve linting issues

---
 .../actions-bar/desktop-share/component.jsx   | 29 +++++++++----------
 1 file changed, 14 insertions(+), 15 deletions(-)

diff --git a/bigbluebutton-html5/imports/ui/components/actions-bar/desktop-share/component.jsx b/bigbluebutton-html5/imports/ui/components/actions-bar/desktop-share/component.jsx
index 3ab28abd21..504d7d9acb 100755
--- a/bigbluebutton-html5/imports/ui/components/actions-bar/desktop-share/component.jsx
+++ b/bigbluebutton-html5/imports/ui/components/actions-bar/desktop-share/component.jsx
@@ -43,21 +43,20 @@ const DesktopShare = ({
   isUserPresenter,
 }) => (
   (screenSharingCheck && !isMobileBrowser && isUserPresenter ?
-        <Button
-          className={styles.button}
-          icon="desktop"
-          label={intl.formatMessage(isVideoBroadcasting ?
-          intlMessages.stopDesktopShareLabel : intlMessages.desktopShareLabel)}
-          description={intl.formatMessage(isVideoBroadcasting ?
-          intlMessages.stopDesktopShareDesc : intlMessages.desktopShareDesc)}
-          color="primary"
-          icon="desktop"
-          ghost={false}
-          hideLabel
-          circle
-          size="lg"
-          onClick={isVideoBroadcasting ? handleUnshareScreen : handleShareScreen}
-        />
+    <Button
+      className={styles.button}
+      icon="desktop"
+      label={intl.formatMessage(isVideoBroadcasting ?
+      intlMessages.stopDesktopShareLabel : intlMessages.desktopShareLabel)}
+      description={intl.formatMessage(isVideoBroadcasting ?
+      intlMessages.stopDesktopShareDesc : intlMessages.desktopShareDesc)}
+      color="primary"
+      ghost={false}
+      hideLabel
+      circle
+      size="lg"
+      onClick={isVideoBroadcasting ? handleUnshareScreen : handleShareScreen}
+    />
     : null)
 );
 

From 5e750bb3c6d9d5a66a3feec471016b11ba249185 Mon Sep 17 00:00:00 2001
From: Anton Georgiev <anto.georgiev@gmail.com>
Date: Tue, 24 Jul 2018 21:10:30 +0000
Subject: [PATCH 5/6] resolve conflicts for 5822

---
 .../ui/components/actions-bar/desktop-share/component.jsx        | 1 +
 1 file changed, 1 insertion(+)

diff --git a/bigbluebutton-html5/imports/ui/components/actions-bar/desktop-share/component.jsx b/bigbluebutton-html5/imports/ui/components/actions-bar/desktop-share/component.jsx
index 504d7d9acb..8e9d680d6e 100755
--- a/bigbluebutton-html5/imports/ui/components/actions-bar/desktop-share/component.jsx
+++ b/bigbluebutton-html5/imports/ui/components/actions-bar/desktop-share/component.jsx
@@ -56,6 +56,7 @@ const DesktopShare = ({
       circle
       size="lg"
       onClick={isVideoBroadcasting ? handleUnshareScreen : handleShareScreen}
+      id={isVideoBroadcasting ? 'unshare-screen-button' : 'share-screen-button'}
     />
     : null)
 );

From b4b4f329292870cc50b7617519639583601b6962 Mon Sep 17 00:00:00 2001
From: Anton Georgiev <anto.georgiev@gmail.com>
Date: Tue, 24 Jul 2018 22:34:53 +0000
Subject: [PATCH 6/6] pass key->value pairs in "customdata" array

---
 .../org/bigbluebutton/web/controllers/ApiController.groovy     | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/bigbluebutton-web/grails-app/controllers/org/bigbluebutton/web/controllers/ApiController.groovy b/bigbluebutton-web/grails-app/controllers/org/bigbluebutton/web/controllers/ApiController.groovy
index 338a2e5b2b..3fc7319b82 100755
--- a/bigbluebutton-web/grails-app/controllers/org/bigbluebutton/web/controllers/ApiController.groovy
+++ b/bigbluebutton-web/grails-app/controllers/org/bigbluebutton/web/controllers/ApiController.groovy
@@ -1388,8 +1388,7 @@ class ApiController {
               defaultLayout = us.defaultLayout
               avatarURL = us.avatarURL
               customdata = array {
-                userCustomData.each { k, v ->
-                  // Somehow we need to prepend something (custdata) for the JSON to work
+                meeting.getUserCustomData(us.externUserID).each { k, v ->
                   custdata "$k" : v
                 }
               }