? ? Pending

User tests: Successful: Unsuccessful:

avatar Hackwar
Hackwar
13 Jun 2021

This is the Pull Request for the J3 Issue #34242 in 4.0.

Summary of Changes

When indexing, due to a logic error, the words of a text were only stored to the database in chunks of 128 at a time. If a chunk had less than 128 words, it wasn't saved to the database. This change fixes that. However, in 4.0, the bug had been fixed in a different way. In order to keep the code in sync, the better and shorter solution of 3.9 is used here.

Testing Instructions

Have Smart Search be set up. Write a short article (less than 128 words) with a unique word and save it. Then go to the frontend and search for that unique word. You don't get a result.
Apply the patch, reindex the content and search for it again. Now you get the article as result.

avatar Hackwar Hackwar - open - 13 Jun 2021
avatar Hackwar Hackwar - change - 13 Jun 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 13 Jun 2021
Category Administration com_finder
avatar sandramay0905
sandramay0905 - comment - 14 Jun 2021

However, in 4.0, the bug had been fixed (...) You don't get a result.

I created a article in english on a multilanguage site having ten words including unique word "sousousouxxx" and get a result with and without this pull request.

@Hackwar I'm not sure if this is a successfully test.

avatar Hackwar Hackwar - change - 14 Jun 2021
Labels Added: ?
avatar Hackwar
Hackwar - comment - 14 Jun 2021

However, in 4.0, the bug had been fixed (...) You don't get a result.

I created a article in english on a multilanguage site having ten words including unique word "sousousouxxx" and get a result with and without this pull request.

@Hackwar I'm not sure if this is a successfully test.

As I wrote, this worked in 4.0 already, but this code is the better solution. So, yes, this would be a successfull test.

avatar sandramay0905 sandramay0905 - test_item - 14 Jun 2021 - Tested successfully
avatar sandramay0905
sandramay0905 - comment - 14 Jun 2021

I have tested this item successfully on 12299c1

Thanks for information @Hackwar I was not sure cause of "You don't get a result."


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34503.

avatar richard67 richard67 - alter_testresult - 14 Jun 2021 - sandramay0905: Tested successfully
avatar richard67
richard67 - comment - 14 Jun 2021

@Hackwar I've allowed myself to commit another fix for PHPCS and restored the previous test result in the issue tracker since the change which invalidated it was only the code style fix.

avatar RickR2H RickR2H - test_item - 14 Jun 2021 - Tested successfully
avatar RickR2H
RickR2H - comment - 14 Jun 2021

I have tested this item successfully on 3f281d7

Search result show the unique word from the short article.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34503.

avatar richard67 richard67 - change - 14 Jun 2021
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 14 Jun 2021

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34503.

avatar chmst chmst - change - 14 Jun 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-06-14 20:16:14
Closed_By chmst
Labels Added: ?
avatar chmst chmst - close - 14 Jun 2021
avatar chmst chmst - merge - 14 Jun 2021
avatar chmst
chmst - comment - 14 Jun 2021

Thanks!

Add a Comment

Login with GitHub to post a comment