From f7f0a65f56d9fcf9bd9a6f78fa2a626c7c2cbd94 Mon Sep 17 00:00:00 2001
From: dahoud <41957584+DahoudG@users.noreply.github.com>
Date: Sat, 11 Apr 2026 03:01:28 +0000
Subject: [PATCH] fix: correct builder method names for boolean fields in
 response DTOs
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

ConversationResponse/MessageResponse fields (muted, pinned, archived,
edited, deleted) are primitive booleans — Lombok @Builder generates
.muted() not .isMuted(). Also use Boolean.TRUE.equals() for null-safe
unboxing from entity Boolean wrapper fields.
---
 .../unionflow/server/service/ConversationService.java  | 10 +++++-----
 .../lions/unionflow/server/service/MessageService.java |  4 ++--
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/main/java/dev/lions/unionflow/server/service/ConversationService.java b/src/main/java/dev/lions/unionflow/server/service/ConversationService.java
index 9194314..ed45840 100644
--- a/src/main/java/dev/lions/unionflow/server/service/ConversationService.java
+++ b/src/main/java/dev/lions/unionflow/server/service/ConversationService.java
@@ -177,9 +177,9 @@ public class ConversationService {
             .organisationId(c.getOrganisation() != null ? c.getOrganisation().getId() : null)
             .lastMessage(lastMsg != null ? convertMessageToResponse(lastMsg) : null)
             .unreadCount((int) unreadCount)
-            .isMuted(c.getIsMuted())
-            .isPinned(c.getIsPinned())
-            .isArchived(c.getIsArchived())
+            .muted(Boolean.TRUE.equals(c.getIsMuted()))
+            .pinned(Boolean.TRUE.equals(c.getIsPinned()))
+            .archived(Boolean.TRUE.equals(c.getIsArchived()))
             .createdAt(c.getDateCreation())
             .updatedAt(c.getUpdatedAt())
             .avatarUrl(c.getAvatarUrl())
@@ -201,8 +201,8 @@ public class ConversationService {
             .status(m.getStatus())
             .priority(m.getPriority())
             .createdAt(m.getDateCreation())
-            .isEdited(m.getIsEdited())
-            .isDeleted(m.getIsDeleted())
+            .edited(Boolean.TRUE.equals(m.getIsEdited()))
+            .deleted(Boolean.TRUE.equals(m.getIsDeleted()))
             .build();
     }
 }
diff --git a/src/main/java/dev/lions/unionflow/server/service/MessageService.java b/src/main/java/dev/lions/unionflow/server/service/MessageService.java
index 1ceae03..c8f6546 100644
--- a/src/main/java/dev/lions/unionflow/server/service/MessageService.java
+++ b/src/main/java/dev/lions/unionflow/server/service/MessageService.java
@@ -195,9 +195,9 @@ public class MessageService {
             .createdAt(m.getDateCreation())
             .readAt(m.getReadAt())
             .attachments(attachments)
-            .isEdited(m.getIsEdited())
+            .edited(Boolean.TRUE.equals(m.getIsEdited()))
             .editedAt(m.getEditedAt())
-            .isDeleted(m.getIsDeleted())
+            .deleted(Boolean.TRUE.equals(m.getIsDeleted()))
             .build();
     }
 }