Merge pull request #454 from Ketteiteki/update_delete_message_flow

Update delete message flow

Author: kolosovpetro

MangoAPI.BusinessLogic/ApiCommands/Messages/DeleteMessageCommandHandler.cs

@@ -37,11 +37,7 @@ public async Task<Result<DeleteMessageResponse>> Handle(
     {
         var checkMessage = await dbContext.Messages
             .Include(x => x.User)
-            .Select(x=> new
-            {
-                MessageId = x.Id,
-                UserId = x.User.Id
-            })
+            .Select(x => new { MessageId = x.Id, UserId = x.User.Id })
             .FirstOrDefaultAsync(t => t.MessageId == request.MessageId, cancellationToken);
 
         if (checkMessage == null)
@@ -85,7 +81,7 @@ public async Task<Result<DeleteMessageResponse>> Handle(
 
         var deleteNotification = isMessageLast
             ? UpdateChatLastMessageAndReturnNotification(chat, deletedMessage)
-            : CreateNotLastMessageNotification(chat.Id, deletedMessage.Id);
+            : CreateNotLastMessageNotification(chat.Id, deletedMessage.Id, deletedMessage.UserId);
 
         dbContext.Messages.Remove(deletedMessage);
         dbContext.Chats.Update(chat);
@@ -97,7 +93,7 @@ public async Task<Result<DeleteMessageResponse>> Handle(
         return responseFactory.SuccessResponse(DeleteMessageResponse.FromSuccess(deletedMessage));
     }
 
-    private static MessageDeleteNotification UpdateChatLastMessageAndReturnNotification(
+    private static DeleteMessageNotification UpdateChatLastMessageAndReturnNotification(
         ChatEntity chat,
         MessageEntity deletedMessage)
     {
@@ -110,7 +106,8 @@ private static MessageDeleteNotification UpdateChatLastMessageAndReturnNotificat
             newLastMessage?.CreatedAt,
             newLastMessage?.Id);
 
-        var deleteNotification = new MessageDeleteNotification(
+        var deleteNotification = new DeleteMessageNotification(
+            deletedMessage.UserId,
             chat.Id,
             deletedMessage.Id,
             newLastMessage?.Text,
@@ -122,9 +119,13 @@ private static MessageDeleteNotification UpdateChatLastMessageAndReturnNotificat
         return deleteNotification;
     }
 
-    private static MessageDeleteNotification CreateNotLastMessageNotification(Guid chatId, Guid deletedMessageId)
+    private static DeleteMessageNotification CreateNotLastMessageNotification(
+        Guid chatId,
+        Guid deletedMessageId,
+        Guid userId)
     {
-        var deleteNotification = new MessageDeleteNotification(
+        var deleteNotification = new DeleteMessageNotification(
+            userId,
             chatId,
             deletedMessageId,
             NewLastMessageText: string.Empty,

MangoAPI.BusinessLogic/HubConfig/IHubClient.cs

@@ -23,5 +23,5 @@ public interface IHubClient
     /// <summary>
     /// Notifies chat subscribers on the message delete via SignalR.
     /// </summary>
-    Task MessageDeletedAsync(MessageDeleteNotification notification);
+    Task MessageDeletedAsync(DeleteMessageNotification notification);
 }

…tifications/MessageDeleteNotification.cs …tifications/DeleteMessageNotification.csMangoAPI.BusinessLogic/Notifications/MessageDeleteNotification.cs renamed to MangoAPI.BusinessLogic/Notifications/DeleteMessageNotification.cs MangoAPI.BusinessLogic/Notifications/MessageDeleteNotification.cs renamed to MangoAPI.BusinessLogic/Notifications/DeleteMessageNotification.cs

@@ -2,7 +2,8 @@
 
 namespace MangoAPI.BusinessLogic.Notifications;
 
-public record MessageDeleteNotification(
+public record DeleteMessageNotification(
+    Guid UserId,
     Guid ChatId,
     Guid DeletedMessageId,
     string NewLastMessageText,

MangoAPI.Client/src/app/components/chats/chats.component.html

@@ -270,7 +270,7 @@ <h5>Chats</h5>
                           data-target="#createGroup"
                           data-toggle="modal"
                           role="button"
-                          (click)="deleteMessage(message)"
+                          (click)="onDeleteMessageClick(message)"
                         >
                           <img
                             alt="Delete Message Icon"

MangoAPI.Client/src/app/components/chats/chats.component.ts

@@ -348,14 +348,46 @@ export class ChatsComponent implements OnInit {
     await this.onSendMessageClick().then((r) => r);
   }
 
-  async deleteMessage(message: Message) {
+  async onDeleteMessageClick(message: Message) {
     const deleteMessageCommand = new DeleteMessageCommand(message.messageId, message.chatId);
 
     const messageIndex = this.messages.findIndex((x) => x.messageId === message.messageId);
 
     if (messageIndex === -1) return;
 
-    this.messages.splice(messageIndex, 1);
+    const lastMessage = this.messages[this.messages.length - 1];
+
+    const deletedMessage = this.messages.splice(messageIndex, 1)[0];
+
+    const newLastMessage = this.messages[this.messages.length - 1];
+
+    const lastMessageDeleted = lastMessage.messageId === deletedMessage.messageId;
+
+    const chatIndex = this.chats.findIndex((x) => x.chatId === message.chatId);
+
+    const chat = this.chats[chatIndex];
+
+    if (lastMessageDeleted && newLastMessage) {
+      chat.lastMessageId = newLastMessage.messageId;
+      chat.lastMessageText = newLastMessage.text;
+      chat.lastMessageAuthor = newLastMessage.displayName;
+      chat.lastMessageTime = newLastMessage.createdAt;
+
+      const deleteMessageSub$ = this._messagesService.deleteMessage(deleteMessageCommand);
+      await firstValueFrom<DeleteMessageResponse>(deleteMessageSub$);
+      return;
+    }
+
+    if (lastMessageDeleted && !newLastMessage) {
+      chat.lastMessageId = '';
+      chat.lastMessageText = '';
+      chat.lastMessageAuthor = '';
+      chat.lastMessageTime = '';
+
+      const deleteMessageSub$ = this._messagesService.deleteMessage(deleteMessageCommand);
+      await firstValueFrom<DeleteMessageResponse>(deleteMessageSub$);
+      return;
+    }
 
     const deleteMessageSub$ = this._messagesService.deleteMessage(deleteMessageCommand);
     await firstValueFrom<DeleteMessageResponse>(deleteMessageSub$);
@@ -490,25 +522,26 @@ export class ChatsComponent implements OnInit {
   }
 
   private onMessageDeleteHandler(notification: DeleteMessageNotification) {
-    const chatIndex = this.chats.findIndex((x) => x.chatId === notification.chatId);
+    const selfMessage = notification.userId === this.userId;
 
-    if (notification.isLastMessage && chatIndex !== -1) {
-      const updatedChat = this.chats[chatIndex];
+    if (selfMessage) return;
 
-      updatedChat.lastMessageAuthor = notification.newLastMessageDisplayName;
-      updatedChat.lastMessageText = notification.newLastMessageText;
-      updatedChat.lastMessageTime = notification.newLastMessageTime;
-      updatedChat.lastMessageId = notification.newLastMessageId;
+    if (this.activeChatId === notification.chatId) {
+      const messageIndex = this.messages.findIndex(
+        (x) => x.messageId === notification.deletedMessageId
+      );
+      this.messages.splice(messageIndex, 1);
     }
 
-    if (this.activeChatId !== notification.chatId) return;
-
-    const messageIndex = this.messages.findIndex(
-      (x) => x.messageId === notification.deletedMessageId
-    );
+    const chatIndex = this.chats.findIndex((x) => x.chatId === notification.chatId);
+    const chat = this.chats[chatIndex];
 
-    if (messageIndex !== -1) {
-      this.messages.splice(messageIndex, 1);
+    if (notification.isLastMessage) {
+      chat.lastMessageId = notification.newLastMessageId;
+      chat.lastMessageText = notification.newLastMessageText;
+      chat.lastMessageAuthor = notification.newLastMessageDisplayName;
+      chat.lastMessageTime = notification.newLastMessageTime;
+      return;
     }
   }
 

MangoAPI.Client/src/app/types/notifications/DeleteMessageNotification.ts

@@ -1,4 +1,5 @@
 export interface DeleteMessageNotification {
+  userId: string;
   chatId: string;
   deletedMessageId: string;
   newLastMessageText: string;