fix: implementation fixes after @Guileas's manual review

Author: williamdes

src/Resources/themes/default/doctum.js.twig

@@ -24,7 +24,7 @@ var Doctum = {
     doctumSearchAutoCompleteProgressBarPercent: 0,
     /** @var autoComplete|null */
     autoCompleteJS: null,
-    querySearchSecurityRegex: /([^a-zA-Z:\\\\_\s])/gi,
+    querySearchSecurityRegex: /([^0-9a-zA-Z:\\\\_\s])/gi,
     buildTreeNode: function (treeNode, htmlNode, treeOpenLevel) {
         var ulNode = document.createElement('ul');
         for (var childKey in treeNode.c) {
@@ -152,7 +152,6 @@ var Doctum = {
                     document.getElementById('search-form').submit();
                 });
                 Doctum.doctumSearchAutoComplete.addEventListener('navigate', function (event) {
-                    Doctum.markInProgress();
                     // Set selection in text box
                     if (typeof event.detail.selection.value === 'object') {
                         Doctum.doctumSearchAutoComplete.value = event.detail.selection.value.n;
@@ -314,6 +313,9 @@ var Doctum = {
                 query: (input) => {
                     return Doctum.cleanSearchQuery(input);
                 },
+                trigger: (query) => {
+                    return Doctum.cleanSearchQuery(query).length > 0;
+                },
                 resultsList: {
                     tag: 'ul',
                     class: 'auto-complete-dropdown-menu',

src/Resources/themes/default/search.twig

@@ -98,7 +98,7 @@
                 return DoctumSearch.longTypes[type] || DoctumSearch.longTypes['_'];
             },
             pageFullyLoaded: function (event) {// Will get fired by the main doctum.js script
-                DoctumSearch.searchTerm = DoctumSearch.cleanSearchTerm();
+                DoctumSearch.searchTerm = Doctum.cleanSearchQuery(DoctumSearch.cleanSearchTerm());
                 DoctumSearch.doctumSearchPageAutoCompleteProgressBarContainer = document.getElementById('search-page-progress-bar-container');
                 DoctumSearch.doctumSearchPageAutoCompleteProgressBar = document.getElementById('search-page-progress-bar');
                 DoctumSearch.pageFullyLoaded = true;
@@ -142,6 +142,9 @@
                         query: (input) => {
                             return Doctum.cleanSearchQuery(input);
                         },
+                        trigger: (query) => {
+                            return Doctum.cleanSearchQuery(query).length > 0;
+                        },
                         resultsList: {
                             tag: 'ul',
                             class: 'search-results',