Repository: openmeetings
Updated Branches:
  refs/heads/master 9504ae3f4 -> 0eaffd3a2


[OPENMEETINGS-1673] recording test should be fixed


Project: http://git-wip-us.apache.org/repos/asf/openmeetings/repo
Commit: http://git-wip-us.apache.org/repos/asf/openmeetings/commit/0eaffd3a
Tree: http://git-wip-us.apache.org/repos/asf/openmeetings/tree/0eaffd3a
Diff: http://git-wip-us.apache.org/repos/asf/openmeetings/diff/0eaffd3a

Branch: refs/heads/master
Commit: 0eaffd3a24e3a89d16f0c2261bee3efb6daddda2
Parents: 9504ae3
Author: Maxim Solodovnik <[email protected]>
Authored: Mon Jul 24 20:07:10 2017 +0700
Committer: Maxim Solodovnik <[email protected]>
Committed: Mon Jul 24 20:07:10 2017 +0700

----------------------------------------------------------------------
 .../java/org/apache/openmeetings/db/entity/basic/Client.java     | 3 +++
 .../main/java/org/apache/openmeetings/web/room/RoomPanel.java    | 4 ++--
 2 files changed, 5 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/openmeetings/blob/0eaffd3a/openmeetings-db/src/main/java/org/apache/openmeetings/db/entity/basic/Client.java
----------------------------------------------------------------------
diff --git 
a/openmeetings-db/src/main/java/org/apache/openmeetings/db/entity/basic/Client.java
 
b/openmeetings-db/src/main/java/org/apache/openmeetings/db/entity/basic/Client.java
index d103bea..ed5424d 100644
--- 
a/openmeetings-db/src/main/java/org/apache/openmeetings/db/entity/basic/Client.java
+++ 
b/openmeetings-db/src/main/java/org/apache/openmeetings/db/entity/basic/Client.java
@@ -301,6 +301,9 @@ public class Client implements IClient {
        }
 
        public void removeStream(String broadcastId) {
+               if (broadcastId == null) {
+                       return;
+               }
                streams.remove(new Stream(1L, broadcastId, false));
        }
 

http://git-wip-us.apache.org/repos/asf/openmeetings/blob/0eaffd3a/openmeetings-web/src/main/java/org/apache/openmeetings/web/room/RoomPanel.java
----------------------------------------------------------------------
diff --git 
a/openmeetings-web/src/main/java/org/apache/openmeetings/web/room/RoomPanel.java
 
b/openmeetings-web/src/main/java/org/apache/openmeetings/web/room/RoomPanel.java
index f64d9f6..ee6c671 100644
--- 
a/openmeetings-web/src/main/java/org/apache/openmeetings/web/room/RoomPanel.java
+++ 
b/openmeetings-web/src/main/java/org/apache/openmeetings/web/room/RoomPanel.java
@@ -120,7 +120,7 @@ public class RoomPanel extends BasePanel {
                                        , cp.getRemoteAddress()
                                        , "" + r.getId());
                        //TODO add all broadcasting clients
-                       JSONObject options = VideoSettings.getInitJson(cp, "" + 
r.getId(), getClient().getUid());
+                       JSONObject options = VideoSettings.getInitJson(cp, "" + 
r.getId(), getClient().getSid());
                        options.put("interview", Room.Type.interview == 
r.getType());
                        
target.appendJavaScript(String.format("VideoManager.init(%s);", options));
                        WebSocketHelper.sendRoom(new RoomMessage(r.getId(), 
getUserId(), RoomMessage.Type.roomEnter));
@@ -435,7 +435,7 @@ public class RoomPanel extends BasePanel {
                                                        return;
                                                }
                                                if 
(getClient().getUid().equals(c.getUid())) {
-                                                       
c.removeStream(obj.getString("broadcastId"));
+                                                       
c.removeStream(obj.optString("broadcastId"));
                                                }
                                                
handler.appendJavaScript(String.format("VideoManager.close('%s');", 
obj.getString("uid")));
                                        }

Reply via email to