Fix(SavedSearch): Force user to validate criteria before saved search

glpi-project · Jan 16, 2025 · 08df0af · 08df0af
1 parent 0a73477
commit 08df0af
Showing 1 changed file with 14 additions and 0 deletions.
diff --git a/js/modules/Search/Table.js b/js/modules/Search/Table.js
@@ -44,6 +44,19 @@ window.GLPI.Search.Table = class Table extends GenericView {
         super(element_id);
 
         this.shiftSelectAllCheckbox();
+        this.toggleSavedSearch(true);
+    }
+
+    toggleSavedSearch(isDisable) {
+        if (isDisable) {
+            $('.bookmark_record')
+                .attr('title', __('Submit current search before saving it'))
+                .prop('disabled', true);
+        } else {
+            $('.bookmark_record')
+                .attr('title', __('Save current search'))
+                .prop('disabled', false);
+        }
     }
 
     getElement() {
@@ -199,6 +212,7 @@ window.GLPI.Search.Table = class Table extends GenericView {
                 this.getElement().trigger('search_refresh', [this.getElement()]);
                 this.hideLoadingSpinner();
                 this.shiftSelectAllCheckbox();
+                this.toggleSavedSearch(false);
             }, () => {
                 handle_search_failure();
             });