[#44157] - [5.2] mod_articles : missing onContentPrepare plugin event
- Closed
- 3 Oct 2024
- Medium
- Build: 5.2-dev
- # 44157
- Diff
- conseilgouz:5.2-dev
User tests: Successful: Unsuccessful:
Summary of Changes
In mod_articles_news, there's a trigger event onContentPrepare (line 170 in ArticlesNewsHelper.php).
It's missing in mod_modules.
This PR adds this missing event.
Testing Instructions
Actual result BEFORE applying this Pull Request
Expected result AFTER applying this Pull Request
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
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Modules Front End |
| Labels |
Added:
PR-5.2-dev
|
||
| Category | Modules Front End | ⇒ | Language & Strings Modules Front End |
mod_articles.ini file has been updated.
@Kostelano : thank you for telling me about this.
The event already trigered by
But only for text
The event is triggered only for displayIntroText.
If any other item field is affected by this event, it will not be done.
The event is triggered only for displayIntroText.
Yeah.
The problem, that the event will be triggered TWICE now.
This need a complex solution.
The problem, that the event will be triggered TWICE now.
This need a complex solution.
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2024-10-03 08:45:45 |
| Closed_By | ⇒ | conseilgouz | |
| Labels |
Added:
Language Change
|
||
If this should be controlled by this option, please add the information to the language line:
MOD_ARTICLES_FIELD_TRIGGER_EVENTS_DESC="Enable this option to trigger three plugin events (afterDisplayTitle, beforeDisplayContent, afterDisplayContent) for each article in the results list."