@banta/sdk 5.8.3 → 5.8.5

package/fesm2022/banta-sdk.mjs

@@ -7587,6 +7587,8 @@ i0.ɵɵngDeclareClassMetadata({ minVersion: "12.0.0", version: "17.3.9", ngImpor
                 args: ['class.visible']
             }] } });
 
+const DEFAULT_MAX_MESSAGES = 2000;
+const DEFAULT_MAX_VISIBLE_MESSAGES = 200;
 class CommentViewComponent {
     constructor(backend, elementRef) {
         this.backend = backend;
@@ -7606,7 +7608,7 @@ class CommentViewComponent {
          * While this is called "new" messages, it really represents the messages that would be visible *at the beginning
          * of the sort order*, which can be flipped by the newestLast feature (used for replies mode).
          *
-         * So, when newestLast is false, regardless of the current sortOrder, newMessages are conceptually
+         * So, when newestLast is false (top-level comments), regardless of the current sortOrder, newMessages are conceptually
          * *above* the visible set of messages.
          *
          * When newestLast is true (as in replies mode), regardless of the current sortOrder, newMessages are conceptually
@@ -7624,10 +7626,6 @@ class CommentViewComponent {
          * *above* the visible set of messages.
          */
         this.olderMessages = [];
-        //#endregion
-        //#region Inputs
-        this.maxMessages = 2000;
-        this.maxVisibleMessages = 200;
         this.newestLast = false;
         this.holdNewMessages = false;
         this.showEmptyState = true;
@@ -7667,6 +7665,10 @@ class CommentViewComponent {
     set source(value) { this.setSource(value); }
     get previousMessages() { return this.newestLast ? this.olderMessages : this.newMessages; }
     get nextMessages() { return this.newestLast ? this.newMessages : this.olderMessages; }
+    set maxMessages(value) { this._maxMessages = value; }
+    get maxMessages() { return this._maxMessages ?? DEFAULT_MAX_MESSAGES; }
+    set maxVisibleMessages(value) { this._maxVisibleMessages = value; }
+    get maxVisibleMessages() { return this._maxVisibleMessages ?? DEFAULT_MAX_VISIBLE_MESSAGES; }
     get comments() { return Array.from(this.commentsQuery); }
     //#endregion
     /**
@@ -7689,16 +7691,15 @@ class CommentViewComponent {
             console.error(`Error while loading message in context: Failed to find message ${message.id}, maybe it was deleted!`);
             return false;
         }
-        let pageSize = this.maxVisibleMessages;
         let items = [].concat(this.olderMessages, this.messages, this.newMessages);
         let index = items.findIndex(x => x.id === message.id);
         if (index < 0) {
             console.error(`Error while loading message in context: Message was not present in message list!`);
             return false;
         }
-        let startIndex = Math.max(0, index - pageSize / 2);
+        let startIndex = Math.max(0, index - this.maxVisibleMessages / 2);
         this.newMessages = items.splice(0, startIndex);
-        this.messages = items.splice(0, pageSize);
+        this.messages = items.splice(0, this.maxVisibleMessages);
         this.olderMessages = items;
         this.isViewingMore = true;
     }
@@ -7936,12 +7937,14 @@ class CommentViewComponent {
         return this.previousMessages.length > 0;
     }
     get pageSize() {
-        return Math.min(20, this.maxVisibleMessages);
+        return Math.min(20, this.maxVisibleMessages || 20);
     }
     async showPrevious() {
         this.isViewingMore = true;
         let nextPageSize = this.pageSize;
         this.isLoadingMore = false;
+        if (isNaN(nextPageSize))
+            throw new Error(`Not safe to load more with NaN page size`);
         if (this.previousMessages.length > 0) {
             const storedMessages = this.previousMessages.splice(Math.max(0, this.previousMessages.length - nextPageSize), nextPageSize);
             this.messages = [...storedMessages, ...this.messages];
@@ -8009,6 +8012,8 @@ class CommentViewComponent {
     async showNext() {
         this.isViewingMore = true;
         let nextPageSize = this.pageSize;
+        if (isNaN(nextPageSize))
+            throw new Error(`Not safe to load more with NaN page size`);
         this.isLoadingMore = false;
         if (this.nextMessages.length > 0) {
             const storedMessages = this.nextMessages.splice(0, nextPageSize);
@@ -8049,6 +8054,8 @@ class CommentViewComponent {
     async showMore() {
         this.isViewingMore = true;
         let nextPageSize = this.pageSize;
+        if (isNaN(nextPageSize))
+            throw new Error(`Not safe to load more with NaN page size`);
         this.isLoadingMore = false;
         if (this.olderMessages.length > 0) {
             const storedMessages = this.olderMessages.splice(0, nextPageSize);
@@ -8092,14 +8099,20 @@ class CommentViewComponent {
         // Extract the messages that do not fit in the maxVisibleMessages buffer.
         if (this.messages.length > this.maxVisibleMessages) {
             let overflow = [];
-            if (this.newestLast)
+            // Move overflowing messages into newMessages.
+            // Regardless of the order (newestLast), newMessages represents the direction that is being loaded, 
+            // since it's definition depends on that order. 
+            if (this.newestLast) {
                 overflow = this.messages.splice(this.maxVisibleMessages, this.messages.length);
-            else
+                this.newMessages = overflow.concat(this.newMessages);
+                this.newMessages.splice(this.maxMessages - this.maxVisibleMessages, this.newMessages.length);
+            }
+            else {
                 overflow = this.messages.splice(0, this.messages.length - this.maxVisibleMessages);
-            // Regardless of the order (newestLast), newMessages represents the direction that is being pushed, since it's definition
-            // depends on that order. Move overflowing messages into newMessages.
-            this.newMessages = overflow.concat(this.newMessages);
-            this.newMessages.splice(this.maxMessages - this.maxVisibleMessages, this.newMessages.length);
+                this.newMessages.push(...overflow);
+                if (this.newMessages.length > this.maxMessages - this.maxVisibleMessages)
+                    this.newMessages.splice(0, this.newMessages.length - (this.maxMessages - this.maxVisibleMessages));
+            }
         }
     }
     addMessage(message) {
@@ -9440,7 +9453,9 @@ class BantaCommentsComponent {
         }
         this._selected.next(message);
         console.log(`[Banta] Opening thread for ${this.topicID}/${message.id}...`);
-        let selectedMessageThread = await this.backend.getSourceForThread(this.topicID, message.id);
+        let selectedMessageThread = await this.backend.getSourceForThread(this.topicID, message.id, {
+            metadata: this.metadata
+        });
         if (!selectedMessageThread) {
             console.warn(`Failed to locate thread for message ${this.topicID}/${message.id}!`);
             return null;