@wazo/sdk 0.37.28 → 0.37.29

package/lib/simple/room/SipRoom.js

@@ -3258,6 +3258,7 @@ var updateFrom = (instance     , from     ) => {
 
                              
                     
+                     
                             
               
                  
@@ -3305,14 +3306,14 @@ class CallSession {
 
                       
 
+                     // = creationTime
+
                     
 
                  
 
                     
 
-                    
-
                             
 
                           
@@ -3376,6 +3377,7 @@ class CallSession {
     recordingPaused,
     videoRemotelyDowngraded,
     sipSession,
+    answerTime,
   }                      ) {
     this.callId = callId;
     this.sipCallId = sipCallId;
@@ -3403,6 +3405,7 @@ class CallSession {
     this.recordingPaused = recordingPaused || false;
     this.videoRemotelyDowngraded = videoRemotelyDowngraded;
     this.sipSession = sipSession;
+    this.answerTime = answerTime || this.answerTime;
 
     // Useful to compare instead of instanceof with minified code
     this.type = 'CallSession';
@@ -3433,7 +3436,7 @@ class CallSession {
   }
 
   answer() {
-    this.answered = true;
+    this.answerTime = new Date();
   }
 
   systemAnswer() {
@@ -3589,6 +3592,15 @@ class CallSession {
     return newFrom(callSession, CallSession);
   }
 
+  // Retro-compatibility: `answered` was a boolean before. We can reproduce the behaviour with a getter/setter
+  set answered(value         ) {
+    this.answerTime = value ? new Date() : null;
+  }
+
+  get answered()          {
+    return !!this.answerTime;
+  }
+
   static parseCall(call      )              {
     return new CallSession({
       callId: call.id,
@@ -9346,6 +9358,7 @@ class WebRTCPhone extends Emitter                  {
       this.removeIncomingSessions(sipSessionId);
     }
 
+    callSession.answerTime = new Date();
     this.eventEmitter.emit(ON_CALL_ACCEPTED, callSession, cameraEnabled);
 
     return callSession;
@@ -9561,6 +9574,7 @@ class WebRTCPhone extends Emitter                  {
     }
 
     this.shouldSendReinvite = false;
+    callSession.answerTime = new Date();
     this.acceptedSessions[callSession.getId()] = true;
 
     this.eventEmitter.emit(ON_CALL_ANSWERED, callSession);

package/lib/simple/room/Stream.js

@@ -3258,6 +3258,7 @@ var updateFrom = (instance     , from     ) => {
 
                              
                     
+                     
                             
               
                  
@@ -3305,14 +3306,14 @@ class CallSession {
 
                       
 
+                     // = creationTime
+
                     
 
                  
 
                     
 
-                    
-
                             
 
                           
@@ -3376,6 +3377,7 @@ class CallSession {
     recordingPaused,
     videoRemotelyDowngraded,
     sipSession,
+    answerTime,
   }                      ) {
     this.callId = callId;
     this.sipCallId = sipCallId;
@@ -3403,6 +3405,7 @@ class CallSession {
     this.recordingPaused = recordingPaused || false;
     this.videoRemotelyDowngraded = videoRemotelyDowngraded;
     this.sipSession = sipSession;
+    this.answerTime = answerTime || this.answerTime;
 
     // Useful to compare instead of instanceof with minified code
     this.type = 'CallSession';
@@ -3433,7 +3436,7 @@ class CallSession {
   }
 
   answer() {
-    this.answered = true;
+    this.answerTime = new Date();
   }
 
   systemAnswer() {
@@ -3589,6 +3592,15 @@ class CallSession {
     return newFrom(callSession, CallSession);
   }
 
+  // Retro-compatibility: `answered` was a boolean before. We can reproduce the behaviour with a getter/setter
+  set answered(value         ) {
+    this.answerTime = value ? new Date() : null;
+  }
+
+  get answered()          {
+    return !!this.answerTime;
+  }
+
   static parseCall(call      )              {
     return new CallSession({
       callId: call.id,
@@ -9346,6 +9358,7 @@ class WebRTCPhone extends Emitter                  {
       this.removeIncomingSessions(sipSessionId);
     }
 
+    callSession.answerTime = new Date();
     this.eventEmitter.emit(ON_CALL_ACCEPTED, callSession, cameraEnabled);
 
     return callSession;
@@ -9561,6 +9574,7 @@ class WebRTCPhone extends Emitter                  {
     }
 
     this.shouldSendReinvite = false;
+    callSession.answerTime = new Date();
     this.acceptedSessions[callSession.getId()] = true;
 
     this.eventEmitter.emit(ON_CALL_ANSWERED, callSession);

package/lib/simple/room/__tests__/RemoteParticipant.test.js

@@ -3258,6 +3258,7 @@ var updateFrom = (instance     , from     ) => {
 
                              
                     
+                     
                             
               
                  
@@ -3305,14 +3306,14 @@ class CallSession {
 
                       
 
+                     // = creationTime
+
                     
 
                  
 
                     
 
-                    
-
                             
 
                           
@@ -3376,6 +3377,7 @@ class CallSession {
     recordingPaused,
     videoRemotelyDowngraded,
     sipSession,
+    answerTime,
   }                      ) {
     this.callId = callId;
     this.sipCallId = sipCallId;
@@ -3403,6 +3405,7 @@ class CallSession {
     this.recordingPaused = recordingPaused || false;
     this.videoRemotelyDowngraded = videoRemotelyDowngraded;
     this.sipSession = sipSession;
+    this.answerTime = answerTime || this.answerTime;
 
     // Useful to compare instead of instanceof with minified code
     this.type = 'CallSession';
@@ -3433,7 +3436,7 @@ class CallSession {
   }
 
   answer() {
-    this.answered = true;
+    this.answerTime = new Date();
   }
 
   systemAnswer() {
@@ -3589,6 +3592,15 @@ class CallSession {
     return newFrom(callSession, CallSession);
   }
 
+  // Retro-compatibility: `answered` was a boolean before. We can reproduce the behaviour with a getter/setter
+  set answered(value         ) {
+    this.answerTime = value ? new Date() : null;
+  }
+
+  get answered()          {
+    return !!this.answerTime;
+  }
+
   static parseCall(call      )              {
     return new CallSession({
       callId: call.id,
@@ -9346,6 +9358,7 @@ class WebRTCPhone extends Emitter                  {
       this.removeIncomingSessions(sipSessionId);
     }
 
+    callSession.answerTime = new Date();
     this.eventEmitter.emit(ON_CALL_ACCEPTED, callSession, cameraEnabled);
 
     return callSession;
@@ -9561,6 +9574,7 @@ class WebRTCPhone extends Emitter                  {
     }
 
     this.shouldSendReinvite = false;
+    callSession.answerTime = new Date();
     this.acceptedSessions[callSession.getId()] = true;
 
     this.eventEmitter.emit(ON_CALL_ANSWERED, callSession);

package/lib/simple/utils.js

@@ -3260,6 +3260,7 @@ var updateFrom = (instance     , from     ) => {
 
                              
                     
+                     
                             
               
                  
@@ -3307,14 +3308,14 @@ class CallSession {
 
                       
 
+                     // = creationTime
+
                     
 
                  
 
                     
 
-                    
-
                             
 
                           
@@ -3378,6 +3379,7 @@ class CallSession {
     recordingPaused,
     videoRemotelyDowngraded,
     sipSession,
+    answerTime,
   }                      ) {
     this.callId = callId;
     this.sipCallId = sipCallId;
@@ -3405,6 +3407,7 @@ class CallSession {
     this.recordingPaused = recordingPaused || false;
     this.videoRemotelyDowngraded = videoRemotelyDowngraded;
     this.sipSession = sipSession;
+    this.answerTime = answerTime || this.answerTime;
 
     // Useful to compare instead of instanceof with minified code
     this.type = 'CallSession';
@@ -3435,7 +3438,7 @@ class CallSession {
   }
 
   answer() {
-    this.answered = true;
+    this.answerTime = new Date();
   }
 
   systemAnswer() {
@@ -3591,6 +3594,15 @@ class CallSession {
     return newFrom(callSession, CallSession);
   }
 
+  // Retro-compatibility: `answered` was a boolean before. We can reproduce the behaviour with a getter/setter
+  set answered(value         ) {
+    this.answerTime = value ? new Date() : null;
+  }
+
+  get answered()          {
+    return !!this.answerTime;
+  }
+
   static parseCall(call      )              {
     return new CallSession({
       callId: call.id,
@@ -9348,6 +9360,7 @@ class WebRTCPhone extends Emitter                  {
       this.removeIncomingSessions(sipSessionId);
     }
 
+    callSession.answerTime = new Date();
     this.eventEmitter.emit(ON_CALL_ACCEPTED, callSession, cameraEnabled);
 
     return callSession;
@@ -9563,6 +9576,7 @@ class WebRTCPhone extends Emitter                  {
     }
 
     this.shouldSendReinvite = false;
+    callSession.answerTime = new Date();
     this.acceptedSessions[callSession.getId()] = true;
 
     this.eventEmitter.emit(ON_CALL_ANSWERED, callSession);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wazo/sdk",
-  "version": "0.37.28",
+  "version": "0.37.29",
   "description": "Wazo's JavaScript Software Development Kit.",
   "main": "dist/wazo-sdk.js",
   "author": "Wazo (http://wazo.io)",

package/src/domain/CallSession.js

@@ -8,6 +8,7 @@ import updateFrom from '../utils/update-from';
 
 type CallSessionArguments = {
   answered: boolean,
+  answerTime?: ?Date,
   answeredBySystem: boolean,
   call: ?Call,
   callId: string,
@@ -55,14 +56,14 @@ export default class CallSession {
 
   creationTime: ?Date;
 
-  startTime: number;
+  startTime: number; // = creationTime
+
+  answerTime: ?Date;
 
   endTime: ?Date;
 
   isCaller: boolean;
 
-  answered: boolean;
-
   answeredBySystem: boolean;
 
   dialedExtension: string;
@@ -126,6 +127,7 @@ export default class CallSession {
     recordingPaused,
     videoRemotelyDowngraded,
     sipSession,
+    answerTime,
   }: CallSessionArguments) {
     this.callId = callId;
     this.sipCallId = sipCallId;
@@ -153,6 +155,7 @@ export default class CallSession {
     this.recordingPaused = recordingPaused || false;
     this.videoRemotelyDowngraded = videoRemotelyDowngraded;
     this.sipSession = sipSession;
+    this.answerTime = answerTime || this.answerTime;
 
     // Useful to compare instead of instanceof with minified code
     this.type = 'CallSession';
@@ -183,7 +186,7 @@ export default class CallSession {
   }
 
   answer() {
-    this.answered = true;
+    this.answerTime = new Date();
   }
 
   systemAnswer() {
@@ -339,6 +342,15 @@ export default class CallSession {
     return newFrom(callSession, CallSession);
   }
 
+  // Retro-compatibility: `answered` was a boolean before. We can reproduce the behaviour with a getter/setter
+  set answered(value: boolean) {
+    this.answerTime = value ? new Date() : null;
+  }
+
+  get answered(): boolean {
+    return !!this.answerTime;
+  }
+
   static parseCall(call: Call): CallSession {
     return new CallSession({
       callId: call.id,

package/src/domain/Phone/WebRTCPhone.js

@@ -630,6 +630,7 @@ export default class WebRTCPhone extends Emitter implements Phone {
       this.removeIncomingSessions(sipSessionId);
     }
 
+    callSession.answerTime = new Date();
     this.eventEmitter.emit(ON_CALL_ACCEPTED, callSession, cameraEnabled);
 
     return callSession;
@@ -845,6 +846,7 @@ export default class WebRTCPhone extends Emitter implements Phone {
     }
 
     this.shouldSendReinvite = false;
+    callSession.answerTime = new Date();
     this.acceptedSessions[callSession.getId()] = true;
 
     this.eventEmitter.emit(ON_CALL_ANSWERED, callSession);

package/src/domain/__tests__/CallSession.test.js

@@ -37,4 +37,21 @@ describe('CallSession domain', () => {
     expect(cs1.is(cs2)).toBeTruthy();
     expect(cs2.is(cs1)).toBeTruthy();
   });
+
+  it('should set answerTime when setting answered', () => {
+    const cs1 = new CallSession({});
+    cs1.answered = true;
+
+    const cs2 = new CallSession({});
+    cs2.answer();
+
+    const cs3 = new CallSession({ answered: true });
+
+    expect(cs1.answered).toBeTruthy();
+    expect(cs1.answerTime).not.toBeNull();
+    expect(cs2.answered).toBeTruthy();
+    expect(cs2.answerTime).not.toBeNull();
+    expect(cs3.answered).toBeTruthy();
+    expect(cs3.answerTime).not.toBeNull();
+  });
 });