? bug PR-4.3-dev ? Pending

User tests: Successful: Unsuccessful:

avatar MacJoom
MacJoom
20 Oct 2022

Pull Request for Issue #38599 .
finder aka smart search index contains macros of loadmodule in search results

Summary of Changes

in onContentPrepare of loadmodule:
while indexing remove macros (loadposition, loadmoduleid, loadmodule) from article text with preg_replace

Testing Instructions (from #38599)

Enable smart search aka finder and publish a finder search module somewhere or connect a menu item to the smart search item type.
Enable the load modules plugin.
Enable the smart search content plugin.
Create a site module of type custom with some content and place it at custom positon herewego.
Create an article and place a loadposition macro in that article like this:
{loadposition herewego}

Now the finder automatically indexes the new article.
Go to the search box and search for loadposition or herewego.

Within the Smart Search component you can clear the index under maintenace and reindex with the index button
Do a clear before applying the PR.

Actual result BEFORE applying this Pull Request

I would not expect to see search results containing the article with the loadposition macro.

e.g. {loadposition herewego} in the search results

I would not expect the search suggestions list containing the search items loadposition and herewego.
Instead I would expect that the macro is deleted in this case before indexing.

Expected result AFTER applying this Pull Request

The search result does contain the article where the loadposition macro is placed and the suggestion list does not contain both loadposition and herewego.

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

avatar MacJoom MacJoom - open - 20 Oct 2022
avatar MacJoom MacJoom - change - 20 Oct 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 20 Oct 2022
Category Front End Plugins
avatar MacJoom MacJoom - change - 20 Oct 2022
The description was changed
avatar MacJoom MacJoom - edited - 20 Oct 2022
avatar MacJoom MacJoom - change - 20 Oct 2022
Title
Bugfix for Issue #88599 - Smart Search indexing macros from articles
Bugfix for Issue #38599 - Smart Search indexing macros from articles
avatar MacJoom MacJoom - edited - 20 Oct 2022
avatar Quy
Quy - comment - 24 Jan 2023

Please fix conflicts. Thanks.

avatar MacJoom MacJoom - change - 28 Jan 2023
Labels Added: Conflicting Files ?
avatar MacJoom MacJoom - change - 28 Jan 2023
The description was changed
avatar MacJoom MacJoom - edited - 28 Jan 2023
avatar joomla-cms-bot joomla-cms-bot - change - 29 Jan 2023
Category Front End Plugins Repository NPM Change Front End Plugins
avatar MacJoom MacJoom - change - 29 Jan 2023
Labels Removed: Conflicting Files
avatar MacJoom
MacJoom - comment - 29 Jan 2023

I don't know why _modals.css is in this commit... - can be removed

avatar MacJoom MacJoom - change - 12 Mar 2023
Labels Added: NPM Resource Changed ?
avatar joomla-cms-bot joomla-cms-bot - change - 12 Mar 2023
Category Front End Plugins Repository NPM Change Front End Plugins
avatar MacJoom MacJoom - change - 13 Mar 2023
Labels Added: PR-4.3-dev
Removed: NPM Resource Changed ?
avatar MacJoom MacJoom - change - 17 May 2023
Labels Added: bug
avatar ChrisHoefliger ChrisHoefliger - test_item - 26 Aug 2023 - Tested successfully
avatar ChrisHoefliger
ChrisHoefliger - comment - 26 Aug 2023

I have tested this item ✅ successfully on 9a37f4e

Tested on Joomla 4.3.4 @ PBF 2023


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

avatar rachelwalraven rachelwalraven - test_item - 26 Aug 2023 - Tested successfully
avatar rachelwalraven
rachelwalraven - comment - 26 Aug 2023

I have tested this item ✅ successfully on 9a37f4e

On Joomla 4.3 with PHP 8.1.22


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

avatar alikon alikon - change - 26 Aug 2023
Status Pending Ready to Commit
avatar alikon
alikon - comment - 26 Aug 2023

RTC


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

avatar obuisard obuisard - change - 29 Aug 2023
Labels Added: ?
avatar obuisard obuisard - close - 29 Aug 2023
avatar obuisard obuisard - merge - 29 Aug 2023
avatar obuisard obuisard - change - 29 Aug 2023
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2023-08-29 01:33:12
Closed_By obuisard
avatar obuisard
obuisard - comment - 29 Aug 2023

Thank you Martin @MacJoom for this PR!

Add a Comment

Login with GitHub to post a comment