@babblevoice/babble-drachtio-callmanager 2.2.7 → 2.3.1

package/lib/call.js

@@ -130,6 +130,7 @@ class call {
       @property { boolean } authed
       @property { boolean } cleaned
       @property { boolean } refered
+      @property { boolean } picked
     */
 
     /** @member { callstate } */
@@ -143,7 +144,8 @@ class call {
       "held": false,
       "authed": false,
       "cleaned": false,
-      "refered": false
+      "refered": false,
+      "picked": false
     }
 
     /**
@@ -873,6 +875,7 @@ class call {
   It wuld be normal to bridge this call to another after this call has been made.
   */
   pick() {
+    this.state.picked = true
     this._em.emit( "call.pick", this )
   }
 

package/lib/store.js

@@ -3,7 +3,7 @@
 let storebycallid = new Map()
 let storebyuuid = new Map()
 let storebyentity = new Map()
-let storebyentityrealm = new Map()
+let storebyrealm = new Map()
 
 /** @module store */
 
@@ -104,6 +104,7 @@ function fixentity( c ) {
  *  @param {object} c.entity - entity object
  *  @param {object} c.entity.uri
  *  @param {object} c._entity
+ *  @param {object} c.destination
  *  @param {object} [ c._state ]
  *  @param {boolean} [ c._state._hangup ]
  *  @return {Promise} - which resolves on completion (for future in case we support redis or similar)
@@ -118,21 +119,29 @@ module.exports.set = async ( c ) => {
   storebyuuid.set( c.uuid, c )
   addtocallidset( c )
 
+  let realm = ""
+  const destination = c.destination
+  if( destination ) realm = c.destination.host
+
   const entity = fixentity( c )
-  if( "boolean" == typeof entity ) return entity
+  if( "boolean" !== typeof entity ) {
+
+    let storedentity = storebyentity.get( entity.uri )
+    if( !storedentity ) {
+      storedentity = new Map()
+      storebyentity.set( entity.uri, storedentity )
+    }
 
-  let storedentity = storebyentity.get( entity.uri )
-  if( !storedentity ) {
-    storedentity = new Map()
-    storebyentity.set( entity.uri, storedentity )
+    storedentity.set( c.uuid, c )
+    realm = entity.realm
   }
 
-  storedentity.set( c.uuid, c )
+  if( !realm ) return true
 
-  let storedentityrealm = storebyentityrealm.get( entity.realm )
+  let storedentityrealm = storebyrealm.get( realm )
   if( !storedentityrealm ) {
     storedentityrealm = new Map()
-    storebyentityrealm.set( entity.realm, storedentityrealm )
+    storebyrealm.set( realm, storedentityrealm )
   }
   storedentityrealm.set( c.uuid, c )
 
@@ -154,11 +163,14 @@ module.exports.getbyentity = async ( uri ) => {
  *  @param {string} realm - the entity realm
  *  @return {Promise} which resolves to either a set containing calls for this entity or undefined
  */
-module.exports.getbyentityrealm = async ( realm ) => {
-  if( !storebyentityrealm.has( realm ) ) return false
-  return storebyentityrealm.get( realm )
+module.exports.getbyrealm = async ( realm ) => {
+  if( !storebyrealm.has( realm ) ) return false
+  return storebyrealm.get( realm )
 }
 
+/* backwards compat - is now realm more generally */
+module.exports.getbyentityrealm = module.exports.getbyrealm
+
 /**
  *  Returns a unique call by call id and sip tags.
  *  @param {object} sip - sip params required
@@ -201,22 +213,27 @@ module.exports.delete = async function( c ) {
 
   deletefromcallidset( c )
 
-  const entity = fixentity( c )
-  if( "boolean" == typeof entity ) return entity
+  let realm = ""
+  const destination = c.destination
+  if( destination ) realm = c.destination.host
 
-  const entityentries = storebyentity.get( entity.uri )
-  if( undefined !== entityentries ) {
-    entityentries.delete( c.uuid )
-    if( 0 === entityentries.size ) {
-      storebyentity.delete( entity.uri )
+  const entity = fixentity( c )
+  if( "boolean" !== typeof entity ) {
+    const entityentries = storebyentity.get( entity.uri )
+    if( undefined !== entityentries ) {
+      entityentries.delete( c.uuid )
+      if( 0 === entityentries.size ) {
+        storebyentity.delete( entity.uri )
+      }
     }
+    realm = entity.realm
   }
 
-  const storedentityrealm = storebyentityrealm.get( entity.realm )
-  if( storedentityrealm ) {
-    storedentityrealm.delete( c.uuid )
-    if( 0 === storedentityrealm.size ) {
-      storebyentityrealm.delete( entity.realm )
+  const storedrealm = storebyrealm.get( realm )
+  if( storedrealm ) {
+    storedrealm.delete( c.uuid )
+    if( 0 === storedrealm.size ) {
+      storebyrealm.delete( entity.realm )
     }
   }
 }
@@ -228,7 +245,7 @@ module.exports.clear = async() => {
   storebycallid = new Map()
   storebyuuid = new Map()
   storebyentity = new Map()
-  storebyentityrealm = new Map()
+  storebyrealm = new Map()
 }
 
 /**
@@ -239,7 +256,7 @@ module.exports.stats = async () => {
     "storebycallid": storebycallid.size,
     "storebyuuid": storebyuuid.size,
     "storebyentity": storebyentity.size,
-    "storebyentityrealm": storebyentityrealm.size
+    "storebyrealm": storebyrealm.size
   }
 }
 
@@ -247,5 +264,5 @@ module.exports.stats = async () => {
  * Debug only
  */
 module.exports._dump = () => {
-  console.log( storebycallid, storebyuuid, storebyentity, storebyentityrealm ) 
+  console.log( storebycallid, storebyuuid, storebyentity, storebyrealm ) 
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@babblevoice/babble-drachtio-callmanager",
-  "version": "2.2.7",
+  "version": "2.3.1",
   "description": "Call processing to create a PBX",
   "main": "index.js",
   "scripts": {

package/test/unit/store.js

@@ -30,7 +30,7 @@ describe( "callmanager - store", function() {
       "storebycallid": 1,
       "storebyuuid": 1,
       "storebyentity": 1,
-      "storebyentityrealm": 1
+      "storebyrealm": 1
     } )
   } )
 
@@ -53,7 +53,7 @@ describe( "callmanager - store", function() {
       "storebycallid": 1,
       "storebyuuid": 1,
       "storebyentity": 1,
-      "storebyentityrealm": 1
+      "storebyrealm": 1
     } )
 
     dummycall.sip.tags.local = "4444"
@@ -63,7 +63,7 @@ describe( "callmanager - store", function() {
       "storebycallid": 1,
       "storebyuuid": 1,
       "storebyentity": 1,
-      "storebyentityrealm": 1
+      "storebyrealm": 1
     } )
 
     let searchfor = {
@@ -119,7 +119,7 @@ describe( "callmanager - store", function() {
       "storebycallid": 2,
       "storebyuuid": 2,
       "storebyentity": 0,
-      "storebyentityrealm": 0
+      "storebyrealm": 0
     } )
 
     dummycall1._entity = {
@@ -132,7 +132,7 @@ describe( "callmanager - store", function() {
       "storebycallid": 2,
       "storebyuuid": 2,
       "storebyentity": 1,
-      "storebyentityrealm": 1
+      "storebyrealm": 1
     } )
 
     await callstore.set( dummycall3 )
@@ -141,7 +141,7 @@ describe( "callmanager - store", function() {
       "storebycallid": 3,
       "storebyuuid": 3,
       "storebyentity": 1,
-      "storebyentityrealm": 1
+      "storebyrealm": 1
     } )
 
     let c = await callstore.getbycallid( dummycall1.sip )
@@ -158,7 +158,7 @@ describe( "callmanager - store", function() {
       "storebycallid": 0,
       "storebyuuid": 0,
       "storebyentity": 0,
-      "storebyentityrealm": 0
+      "storebyrealm": 0
     } )
   } )
 } )