User tests: Successful: Unsuccessful:
This PR moves the class alias loading to the constructor function of the plugin to load the alias at the earliest possible execution point.
Additional this PR moves all es5 assets from all joomla.asset.json into the es5.asset.json and make loading them optional.
Full CMS with and without plugin active
Should work
Should still work
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
Category | ⇒ | SQL Administration com_admin Postgresql Language & Strings Repository NPM Change |
Status | New | ⇒ | Pending |
Labels |
Added:
Language Change
NPM Resource Changed
PR-5.0-dev
|
I see that there are still es5 references in
administrator\templates\atum\joomla.asset.json
build\media_source\com_joomlaupdate\joomla.asset.json
build\media_source\plg_system_jooa11y\joomla.asset.json
Category | SQL Administration com_admin Postgresql Language & Strings Repository NPM Change | ⇒ | SQL Administration com_admin Postgresql Language & Strings Templates (admin) Repository NPM Change |
I see that there are still es5 references in administrator\templates\atum\joomla.asset.json build\media_source\com_joomlaupdate\joomla.asset.json build\media_source\plg_system_jooa11y\joomla.asset.json
Thanks, don't know why my initial search didn't found the 3 assets.... in the build folder... the template I didn't checked tbh
I just grepped for es5 in all joomla.asset.json files
Does this need changing as well
joomla-cms/build/build-modules-js/settings.json
Lines 77 to 87 in e7e1fb4
Does this need changing as well
joomla-cms/build/build-modules-js/settings.json
Lines 77 to 87 in e7e1fb4
I'm not sure I have seen it, @Fedik @dgrammatiko ?
All what in settings will be in vendor/assets.json
When you added it in to the legacy.json, then remove it from settings.
thanks moved it to the plugin
Status | Pending | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2023-07-22 09:35:20 |
Closed_By | ⇒ | HLeithner |
Hopefully this solve all the JPlugin issues, merging it to get it into alpha3
Do you have anything to say @SharkyKZ or would you only like to spread bad vibes?