From 199558b6b934aad57fd9158de7a766e98a846d1a Mon Sep 17 00:00:00 2001
From: Ghazi Triki <ghazi.triki@riadvice.tn>
Date: Sat, 21 Jul 2018 18:21:19 +0100
Subject: [PATCH] Fix pen only multi-user whiteboard  and make it work for late
 joiners.

---
 .../modules/whiteboard/views/WhiteboardCanvas.as |  2 --
 .../whiteboard/views/WhiteboardToolbar.mxml      | 16 +++++++++++-----
 2 files changed, 11 insertions(+), 7 deletions(-)

diff --git a/bigbluebutton-client/src/org/bigbluebutton/modules/whiteboard/views/WhiteboardCanvas.as b/bigbluebutton-client/src/org/bigbluebutton/modules/whiteboard/views/WhiteboardCanvas.as
index bd8760e5b8..71c8b5934f 100755
--- a/bigbluebutton-client/src/org/bigbluebutton/modules/whiteboard/views/WhiteboardCanvas.as
+++ b/bigbluebutton-client/src/org/bigbluebutton/modules/whiteboard/views/WhiteboardCanvas.as
@@ -26,8 +26,6 @@ package org.bigbluebutton.modules.whiteboard.views {
 	import flash.events.MouseEvent;
 	import flash.geom.Point;
 	
-	import melomel.core.UI;
-	
 	import mx.containers.Canvas;
 	import mx.core.IChildList;
 	import mx.managers.CursorManager;
diff --git a/bigbluebutton-client/src/org/bigbluebutton/modules/whiteboard/views/WhiteboardToolbar.mxml b/bigbluebutton-client/src/org/bigbluebutton/modules/whiteboard/views/WhiteboardToolbar.mxml
index e7713b1505..8d512a1a73 100755
--- a/bigbluebutton-client/src/org/bigbluebutton/modules/whiteboard/views/WhiteboardToolbar.mxml
+++ b/bigbluebutton-client/src/org/bigbluebutton/modules/whiteboard/views/WhiteboardToolbar.mxml
@@ -60,6 +60,7 @@ with BigBlueButton; if not, see <http://www.gnu.org/licenses/>.
 		import org.bigbluebutton.modules.whiteboard.events.StopWhiteboardModuleEvent;
 		import org.bigbluebutton.modules.whiteboard.events.WhiteboardAccessEvent;
 		import org.bigbluebutton.modules.whiteboard.events.WhiteboardButtonEvent;
+		import org.bigbluebutton.modules.whiteboard.models.AnnotationType;
 		import org.bigbluebutton.modules.whiteboard.views.models.WhiteboardOptions;
 		import org.bigbluebutton.util.i18n.ResourceUtil;
 			
@@ -96,9 +97,13 @@ with BigBlueButton; if not, see <http://www.gnu.org/licenses/>.
           changeThickness();
           changeColor();
         }
+
+        penOnlyMultiUser();
       }
       
       public function setCanvas(c:WhiteboardCanvas):void {
+        wbOptions = Options.getOptions(WhiteboardOptions) as WhiteboardOptions;
+
         canvas = c;
         
         if (cpik) {
@@ -199,7 +204,6 @@ with BigBlueButton; if not, see <http://www.gnu.org/licenses/>.
 			}
 			
 			public function positionToolbar(container:MDIWindow):void {
-				wbOptions = Options.getOptions(WhiteboardOptions) as WhiteboardOptions;
         		LOGGER.debug("Positioning whiteboard toolbar");
 				containerToOverlay = container;
 				containerToOverlay.addEventListener(MoveEvent.MOVE, setPositionAndDepth);
@@ -229,10 +233,12 @@ with BigBlueButton; if not, see <http://www.gnu.org/licenses/>.
                 } else {
                     hideToolbar();
                 }
-				if (multiUser && wbOptions.onlyPenMUW) {
-					scribbleBtn.dispatchEvent(new MouseEvent(MouseEvent.CLICK));
-				} else if (!multiUser) {
-					
+                penOnlyMultiUser();
+            }
+
+            private function penOnlyMultiUser() : void {
+				if (multiUser && wbOptions.onlyPenMUW && !isPresenter) {
+					setToolType(WhiteboardConstants.TYPE_SHAPE, AnnotationType.PENCIL);
 				}
             }