quickblox 2.24.0 → 2.24.1-alpha.0

package/src/modules/chat/qbChat.js

@@ -44,6 +44,7 @@ NOTICE_KNOWN_MODULE_IDS[noticeConsts.NOTICE_MODULE_IDENTIFIER.UPDATED_MESSAGE] =
 NOTICE_KNOWN_MODULE_IDS[noticeConsts.NOTICE_MODULE_IDENTIFIER.DELETED_MESSAGE] = true;
 NOTICE_KNOWN_MODULE_IDS[noticeConsts.NOTICE_MODULE_IDENTIFIER.UPDATED_DIALOG] = true;
 NOTICE_KNOWN_MODULE_IDS[noticeConsts.NOTICE_MODULE_IDENTIFIER.DELETED_DIALOG] = true;
+NOTICE_KNOWN_MODULE_IDS[noticeConsts.NOTICE_MODULE_IDENTIFIER.CREATED_DIALOG] = true;
 
 /**
  * Cross-env: get all direct child elements of `parent` whose tag name equals
@@ -327,6 +328,91 @@ function _parseCustomData(extraParams) {
     return out;
 }
 
+/**
+ * Build a partial QBChatDialog object from a dialog-notice <extraParams>.
+ * Shared by NoticeUpdatedDialog and NoticeCreatedDialog (SR-2952) — the server
+ * sends a full dialog snapshot in both cases. Fields with null/empty values are
+ * omitted by the server, so every read is guarded.
+ *
+ * CreatedDialog additionally carries created_at/updated_at/user_id (creator) and
+ * has no last_message* fields (dialog just created); UpdatedDialog carries
+ * last_message* and (per server contract) may also carry created_at/updated_at/
+ * user_id. Reading them here for both is safe — absent fields are simply skipped.
+ *
+ * @param {Element} extraParams
+ * @param {String} dialogId - already-resolved dialog_id text.
+ * @return {Object} partial dialog snapshot.
+ */
+function _parseDialogSnapshot(extraParams, dialogId) {
+    var dlg = {};
+    dlg._id = dialogId;
+
+    var name = chatUtils.getElementText(extraParams, 'name');
+    if (name !== undefined && name !== null && name !== '') {
+        dlg.name = name;
+    }
+    var photo = chatUtils.getElementText(extraParams, 'photo');
+    if (photo !== undefined && photo !== null && photo !== '') {
+        dlg.photo = photo;
+    }
+    var typeText = chatUtils.getElementText(extraParams, 'type');
+    var typeNum = parseInt(typeText, 10);
+    if (!isNaN(typeNum)) {
+        dlg.type = typeNum;
+    }
+    var occupants = _parseNumericIdsCsv(chatUtils.getElementText(extraParams, 'occupants_ids'));
+    if (occupants.length > 0) {
+        dlg.occupants_ids = occupants;
+    }
+    var admins = _parseNumericIdsCsv(chatUtils.getElementText(extraParams, 'admin_ids'));
+    if (admins.length > 0) {
+        dlg.admin_ids = admins;
+    }
+    var isJoinReqText = chatUtils.getElementText(extraParams, 'is_join_required');
+    if (isJoinReqText !== undefined && isJoinReqText !== null && isJoinReqText !== '') {
+        var isJoinReq = parseInt(isJoinReqText, 10);
+        dlg.is_join_required = isNaN(isJoinReq) ? isJoinReqText : isJoinReq;
+    }
+    var roomJid = chatUtils.getElementText(extraParams, 'xmpp_room_jid');
+    if (roomJid !== undefined && roomJid !== null && roomJid !== '') {
+        // Server may pretty-print xmpp_room_jid with surrounding whitespace.
+        dlg.xmpp_room_jid = roomJid.trim();
+    }
+    var customData = _parseCustomData(extraParams);
+    if (customData !== undefined) {
+        dlg.custom_data = customData;
+    }
+    // Creation / modification metadata (present on CreatedDialog; also valid on
+    // UpdatedDialog per server contract). Kept as ISO-8601 strings verbatim.
+    var createdAt = chatUtils.getElementText(extraParams, 'created_at');
+    if (createdAt !== undefined && createdAt !== null && createdAt !== '') {
+        dlg.created_at = createdAt;
+    }
+    var updatedAt = chatUtils.getElementText(extraParams, 'updated_at');
+    if (updatedAt !== undefined && updatedAt !== null && updatedAt !== '') {
+        dlg.updated_at = updatedAt;
+    }
+    var userIdText = chatUtils.getElementText(extraParams, 'user_id');
+    var userId = parseInt(userIdText, 10);
+    if (!isNaN(userId)) {
+        dlg.user_id = userId;
+    }
+    // Last message fields (present on UpdatedDialog; absent on a freshly
+    // CreatedDialog — the guards below simply skip them then).
+    var lmText = chatUtils.getElementText(extraParams, 'last_message');
+    if (lmText) { dlg.last_message = lmText; }
+    var lmId = chatUtils.getElementText(extraParams, 'last_message_id');
+    if (lmId) { dlg.last_message_id = lmId; }
+    var lmDsText = chatUtils.getElementText(extraParams, 'last_message_date_sent');
+    var lmDs = parseInt(lmDsText, 10);
+    if (!isNaN(lmDs)) { dlg.last_message_date_sent = lmDs; }
+    var lmUidText = chatUtils.getElementText(extraParams, 'last_message_user_id');
+    var lmUid = parseInt(lmUidText, 10);
+    if (!isNaN(lmUid)) { dlg.last_message_user_id = lmUid; }
+
+    return dlg;
+}
+
 /**
  * Parse a Notice headline stanza into {type, payload} ready for routing.
  * Returns null if stanza is not a Notice (no urn:xmpp:notice:0 namespace, or
@@ -422,59 +508,12 @@ function _parseNoticeStanza(stanza) {
                 message: msg
             };
         }
-    } else if (type === IDS.UPDATED_DIALOG) {
-        // Build a partial QBChatDialog object from extraParams (full server snapshot).
-        var dlg = {};
-        dlg._id = dialogId;
-        var name = chatUtils.getElementText(extraParams, 'name');
-        if (name !== undefined && name !== null && name !== '') {
-            dlg.name = name;
-        }
-        var photo = chatUtils.getElementText(extraParams, 'photo');
-        if (photo !== undefined && photo !== null && photo !== '') {
-            dlg.photo = photo;
-        }
-        var typeText = chatUtils.getElementText(extraParams, 'type');
-        var typeNum = parseInt(typeText, 10);
-        if (!isNaN(typeNum)) {
-            dlg.type = typeNum;
-        }
-        var occupants = _parseNumericIdsCsv(chatUtils.getElementText(extraParams, 'occupants_ids'));
-        if (occupants.length > 0) {
-            dlg.occupants_ids = occupants;
-        }
-        var admins = _parseNumericIdsCsv(chatUtils.getElementText(extraParams, 'admin_ids'));
-        if (admins.length > 0) {
-            dlg.admin_ids = admins;
-        }
-        var isJoinReqText = chatUtils.getElementText(extraParams, 'is_join_required');
-        if (isJoinReqText !== undefined && isJoinReqText !== null && isJoinReqText !== '') {
-            var isJoinReq = parseInt(isJoinReqText, 10);
-            dlg.is_join_required = isNaN(isJoinReq) ? isJoinReqText : isJoinReq;
-        }
-        var roomJid = chatUtils.getElementText(extraParams, 'xmpp_room_jid');
-        if (roomJid !== undefined && roomJid !== null && roomJid !== '') {
-            dlg.xmpp_room_jid = roomJid;
-        }
-        var customData = _parseCustomData(extraParams);
-        if (customData !== undefined) {
-            dlg.custom_data = customData;
-        }
-        // Last message fields.
-        var lmText = chatUtils.getElementText(extraParams, 'last_message');
-        if (lmText) { dlg.last_message = lmText; }
-        var lmId = chatUtils.getElementText(extraParams, 'last_message_id');
-        if (lmId) { dlg.last_message_id = lmId; }
-        var lmDsText = chatUtils.getElementText(extraParams, 'last_message_date_sent');
-        var lmDs = parseInt(lmDsText, 10);
-        if (!isNaN(lmDs)) { dlg.last_message_date_sent = lmDs; }
-        var lmUidText = chatUtils.getElementText(extraParams, 'last_message_user_id');
-        var lmUid = parseInt(lmUidText, 10);
-        if (!isNaN(lmUid)) { dlg.last_message_user_id = lmUid; }
-
+    } else if (type === IDS.UPDATED_DIALOG || type === IDS.CREATED_DIALOG) {
+        // Full server dialog snapshot. UpdatedDialog and CreatedDialog (SR-2952)
+        // share the same shape; they differ only by moduleIdentifier (routing).
         payload = {
             kind: 'dialog',
-            dialog: dlg
+            dialog: _parseDialogSnapshot(extraParams, dialogId)
         };
     } else {
         return null;
@@ -521,6 +560,10 @@ function _routeNoticeEvent(chatProxy, parsed) {
         if (typeof chatProxy.onDialogUpdatedListener === 'function') {
             Utils.safeCallbackCall(chatProxy.onDialogUpdatedListener, p.dialog);
         }
+    } else if (type === IDS.CREATED_DIALOG) {
+        if (typeof chatProxy.onDialogCreatedListener === 'function') {
+            Utils.safeCallbackCall(chatProxy.onDialogCreatedListener, p.dialog);
+        }
     }
 }
 
@@ -621,6 +664,15 @@ function ChatProxy(service) {
     this.onMessageReactionChangedListener = null;
     this.onDialogDeletedListener = null;
     this.onDialogUpdatedListener = null;
+    /**
+     * NoticeCreatedDialog listener (SR-2952). Fired when a Group or Private
+     * dialog is created and the current user has the Notice feature enabled.
+     * Signature: onDialogCreatedListener(dialog). The dialog is a partial
+     * snapshot from the creation notice — it carries created_at/updated_at/
+     * user_id (creator) but no last_message* fields (the dialog is brand new),
+     * and for a private dialog no xmpp_room_jid. Not fired for Public dialogs.
+     */
+    this.onDialogCreatedListener = null;
 
     // [QC-1550] XMPP connection is considered "verified" only after the first
     // successful pong response. Strophe emits Status.CONNECTED at the transport
