TF-3006 Mail address handle in quick search bar

lib/features/mailbox_dashboard/presentation/action/dashboard_action.dart

@@ -56,13 +56,15 @@ class OpenEmailDetailedFromSuggestionQuickSearchAction extends DashBoardAction {
   List<Object?> get props => [presentationEmail];
 }
 
-class StartSearchEmailAction extends DashBoardAction {
-  final QuickSearchFilter? filter;
+class StartSearchEmailAction extends DashBoardAction {}
 
-  StartSearchEmailAction({this.filter});
+class StartSearchEmailBySearchFilterAction extends DashBoardAction {
+  final QuickSearchFilter searchFilter;
+
+  StartSearchEmailBySearchFilterAction(this.searchFilter);
 
   @override
-  List<Object?> get props => [filter];
+  List<Object?> get props => [searchFilter];
 }
 
 class EmptyTrashAction extends DashBoardAction {}
@@ -155,4 +157,6 @@ class CancelSelectionSearchEmailAction extends DashBoardAction {}
 
 class OpenAdvancedSearchViewAction extends DashBoardAction {}
 
-class ClearSearchFilterAppliedAction extends DashBoardAction {}
+class ClearSearchFilterAppliedAction extends DashBoardAction {}
+
+class ClearAdvancedSearchFilterEmailAction extends DashBoardAction {}

lib/features/mailbox_dashboard/presentation/controller/advanced_filter_controller.dart

@@ -79,14 +79,9 @@ class AdvancedFilterController extends BaseController {
   }
 
   void clearSearchFilter() {
-    searchController.clearSearchFilter();
     _resetAllToOriginalValue();
     _clearAllTextFieldInput();
-    searchController.searchInputController.clear();
-    searchController.clearSortOrder();
-    searchController.deactivateAdvancedSearch();
-    searchController.isAdvancedSearchViewOpen.value = false;
-    _mailboxDashBoardController.searchEmail();
+    _mailboxDashBoardController.handleClearAdvancedSearchFilterEmail();
   }
 
   void _updateFilterEmailFromAdvancedSearchView() {
@@ -119,8 +114,6 @@ class AdvancedFilterController extends BaseController {
       searchController.updateFilterEmail(toOption: const None());
     }
 
-    searchController.updateFilterEmail(sortOrderOption: searchController.sortOrderFiltered.value.getSortOrder());
-
     searchController.updateFilterEmail(
       mailboxOption: optionOf(_destinationMailboxSelected),
       subjectOption: optionOf(subjectFilterInputController.text),
@@ -170,7 +163,7 @@ class AdvancedFilterController extends BaseController {
       searchController.updateFilterEmail(beforeOption: const None());
     }
     searchController.isAdvancedSearchViewOpen.value = false;
-    _mailboxDashBoardController.searchEmail();
+    _mailboxDashBoardController.handleAdvancedSearchEmail();
   }
 
   Future<List<EmailAddress>> getAutoCompleteSuggestion(String word) async {
@@ -405,17 +398,8 @@ class AdvancedFilterController extends BaseController {
           );
         } else if (action is ClearDateRangeToAdvancedSearch) {
           _updateDateRangeTime(action.receiveTime);
-        } else if (action is StartSearchEmailAction) {
-          switch(action.filter) {
-            case QuickSearchFilter.from:
-              _updateFromField();
-              break;
-            case QuickSearchFilter.to:
-              _updateToField();
-              break;
-            default:
-              break;
-          }
+        } else if (action is StartSearchEmailBySearchFilterAction) {
+          _handleSearchEmailBySearchFilterAction(action.searchFilter);
         } else if (action is SearchEmailByFromFieldsAction) {
           searchController.clearSearchFilter();
           _resetAllToOriginalValue();
@@ -480,6 +464,19 @@ class AdvancedFilterController extends BaseController {
     applyAdvancedSearchFilter();
   }
 
+  void _handleSearchEmailBySearchFilterAction(QuickSearchFilter searchFilter) {
+    switch(searchFilter) {
+      case QuickSearchFilter.from:
+        _updateFromField();
+        break;
+      case QuickSearchFilter.to:
+        _updateToField();
+        break;
+      default:
+        break;
+    }
+  }
+
   @override
   void onClose() {
     _removeFocusListener();

lib/features/mailbox_dashboard/presentation/controller/mailbox_dashboard_controller.dart

@@ -665,20 +665,55 @@ class MailboxDashBoardController extends ReloadableController with UserSettingPo
 
   bool isSelectionEnabled() => currentSelectMode.value == SelectMode.ACTIVE;
 
-  void searchEmail({String? queryString}) {
-    log('MailboxDashBoardController::searchEmail():');
+  void handleAdvancedSearchEmail() {
+    log('MailboxDashBoardController::handleAdvancedSearchEmail:');
     clearFilterMessageOption();
-    searchController.clearFilterSuggestion();
-    if (queryString?.isNotEmpty == true) {
-      searchController.updateFilterEmail(textOption: Some(SearchQuery(queryString!)));
+    if (_searchInsideEmailDetailedViewIsActive()) {
+      _closeEmailDetailedView();
     }
-    searchController.updateFilterEmail(sortOrderOption: searchController.sortOrderFiltered.value.getSortOrder());
+    _unSelectedMailbox();
+    searchController.clearFilterSuggestion();
+    FocusManager.instance.primaryFocus?.unfocus();
     dispatchAction(StartSearchEmailAction());
+  }
+
+  void handleClearAdvancedSearchFilterEmail() {
+    log('MailboxDashBoardController::handleClearAdvancedSearchFilterEmail:');
+    clearFilterMessageOption();
+    if (_searchInsideEmailDetailedViewIsActive()) {
+      _closeEmailDetailedView();
+    }
+    _unSelectedMailbox();
+    searchController.clearAllFilterSearch();
     FocusManager.instance.primaryFocus?.unfocus();
+    dispatchAction(ClearAdvancedSearchFilterEmailAction());
+  }
+
+  void searchEmailByQueryString(String queryString) {
+    final isMailAddress = EmailUtils.isEmailAddressValid(queryString);
+    log('MailboxDashBoardController::searchEmailByQueryString():QueryString = $queryString | isMailAddress = $isMailAddress');
+    clearFilterMessageOption();
     if (_searchInsideEmailDetailedViewIsActive()) {
       _closeEmailDetailedView();
     }
     _unSelectedMailbox();
+    searchController.clearFilterSuggestion();
+
+    searchController.updateFilterEmail(
+      textOption: !isMailAddress
+        ? Some(SearchQuery(queryString))
+        : null,
+      fromOption: isMailAddress
+        ? Some({queryString})
+        : null);
+
+    FocusManager.instance.primaryFocus?.unfocus();
+
+    if (isMailAddress) {
+      dispatchAction(StartSearchEmailBySearchFilterAction(QuickSearchFilter.from));
+    } else {
+      dispatchAction(StartSearchEmailAction());
+    }
   }
 
   bool _searchInsideEmailDetailedViewIsActive() {
@@ -1673,7 +1708,7 @@ class MailboxDashBoardController extends ReloadableController with UserSettingPo
 
     if (newContact is EmailAddress) {
       searchController.updateFilterEmail(fromOption: Some({newContact.emailAddress}));
-      dispatchAction(StartSearchEmailAction(filter: QuickSearchFilter.from));
+      dispatchAction(StartSearchEmailBySearchFilterAction(QuickSearchFilter.from));
     }
   }
 
@@ -1691,7 +1726,7 @@ class MailboxDashBoardController extends ReloadableController with UserSettingPo
 
     if (newContact is EmailAddress) {
       searchController.updateFilterEmail(toOption: Some({newContact.emailAddress}));
-      dispatchAction(StartSearchEmailAction(filter: QuickSearchFilter.to));
+      dispatchAction(StartSearchEmailBySearchFilterAction(QuickSearchFilter.to));
     }
   }
 
@@ -1718,15 +1753,7 @@ class MailboxDashBoardController extends ReloadableController with UserSettingPo
 
     if (destinationMailbox != PresentationMailbox.unifiedMailbox &&
         mailboxIdSelected != destinationMailbox.id) {
-
-      searchController.updateFilterEmail(
-        mailboxOption: Some(destinationMailbox),
-        beforeOption: const None(),
-        positionOption: searchController.sortOrderFiltered.value.isScrollByPosition()
-          ? const Some(0)
-          : const None()
-      );
-
+      searchController.updateFilterEmail(mailboxOption: Some(destinationMailbox));
       dispatchAction(StartSearchEmailAction());
     }
   }
@@ -1745,8 +1772,7 @@ class MailboxDashBoardController extends ReloadableController with UserSettingPo
           searchController.updateFilterEmail(
             emailReceiveTimeTypeOption: Some(receiveTime),
             startDateOption: optionOf(startDate?.toUTCDate()),
-            endDateOption: optionOf(endDate?.toUTCDate()),
-            beforeOption: const None()
+            endDateOption: optionOf(endDate?.toUTCDate())
           );
           dispatchAction(StartSearchEmailAction());
         }
@@ -1756,8 +1782,7 @@ class MailboxDashBoardController extends ReloadableController with UserSettingPo
       searchController.updateFilterEmail(
         emailReceiveTimeTypeOption: Some(receiveTime),
         startDateOption: const None(),
-        endDateOption: const None(),
-        beforeOption: const None()
+        endDateOption: const None()
       );
       dispatchAction(StartSearchEmailAction());
     }
@@ -1767,7 +1792,6 @@ class MailboxDashBoardController extends ReloadableController with UserSettingPo
     log('MailboxDashBoardController::selectSortOrderQuickSearchFilter():sortOrder: $sortOrder');
     popBack();
     searchController.sortOrderFiltered.value = sortOrder;
-    searchController.updateFilterEmail(sortOrderOption: sortOrder.getSortOrder());
     dispatchAction(StartSearchEmailAction());
   }
 
@@ -1776,27 +1800,24 @@ class MailboxDashBoardController extends ReloadableController with UserSettingPo
     searchController.updateFilterEmail(
       emailReceiveTimeTypeOption: const Some(EmailReceiveTimeType.allTime),
       startDateOption: const None(),
-      endDateOption: const None(),
-      beforeOption: const None()
+      endDateOption: const None()
     );
     dispatchAction(StartSearchEmailAction());
   }
 
   void _deleteSortOrderSearchFilter() {
     searchController.sortOrderFiltered.value = EmailSortOrderType.mostRecent;
-    searchController.updateFilterEmail(
-      sortOrderOption: EmailSortOrderType.mostRecent.getSortOrder());
     dispatchAction(StartSearchEmailAction());
   }
 
   void _deleteFromSearchFilter() {
     searchController.updateFilterEmail(fromOption: const None());
-    dispatchAction(StartSearchEmailAction(filter: QuickSearchFilter.from));
+    dispatchAction(StartSearchEmailBySearchFilterAction(QuickSearchFilter.from));
   }
 
   void _deleteToSearchFilter() {
     searchController.updateFilterEmail(toOption: const None());
-    dispatchAction(StartSearchEmailAction(filter: QuickSearchFilter.to));
+    dispatchAction(StartSearchEmailBySearchFilterAction(QuickSearchFilter.to));
   }
 
   void _deleteHasAttachmentSearchFilter() {
@@ -1805,13 +1826,7 @@ class MailboxDashBoardController extends ReloadableController with UserSettingPo
   }
 
   void _deleteFolderSearchFilter() {
-    searchController.updateFilterEmail(
-      mailboxOption: const None(),
-      beforeOption: const None(),
-      positionOption: searchController.sortOrderFiltered.value.isScrollByPosition()
-        ? const Some(0)
-        : const None()
-    );
+    searchController.updateFilterEmail(mailboxOption: const None());
     dispatchAction(StartSearchEmailAction());
   }
 

