[#44384] - [5.2]postgres and finder suggestions
- Pending
- Medium
- Build: 5.2-dev
- # 44384
- Diff
- alikon:patch-23
User tests: Successful: Unsuccessful:
Pull Request for Issue #44373
.
Summary of Changes
rewrite the query
Testing Instructions
create content with few words
be sure it indexed by Smart search component
add module Smart search to site
type word into search edit
Actual result BEFORE applying this Pull Request
No suggestions
Expected result AFTER applying this Pull Request
suggestions
Link to documentations
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
Votes
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End com_finder |
| Labels |
Added:
PR-5.2-dev
|
||
It works for me in Postgresql 16.4, MySQL 8 and MariaDB 5.6 @richard67 I'm asking you to merge this PR
@softarius Could you mark your test result in the issue tracker https://issues.joomla.org/tracker/joomla-cms/44384 by using the blue "Test this" button at the top left corner, select your test result (success) and submit? Every non-trivial PR needs 2 successful human tests, and when you mark your test result 50% is done. Thanks in advance.
I have tested this item ✅ successfully on 840858f
Suceessfully tested in Postgresql 16.4, MySQL 8 and MariaDB 5.6
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44384.
@softarius Thanks for testing. Unfortunately I think this PR might not be in time for the upcoming 5.2.2 because packages for that version are already in preparation for being tested by the CMS Release Team, so it would be the next version after that, 5.2.3, which is planned for January 7, 2025. Not my responsibility or decision, I only report what I know. I will try my best to bring this PR forward, but I can't promise that it will help to bring it into 5.2.2.
It works for me in Postgresql 16.4, MySQL 8 and MariaDB 5.6
@richard67 I'm asking you to merge this PR