User tests: Successful: Unsuccessful:
Pull Request for Issue #38599 .
finder aka smart search index contains macros of loadmodule in search results
in onContentPrepare of loadmodule:
while indexing remove macros (loadposition, loadmoduleid, loadmodule) from article text with preg_replace
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.
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.
The search result does contain the article where the loadposition macro is placed and the suggestion list does not contain both loadposition and herewego.
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 | ⇒ | Front End Plugins |
Title |
|
Labels |
Added:
Conflicting Files
?
|
Category | Front End Plugins | ⇒ | Repository NPM Change Front End Plugins |
Labels |
Removed:
Conflicting Files
|
I don't know why _modals.css is in this commit... - can be removed
Labels |
Added:
NPM Resource Changed
?
|
Category | Front End Plugins Repository NPM Change | ⇒ | Front End Plugins |
Labels |
Added:
PR-4.3-dev
Removed: NPM Resource Changed ? |
Labels |
Added:
bug
|
I have tested this item ✅ successfully on 9a37f4e
Tested on Joomla 4.3.4 @ PBF 2023
I have tested this item ✅ successfully on 9a37f4e
On Joomla 4.3 with PHP 8.1.22
Status | Pending | ⇒ | Ready to Commit |
RTC
Labels |
Added:
?
|
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 |
Please fix conflicts. Thanks.