New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding VIEW_INJECT_CUSTOM_TEMPLATE back even in the Twig world #12924
Adding VIEW_INJECT_CUSTOM_TEMPLATE back even in the Twig world #12924
Conversation
This pull request has been mentioned on Mautic Forums. There might be relevant details there: https://forum.mautic.org/t/what-is-mautic-5-way-of-injecting-custom-templates/29846/3 |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## 5.x #12924 +/- ##
============================================
+ Coverage 58.81% 58.85% +0.03%
- Complexity 33627 33628 +1
============================================
Files 2181 2181
Lines 101821 101826 +5
============================================
+ Hits 59889 59925 +36
+ Misses 41932 41901 -31
|
c87fc08
to
ecda012
Compare
This pull request has been mentioned on Mautic Forums. There might be relevant details there: |
Hi @escopecz , I just did the basic test on this and it works OK. When I adjust the actual plugin and test it, I will make a review here. You did something very valuable here, thank you. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works. Nice
you beat me to the review, kuzmany, thanks. |
Description:
The
CoreEvents::VIEW_INJECT_CUSTOM_TEMPLATE
event used to be in the PhpEngine but was removed together with it.I found another place to put it to work as before even with the twig templates.
The IntegrationBundle actually used that event to display object mapping on the contact detail page, so that was broken.
Steps to test this PR:
CoreEvents::VIEW_INJECT_CUSTOM_TEMPLATE
event was removed together with the PhpEngine, this functionality stopped working.