Merge pull request #261 from GetStream/fix-message-with-spaces

Fix message with spaces

Author: szuperaz

package-lock.json

@@ -111,7 +111,7 @@
     "@ctrl/ngx-emoji-mart": "^6.2.0",
     "@ngx-translate/core": "^13.0.0",
     "@ngx-translate/http-loader": "^6.0.0",
-    "@stream-io/stream-chat-css": "2.7.0",
+    "@stream-io/stream-chat-css": "2.8.1",
     "@stream-io/transliterate": "^1.5.2",
     "angular-mentions": "^1.4.0",
     "dayjs": "^1.10.7",

package.json

@@ -441,6 +441,36 @@ describe('MessageInputComponent', () => {
     expect(sendMessageSpy).not.toHaveBeenCalled();
   });
 
+  it(`shouldn't send messages with only spaces`, () => {
+    const textarea = queryTextarea();
+    component.textareaValue = ' '; // single space
+    textarea?.send.next();
+    fixture.detectChanges();
+
+    expect(sendMessageSpy).not.toHaveBeenCalled();
+  });
+
+  it(`should remove text if attachments are uploaded, but text contains only space characters`, () => {
+    attachmentService.mapToAttachments.and.returnValue([
+      {
+        type: 'image',
+        img_url: 'url',
+      },
+    ]);
+    const textarea = queryTextarea();
+    component.textareaValue = '    '; // multiple spaces
+    textarea?.send.next();
+    fixture.detectChanges();
+
+    expect(sendMessageSpy).toHaveBeenCalledWith(
+      '',
+      jasmine.any(Array),
+      jasmine.any(Object),
+      undefined,
+      undefined
+    );
+  });
+
   it('should apply CSS class if attachments are present', () => {
     const cssClass = 'str-chat__input-flat-has-attachments';
 

projects/stream-chat-angular/src/lib/message-input/message-input.component.spec.ts

@@ -290,10 +290,17 @@ export class MessageInputComponent
       return;
     }
     const attachments = this.attachmentService.mapToAttachments();
-    const text = this.textareaValue;
-    if (!text && (!attachments || attachments.length === 0)) {
+    let text = this.textareaValue;
+    const textContainsOnlySpaceChars = !text.replace(/ /g, ''); //spcae
+    if (
+      (!text || textContainsOnlySpaceChars) &&
+      (!attachments || attachments.length === 0)
+    ) {
       return;
     }
+    if (textContainsOnlySpaceChars) {
+      text = '';
+    }
     if (this.containsLinks && !this.canSendLinks) {
       this.notificationService.addTemporaryNotification(
         'streamChat.Sending links is not allowed in this conversation'