Skip to content

Comments

Bugfix for Issue #38599 - Smart Search indexing macros from articles#38998

Merged
obuisard merged 13 commits intojoomla:4.3-devfrom
MacJoom:bugfix-38599
Aug 29, 2023
Merged

Bugfix for Issue #38599 - Smart Search indexing macros from articles#38998
obuisard merged 13 commits intojoomla:4.3-devfrom
MacJoom:bugfix-38599

Conversation

@MacJoom
Copy link
Contributor

@MacJoom MacJoom commented Oct 20, 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

@MacJoom MacJoom changed the title Bugfix for Issue #88599 - Smart Search indexing macros from articles Bugfix for Issue #38599 - Smart Search indexing macros from articles Oct 20, 2022
@Quy
Copy link
Contributor

Quy commented Jan 24, 2023

Please fix conflicts. Thanks.

@joomla-cms-bot joomla-cms-bot added the NPM Resource Changed This Pull Request can't be tested by Patchtester label Jan 29, 2023
@MacJoom
Copy link
Contributor Author

MacJoom commented Jan 29, 2023

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

@Hackwar Hackwar added the Small A PR which only has a small change label Feb 25, 2023
@joomla-cms-bot joomla-cms-bot removed the NPM Resource Changed This Pull Request can't be tested by Patchtester label Mar 12, 2023
@MacJoom MacJoom changed the base branch from 4.2-dev to 4.3-dev March 13, 2023 00:13
@Hackwar Hackwar added the bug label Apr 7, 2023
Move comment before test

Co-authored-by: Quy <quy@nomonkeybiz.com>
@Hackwar Hackwar added the PBF Pizza, Bugs and Fun label Aug 25, 2023
@ChrisHoefliger
Copy link

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.

@rachelwalraven
Copy link

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.

@joomla-cms-bot joomla-cms-bot removed the PBF Pizza, Bugs and Fun label Aug 26, 2023
@alikon
Copy link
Contributor

alikon commented Aug 26, 2023

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Aug 26, 2023
@obuisard obuisard added this to the Joomla! 4.3.5 milestone Aug 29, 2023
@obuisard obuisard merged commit b0619cd into joomla:4.3-dev Aug 29, 2023
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Aug 29, 2023
@obuisard
Copy link
Contributor

Thank you Martin @MacJoom for this PR!

HLeithner pushed a commit that referenced this pull request Sep 4, 2023
* fix calendar positioning (#41477)

* Update joomla-field-media.w-c.es6.js (#41361)

* Fixing md5 Deprecation in document.php (#39880)

* modal field check for danger alert (#39173)

* fix (#41138)

* [4.x] Cassiopeia Install SQL (#37389)

* Fix/showon (#37451)

* Bugfix for Issue #38599 - Smart Search indexing macros from articles (#38998)

* [4,3] joomla alert icons (#39312)

* Update StyleModel.php (#40665)

* Update Query.php (#40687)

* remove hitcount variable from uri after voting (#41474)

* Fix deprecated message in categories when extension is empty (#40604)

* Re-add minus to allowed filenames in com_templates (#41369)

* Invalid delete of all content articles from smart search index (#41288)

* [4][cli] schema update check (#40468)

* Tags: Don't put NULL into trim() (#41509)

Co-authored-by: Quy <quy@nomonkeybiz.com>

* [4] Add multilanguage filter taxonomy rows in com_finder advanced filters (#41470)

* Fix taxonomy filter options

* refactor rest of query to prepared statements

---------

Co-authored-by: Quy <quy@nomonkeybiz.com>

* Check if the pre update check box actually exists (#41467)

* Allow onValidateContact plugin to show error message (#37540)

* [4.4] Backward compatibility handling for plugins that setting the result directly (#41525)

* Backward compatibility handling for plugins that setting the result directly.

* Update libraries/src/Event/AbstractImmutableEvent.php

Co-authored-by: Brian Teeman <brian@teeman.net>

---------

* Tests for com_privacy consent on user site (#41529)

* Tests for com_privacy consent on user site

* Allow ability to enable extension in tests to prevent errors

* remove paramater not needed

* Added fixes to stop  errors in drone build

* Added fixes to stop  errors in drone build

* Added fixes to stop  errors in drone build

* [4.4] system test for com_installer (#41483)

* system test for com_installer

* cs

* Missing System Tests in #41362 (#41479)

* Update Categories.cy.js

* Update Categories.cy.js

* Update Categories.cy.js

* Update Categories.cy.js

* Update Categories.cy.js

* Update Categories.cy.js

added spaces and semicolons

* Update Categories.cy.js

Missing spaces and semicolons

* Update Categories.cy.js

Missing spaces and semicolons

* Update Categories.cy.js

Missing spaces and semicolons

* Update Categories.cy.js

removed tabs

* Update Categories.cy.js

Removed tabs

* Update Categories.cy.js

Removed tabs

* Update Categories.cy.js

Removed tabs

* Update Language.php (#41353)

* Filter for special characters that fail sending test mail (#41469)

* Backporting event changes (#41484)

Co-authored-by: Allon Moritz <allon.moritz@digital-peak.com>

* api tests for endpoints for com_privacy consent (#41544)

* Show the plugin name in the update screen (#41537)

* Show the plugin name in the update screen

* cs

---------

* Replace the special characters in site name for mail templates (#41582)

* Updates the dependencies (#41538)

* Updating dependencies

* Major update cypress

* Thanks for the reminder @brianteeman

* Composer

* weekend updates

* Update dependencies

* Update incompatible HTML file

---------

Co-authored-by: Magnus Singer <magnussinger@icloud.com>
Co-authored-by: Dimitris Grammatikogiannis <d.grammatiko@gmail.com>
Co-authored-by: Crystal Dionysopoulos <9141288+crystalenka@users.noreply.github.com>
Co-authored-by: heelc29 <66922325+heelc29@users.noreply.github.com>
Co-authored-by: Denitz <197527+Denitz@users.noreply.github.com>
Co-authored-by: Brian Teeman <brian@teeman.net>
Co-authored-by: toroworx <toroworx@users.noreply.github.com>
Co-authored-by: Martin Carl Kopp <6154099+MacJoom@users.noreply.github.com>
Co-authored-by: Jonathan Brain <3941269+BrainforgeUK@users.noreply.github.com>
Co-authored-by: Olivier Buisard <olivier.buisard@simplifyyourweb.com>
Co-authored-by: Christiane Maier-Stadtherr <dev@chmst.de>
Co-authored-by: Manuel Huber <ManuelHu@users.noreply.github.com>
Co-authored-by: Nicola Galgano <optimus4joomla@gmail.com>
Co-authored-by: Hannes Papenberg <info@joomlager.de>
Co-authored-by: Quy <quy@nomonkeybiz.com>
Co-authored-by: David Jardin <d.jardin@djumla.de>
Co-authored-by: beefcakefu <103491197+beefcakefu@users.noreply.github.com>
Co-authored-by: Fedir Zinchuk <getthesite@gmail.com>
Co-authored-by: andyforrest <46865032+andyforrest@users.noreply.github.com>
@MacJoom MacJoom deleted the bugfix-38599 branch May 16, 2024 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Small A PR which only has a small change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants