From 06400c703477a3e0de44f5d7b0a66a3c9737a4d9 Mon Sep 17 00:00:00 2001
From: gcampes <gabrieldecampes@gmail.com>
Date: Thu, 21 Sep 2017 15:50:21 -0300
Subject: [PATCH 1/3] fix breakout logout

---
 .../logout-confirmation/component.jsx         | 76 ++++++++++---------
 .../logout-confirmation/container.jsx         | 20 ++---
 .../components/logout-confirmation/service.js |  9 ++-
 3 files changed, 56 insertions(+), 49 deletions(-)

diff --git a/bigbluebutton-html5/imports/ui/components/logout-confirmation/component.jsx b/bigbluebutton-html5/imports/ui/components/logout-confirmation/component.jsx
index 7ed2ae2461..4c035d5cc9 100644
--- a/bigbluebutton-html5/imports/ui/components/logout-confirmation/component.jsx
+++ b/bigbluebutton-html5/imports/ui/components/logout-confirmation/component.jsx
@@ -1,9 +1,17 @@
-import React, { Component } from 'react';
+import React from 'react';
+import PropTypes from 'prop-types';
 import { withRouter } from 'react-router';
 import { defineMessages, injectIntl } from 'react-intl';
 import Button from '/imports/ui/components/button/component';
 import Modal from '/imports/ui/components/modal/fullscreen/component';