@@ -2465,7 +2517,8 @@ ChatProxy.prototype = {
      * Subscribe the current XMPP session to Notice Feature stanzas
      * (urn:xmpp:notice:0). After a successful IQ result the server starts
      * delivering NoticeUpdatedMessage / NoticeDeletedMessage /
-     * NoticeUpdatedDialog / NoticeDeletedDialog headline stanzas.
+     * NoticeCreatedDialog / NoticeUpdatedDialog / NoticeDeletedDialog
+     * headline stanzas.
      *
      * The local enabled flag is reset to false on chat disconnect — consumers
      * must call enableNotices() again after a reconnect (intentional divergence

package/src/modules/chat/qbNoticeConsts.js

@@ -23,7 +23,8 @@ var NOTICE_MODULE_IDENTIFIER = {
     UPDATED_MESSAGE: 'NoticeUpdatedMessage',
     DELETED_MESSAGE: 'NoticeDeletedMessage',
     UPDATED_DIALOG:  'NoticeUpdatedDialog',
-    DELETED_DIALOG:  'NoticeDeletedDialog'
+    DELETED_DIALOG:  'NoticeDeletedDialog',
+    CREATED_DIALOG:  'NoticeCreatedDialog'
 };
 
 module.exports = {

package/src/qbConfig.js

@@ -12,8 +12,8 @@
  */
 
 var config = {
-  version: '2.24.0',
-  buildNumber: '1180',
+  version: '2.24.1-alpha.0',
+  buildNumber: '1181',
   creds: {
     'appId': 0,
     'authKey': '',