[#41825] - [5.0] Better support for JHtml prefix in HTMLHelper
- Fixed in Code Base
- 20 Sep 2023
- Medium
- Build: 5.0-dev
- # 41825
- Diff
- HLeithner:5.0/fix/htmlhelper-jhtml-prefix
User tests: Successful: Unsuccessful:
Pull Request for Issue #41821 .
Summary of Changes
Fallback to core naming schema for 3rd party extensions which uses JHtml as prefix for HTMLHelper classes.
Testing Instructions
Use an extensions which uses JHtmlXXX HTMLHelper
Actual result BEFORE applying this Pull Request
Doesn't work because we check only for the namespace version Joomla\CMS\HTML\HTMLHelper
Expected result AFTER applying this Pull Request
Work because we now check also for the legacy prefix version JHtml
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
Votes
joomla-cms-bot
- | Category | ⇒ | Libraries |
| Status | New | ⇒ | Pending |
I have tested this item ✅ successfully on 34c246c
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/41825.
| Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2023-09-20 09:09:38 |
| Closed_By | ⇒ | HLeithner | |
| Labels |
Added:
PR-5.0-dev
|
||
thanks
Should this not be added into the bc plugin?
if you have a suggestion how I can move this to the b/c plugin.
I mean yes I can trigger an event or check the parameter of the plugin but not sure if this is really a good way.
I'm open for it of course I would like to move much more to the plugin, we will see if this is possible in minor versions.
tested and valided
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/41825.