-import styles from './styles.scss';
+import styles from './styles';
+
+const propTypes = {
+  handleEndMeeting: PropTypes.func.isRequired,
+  intl: PropTypes.object.isRequired,
+  router: PropTypes.object.isRequired,
+  showEndMeeting: PropTypes.bool.isRequired,
+};
 
 const intlMessages = defineMessages({
   title: {
@@ -40,38 +48,38 @@ const intlMessages = defineMessages({
   },
 });
 
-class LeaveConfirmation extends Component {
+const LeaveConfirmation = ({
+  intl,
+  router,
+  handleEndMeeting,
+  showEndMeeting,
+}) => (
+  <Modal
+    title={intl.formatMessage(intlMessages.title)}
+    confirm={{
+      callback: () => router.push('/logout'),
+      label: intl.formatMessage(intlMessages.confirmLabel),
+      description: intl.formatMessage(intlMessages.confirmDesc),
+    }}
+    dismiss={{
+      callback: () => null,
+      label: intl.formatMessage(intlMessages.dismissLabel),
+      description: intl.formatMessage(intlMessages.dismissDesc),
+    }}
+  >
+    {intl.formatMessage(intlMessages.message)}
+    {showEndMeeting ?
+      <Button
+        className={styles.endMeeting}
+        label={intl.formatMessage(intlMessages.endMeetingLabel)}
+        onClick={handleEndMeeting}
+        aria-describedby={'modalEndMeetingDesc'}
+      /> : null
+    }
+    <div id="modalEndMeetingDesc" hidden>{intl.formatMessage(intlMessages.endMeetingDesc)}</div>
+  </Modal>
+);
 
-  render() {
-    const { intl, router, endMeeting, isModerator } = this.props;
-
-    return (
-      <Modal
-        title={intl.formatMessage(intlMessages.title)}
-        confirm={{
-          callback: () => router.push('/logout'),
-          label: intl.formatMessage(intlMessages.confirmLabel),
-          description: intl.formatMessage(intlMessages.confirmDesc),
-        }}
-        dismiss={{
-          callback: () => null,
-          label: intl.formatMessage(intlMessages.dismissLabel),
-          description: intl.formatMessage(intlMessages.dismissDesc),
-        }}
-      >
-        {intl.formatMessage(intlMessages.message)}
-        {isModerator ?
-          <Button
-            className={styles.endMeeting}
-            label={intl.formatMessage(intlMessages.endMeetingLabel)}
-            onClick={endMeeting}
-            aria-describedby={'modalEndMeetingDesc'}
-          /> : null
-        }
-        <div id="modalEndMeetingDesc" hidden>{intl.formatMessage(intlMessages.endMeetingDesc)}</div>
-      </Modal>
-    );
-  }
-}
+LeaveConfirmation.propTypes = propTypes;
 
 export default withRouter(injectIntl(LeaveConfirmation));
diff --git a/bigbluebutton-html5/imports/ui/components/logout-confirmation/container.jsx b/bigbluebutton-html5/imports/ui/components/logout-confirmation/container.jsx
index ded0963894..3865bd746a 100644
--- a/bigbluebutton-html5/imports/ui/components/logout-confirmation/container.jsx
+++ b/bigbluebutton-html5/imports/ui/components/logout-confirmation/container.jsx
@@ -1,22 +1,14 @@
-import React, { Component } from 'react';
+import React from 'react';
 import { createContainer } from 'meteor/react-meteor-data';
 import LogoutConfirmation from './component';
 import LogoutConfirmationService from './service';
 
-class LogoutConfirmationContainer extends Component {
-
-  constructor(props) {
-    super(props);
-  }
-
-  render() {
-    return (
-      <LogoutConfirmation {...this.props} />
-    );
-  }
-}
+const LogoutConfirmationContainer = () => (
+  <LogoutConfirmation {...this.props} />
+);
 
 export default createContainer(() => ({
-  isModerator: LogoutConfirmationService.isModerator(),
+  showEndMeeting: !LogoutConfirmationService.isBreakout() &&
+                  LogoutConfirmationService.isModerator(),
   endMeeting: LogoutConfirmationService.endMeeting,
 }), LogoutConfirmationContainer);
diff --git a/bigbluebutton-html5/imports/ui/components/logout-confirmation/service.js b/bigbluebutton-html5/imports/ui/components/logout-confirmation/service.js
index 44a6037173..3779ffcafa 100644
--- a/bigbluebutton-html5/imports/ui/components/logout-confirmation/service.js
+++ b/bigbluebutton-html5/imports/ui/components/logout-confirmation/service.js
@@ -1,5 +1,6 @@
-import { makeCall } from '/imports/ui/services/api/index.js';
+import { makeCall } from '/imports/ui/services/api/index';
 import Users from '/imports/api/2.0/users';
+import Breakouts from '/imports/api/2.0/breakouts';
 import mapUser from '/imports/ui/services/user/mapUser';
 import Auth from '/imports/ui/services/auth';
 
@@ -14,7 +15,13 @@ const isModerator = () => {
   return (currentUser) ? mapUser(currentUser).isModerator : null;
 };
 
+const isBreakout = () => {
+  console.log('isBreakout', Breakouts.findOne({ breakoutId: Auth.meetingID }));
+  return Breakouts.findOne({ breakoutId: Auth.meetingID });
+};
+
 export default {
   endMeeting,
   isModerator,
+  isBreakout,
 };

From 56e9fedb456409e7d840e53d6db8ecce6f0942ff Mon Sep 17 00:00:00 2001
From: gcampes <gabrieldecampes@gmail.com>
Date: Thu, 21 Sep 2017 15:51:58 -0300
Subject: [PATCH 2/3] removes console.log

---
 .../imports/ui/components/logout-confirmation/service.js     | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/bigbluebutton-html5/imports/ui/components/logout-confirmation/service.js b/bigbluebutton-html5/imports/ui/components/logout-confirmation/service.js
index 3779ffcafa..a3f2312ae2 100644
--- a/bigbluebutton-html5/imports/ui/components/logout-confirmation/service.js
+++ b/bigbluebutton-html5/imports/ui/components/logout-confirmation/service.js
@@ -15,10 +15,7 @@ const isModerator = () => {
   return (currentUser) ? mapUser(currentUser).isModerator : null;
 };
 
-const isBreakout = () => {
-  console.log('isBreakout', Breakouts.findOne({ breakoutId: Auth.meetingID }));
-  return Breakouts.findOne({ breakoutId: Auth.meetingID });
-};
+const isBreakout = () => Breakouts.findOne({ breakoutId: Auth.meetingID });
 
 export default {
   endMeeting,

From b17f2ebe4173687c764322ea2878c4f1561b782c Mon Sep 17 00:00:00 2001
From: gcampes <gabrieldecampes@gmail.com>
Date: Fri, 22 Sep 2017 11:33:12 -0300
Subject: [PATCH 3/3] fix proptypes for intl

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

diff --git a/bigbluebutton-html5/imports/ui/components/logout-confirmation/component.jsx b/bigbluebutton-html5/imports/ui/components/logout-confirmation/component.jsx
index 4c035d5cc9..2b214c2b65 100644
--- a/bigbluebutton-html5/imports/ui/components/logout-confirmation/component.jsx
+++ b/bigbluebutton-html5/imports/ui/components/logout-confirmation/component.jsx
@@ -1,14 +1,14 @@
 import React from 'react';
 import PropTypes from 'prop-types';
 import { withRouter } from 'react-router';
-import { defineMessages, injectIntl } from 'react-intl';
+import { defineMessages, injectIntl, intlShape } from 'react-intl';
 import Button from '/imports/ui/components/button/component';
 import Modal from '/imports/ui/components/modal/fullscreen/component';
 import styles from './styles';
 
 const propTypes = {
   handleEndMeeting: PropTypes.func.isRequired,
-  intl: PropTypes.object.isRequired,
+  intl: PropTypes.shape(intlShape).isRequired,
   router: PropTypes.object.isRequired,
   showEndMeeting: PropTypes.bool.isRequired,
 };