Merge pull request #1652 from PrabhuSuriya/1649-advanced-search-attac…

…hment Advanced search is not filtering by attachments and review others
CTemplar · Feb 11, 2022 · d18bf0e · d18bf0e
2 parents 9b9f244 + 7c584d1
commit d18bf0e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/src/app/mail/mail-list/mail-folder/generic-folder/generic-folder.component.ts b/src/app/mail/mail-list/mail-folder/generic-folder/generic-folder.component.ts
@@ -665,6 +665,7 @@ export class GenericFolderComponent implements OnInit, AfterViewInit {
         this.advancedSearchQuery.size = parameters.size;
         this.advancedSearchQuery.size_operator = parameters.size_operator;
         this.advancedSearchQuery.exact = parameters.exact;
+        this.advancedSearchQuery.have_attachment = parameters.have_attachment;
         this.store.dispatch(
           new GetMails({
             forceReload: true,

diff --git a/src/app/shared/components/advanced-search/advanced-search.component.html b/src/app/shared/components/advanced-search/advanced-search.component.html
@@ -237,8 +237,8 @@
         [(ngModel)]="sameExactly"
         [ngModelOptions]="{ standalone: true }"
       >
+      Same exactly
       </mat-checkbox>
-      <label for="sameExactly" class="fancy-field-empty">Same exactly</label>
     </div>
     <div class="col-6">
       <mat-checkbox
@@ -249,8 +249,8 @@
         [(ngModel)]="hasAttachment"
         [ngModelOptions]="{ standalone: true }"
       >
-      </mat-checkbox>
-      <label for="hasAttachment" class="fancy-field-empty">Has attachment</label>
+      Has attachment
+    </mat-checkbox>
     </div>
   </div>
   <div class="d-flex justify-content-between mt-4">