Skip to content

Conversation

@janschoenherr
Copy link
Contributor

Summary of Changes

Currently it's not possible to manipulate the result within the onAfterDisplay event of the HtmlView class.
The event was introduced here: e8dfbb4

This change makes it possible to change the output of a template script and have the changed output cached with Joomla's progressive caching enabled.

Testing Instructions

No test instructions. No views should be affected.

Actual result BEFORE applying this Pull Request

This change should not cause any change in Joomla's behavior.

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

@janschoenherr janschoenherr changed the title [6.1] Use result of onAfterDisplay event [6.1] Echo result of onAfterDisplay event Jan 29, 2026
@rdeutz
Copy link
Contributor

rdeutz commented Jan 29, 2026

Conflict with #46770 as far as I see it.

@janschoenherr
Copy link
Contributor Author

I will rebase the commit once the other PR is accepted.

@janschoenherr
Copy link
Contributor Author

It's resolved now 👍

@chaitali-tekdi
Copy link

I have tested this item ✅ successfully on 26d40d0

I tested this issue, and it was tested successfully.


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

@komalm
Copy link

komalm commented Jan 31, 2026

I have tested this item ✅ successfully on d22b0ac


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

@Denitz
Copy link
Contributor

Denitz commented Jan 31, 2026

Please note that it was rejected 3 years ago: #41506
To be honest, it's a 101% required PR and it should be merged.

@janschoenherr
Copy link
Contributor Author

janschoenherr commented Feb 2, 2026

@Denitz Thanks, we could also reopen your PR. The import thing to note is that there is no way to modify the output and store that modification in the cache.

@Denitz
Copy link
Contributor

Denitz commented Feb 2, 2026

No need to open my PR, let's use yours - the fresh one.

@Denitz
Copy link
Contributor

Denitz commented Feb 4, 2026

I have tested this item ✅ successfully on 089fb6a


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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants