From 4ce589c2b793cc5217bcc00c05d46061ebf0b8b7 Mon Sep 17 00:00:00 2001
From: dab246 <tdvu@linagora.com>
Date: Wed, 10 Dec 2025 18:41:46 +0700
Subject: [PATCH] TF-4193 Refactor ThreadDetail: extract
 getLabelsForFirstEmail() to clarify label resolution logic

---
 .../get_thread_details_email_views.dart       | 21 ++++++++++++++++---
 1 file changed, 18 insertions(+), 3 deletions(-)

diff --git a/lib/features/thread_detail/presentation/extension/get_thread_details_email_views.dart b/lib/features/thread_detail/presentation/extension/get_thread_details_email_views.dart
index 894796cde..74be4446e 100644
--- a/lib/features/thread_detail/presentation/extension/get_thread_details_email_views.dart
+++ b/lib/features/thread_detail/presentation/extension/get_thread_details_email_views.dart
@@ -46,9 +46,10 @@ extension GetThreadDetailEmailViews on ThreadDetailController {
       final isFirstEmailInThreadDetail = indexOfEmailId == 0;
 
       if (presentationEmail.emailInThreadStatus == EmailInThreadStatus.collapsed) {
-        final emailLabels = labels?.isNotEmpty == true && isFirstEmailInThreadDetail
-            ? emailIdsPresentation.values.last?.getLabelList(labels!)
-            : null;
+        final emailLabels = _getLabelsForFirstEmail(
+          isFirstEmailInThreadDetail: isFirstEmailInThreadDetail,
+          availableLabels: labels,
+        );
 
         return ThreadDetailCollapsedEmail(
           presentationEmail: presentationEmail.copyWith(
@@ -109,4 +110,18 @@ extension GetThreadDetailEmailViews on ThreadDetailController {
       );
     }).toList();
   }
+
+  List<Label>? _getLabelsForFirstEmail({
+    required bool isFirstEmailInThreadDetail,
+    required List<Label>? availableLabels,
+  }) {
+    if (!isFirstEmailInThreadDetail ||
+        availableLabels == null ||
+        availableLabels.isEmpty) {
+      return null;
+    }
+
+    final lastEmail = emailIdsPresentation.values.last;
+    return lastEmail?.getLabelList(availableLabels);
+  }
 }
\ No newline at end of file