User tests: Successful: Unsuccessful:
There are situations where for some reason the AJAX call for suggestions in Smart Search fails. When typing very fast into the search box and hitting enter, the browser has fired an AJAX call for every character typed and the server might not have responded. When hitting enter, the form is send and all AJAX requests are aborted. This lets the JS code return and run into the failure condition, resulting in a flash of error messages being displayed. While it is nice to know that something failed for a developer, this should not be displayed to the general enduser. The JS code right now can spam the screen with system messages for these failed calls, especially since a call is triggered for each character entered into the search box. This PR removes this error handling.
/components/com_finder/src/Controller/SuggestionsController.php
and insertecho '1';
Please select:
Documentation link for docs.joomla.org:
No documentation changes for docs.joomla.org needed
Pull Request link for manual.joomla.org:
No documentation changes for manual.joomla.org needed
Status | New | ⇒ | Pending |
Category | ⇒ | JavaScript Repository NPM Change |
Labels |
Added:
NPM Resource Changed
PR-5.0-dev
|
Done and I found out the reason for the error. I described it in the description at the top.
I have tested the patch in two 4.4.1 Joomla Websites where this message had shown very often and was irritating the visitor of the sites. In both cases the message dissappeared after transferring the files. Thank you Hackwar
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2023-12-10 16:22:54 |
Closed_By | ⇒ | Hackwar |
I suggest to add:
Or it will be impossible to debug in case of error.