Fix jibri presence initial state

JitsiConference.js

@@ -710,18 +710,14 @@ JitsiConference.prototype.isRecordingSupported = function () {
  * and "off" if the recording is not started.
  */
 JitsiConference.prototype.getRecordingState = function () {
-    if(this.room)
-        return this.room.getRecordingState();
-    return Recording.status.OFF;
+    return (this.room) ? this.room.getRecordingState() : undefined;
 }
 
 /**
  * Returns the url of the recorded video.
  */
 JitsiConference.prototype.getRecordingURL = function () {
-    if(this.room)
-        return this.room.getRecordingURL();
-    return null;
+    return (this.room) ? this.room.getRecordingURL() : null;
 }
 
 /**

modules/xmpp/ChatRoom.js

@@ -740,19 +740,15 @@ ChatRoom.prototype.isRecordingSupported = function () {
  * and "off" if the recording is not started.
  */
 ChatRoom.prototype.getRecordingState = function () {
-    if(this.recording)
-        return this.recording.getState();
-    return "off";
-};
+    return (this.recording) ? this.recording.getState() : undefined;
+}
 
 /**
  * Returns the url of the recorded video.
  */
 ChatRoom.prototype.getRecordingURL = function () {
-    if(this.recording)
-        return this.recording.getURL();
-    return null;
-};
+    return (this.recording) ? this.recording.getURL() : null;
+}
 
 /**
  * Starts/stops the recording

modules/xmpp/recording.js

@@ -16,11 +16,10 @@ function Recording(type, eventEmitter, connection, focusMucJid, jirecon,
     this.type = type;
     this._isSupported
         = ( type === Recording.types.JIRECON && !this.jirecon
-            || type === Recording.types.JIBRI && !this._isServiceAvailable)
+            || (type !== Recording.types.JIBRI
+                && type !== Recording.types.COLIBRI))
             ? false : true;
 
-    this._isServiceAvailable = false;
-
     /**
      * The ID of the jirecon recording session. Jirecon generates it when we
      * initially start recording, and it needs to be used in subsequent requests
@@ -52,25 +51,20 @@ Recording.prototype.handleJibriPresence = function (jibri) {
         return;
 
     var newState = attributes.status;
-    console.log("handle jibri presence : ", newState);
-    var oldIsAvailable = this._isServiceAvailable;
-    // The service is available if the statu isn't undefined.
-    this._isServiceAvailable =
-        (newState && newState !== "undefined");
-
-    if (newState === "undefined"
-        || oldIsAvailable != this._isServiceAvailable
-        // If we receive an OFF state without any recording in progress we
-        // consider this to be an initial available state.
-        || (this.state === Recording.status.AVAILABLE
-            && newState === Recording.status.OFF))
-        this.state = (newState === "undefined")
-                        ? Recording.status.UNAVAILABLE
-                        : Recording.status.AVAILABLE;
-    else
-        this.state = attributes.status;
-
-    logger.log("Handle Jibri presence: ", this.state);
+    console.log("Handle jibri presence : ", newState);
+
+    if (newState === this.state)
+        return;
+
+    if (newState === "undefined") {
+        this.state = Recording.status.UNAVAILABLE;
+    }
+    else if (newState === "off") {
+        this.state = Recording.status.AVAILABLE;
+    }
+    else {
+        this.state = newState;
+    }
 
     this.eventEmitter.emit(XMPPEvents.RECORDER_STATE_CHANGED, this.state);
 };
@@ -255,6 +249,7 @@ Recording.prototype.toggleRecording = function (options, statusChangeHandler) {
  * Returns true if the recording is supproted and false if not.
  */
 Recording.prototype.isSupported = function () {
+    console.log("IS SUPPORTED", this._isSupported);
     return this._isSupported;
 };