lib/features/mailbox_dashboard/presentation/controller/search_controller.dart

@@ -170,7 +170,6 @@ class SearchController extends BaseController with DateRangePickerMixin {
     Option<UTCDate>? beforeOption,
     Option<UTCDate>? startDateOption,
     Option<UTCDate>? endDateOption,
-    Option<Set<Comparator>>? sortOrderOption,
     Option<int>? positionOption,
   }) {
     searchEmailFilter.value = searchEmailFilter.value.copyWith(
@@ -185,7 +184,6 @@ class SearchController extends BaseController with DateRangePickerMixin {
       beforeOption: beforeOption,
       startDateOption: startDateOption,
       endDateOption: endDateOption,
-      sortOrderOption: sortOrderOption,
       positionOption: positionOption,
     );
     searchEmailFilter.refresh();
@@ -284,6 +282,15 @@ class SearchController extends BaseController with DateRangePickerMixin {
     sortOrderFiltered.value = EmailSortOrderType.mostRecent;
   }
 
+  void clearAllFilterSearch() {
+    _clearAllTextInputSimpleSearch();
+    clearSortOrder();
+    clearFilterSuggestion();
+    clearSearchFilter();
+    deactivateAdvancedSearch();
+    hideAdvancedSearchFormView();
+  }
+
   void disableAllSearchEmail() {
     _clearAllTextInputSimpleSearch();
     deactivateSimpleSearch();

lib/features/mailbox_dashboard/presentation/model/search/search_email_filter.dart

@@ -4,7 +4,6 @@ import 'package:equatable/equatable.dart';
 import 'package:jmap_dart_client/jmap/core/filter/filter.dart';
 import 'package:jmap_dart_client/jmap/core/filter/filter_operator.dart';
 import 'package:jmap_dart_client/jmap/core/filter/operator/logic_filter_operator.dart';
-import 'package:jmap_dart_client/jmap/core/sort/comparator.dart';
 import 'package:jmap_dart_client/jmap/core/utc_date.dart';
 import 'package:jmap_dart_client/jmap/mail/email/email_filter_condition.dart';
 import 'package:model/email/prefix_email_address.dart';
@@ -28,12 +27,9 @@ class SearchEmailFilter with EquatableMixin, OptionParamMixin {
   final UTCDate? before;
   final UTCDate? startDate;
   final UTCDate? endDate;
-  final Set<Comparator>? sortOrder;
   final int? position;
 
-  factory SearchEmailFilter.initial() => SearchEmailFilter(
-    sortOrder: EmailSortOrderType.mostRecent.getSortOrder().toNullable()
-  );
+  factory SearchEmailFilter.initial() => SearchEmailFilter();
 
   SearchEmailFilter({
     Set<String>? from,
@@ -48,7 +44,6 @@ class SearchEmailFilter with EquatableMixin, OptionParamMixin {
     this.before,
     this.startDate,
     this.endDate,
-    this.sortOrder,
     this.position,
   })  : from = from ?? <String>{},
         to = to ?? <String>{},
@@ -71,7 +66,6 @@ class SearchEmailFilter with EquatableMixin, OptionParamMixin {
     Option<UTCDate>? beforeOption,
     Option<UTCDate>? startDateOption,
     Option<UTCDate>? endDateOption,
-    Option<Set<Comparator>>? sortOrderOption,
     Option<int>? positionOption,
   }) {
     return SearchEmailFilter(
@@ -87,7 +81,6 @@ class SearchEmailFilter with EquatableMixin, OptionParamMixin {
       before: getOptionParam(beforeOption, before),
       startDate: getOptionParam(startDateOption, startDate),
       endDate: getOptionParam(endDateOption, endDate),
-      sortOrder: getOptionParam(sortOrderOption, sortOrder),
       position: getOptionParam(positionOption, position),
     );
   }
@@ -166,7 +159,6 @@ class SearchEmailFilter with EquatableMixin, OptionParamMixin {
     before,
     startDate,
     endDate,
-    sortOrder,
     position,
   ];
 }