Language Change NPM Resource Changed PR-5.0-dev Pending

User tests: Successful: Unsuccessful:

avatar HLeithner
HLeithner
21 Jul 2023

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.

Summary of Changes

  • Move class aliasing to the constructor
  • Move es5 webassets to the plugin

Testing Instructions

Full CMS with and without plugin active

Actual result BEFORE applying this Pull Request

Should work

Expected result AFTER applying this Pull Request

Should still work

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

avatar joomla-cms-bot joomla-cms-bot - change - 21 Jul 2023
Category SQL Administration com_admin Postgresql Language & Strings Repository NPM Change
avatar HLeithner HLeithner - open - 21 Jul 2023
avatar HLeithner HLeithner - change - 21 Jul 2023
Status New Pending
avatar HLeithner
HLeithner - comment - 21 Jul 2023

Do you have anything to say @SharkyKZ or would you only like to spread bad vibes?

avatar HLeithner HLeithner - change - 21 Jul 2023
Labels Added: Language Change NPM Resource Changed PR-5.0-dev
avatar brianteeman
brianteeman - comment - 21 Jul 2023

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

avatar joomla-cms-bot joomla-cms-bot - change - 21 Jul 2023
Category SQL Administration com_admin Postgresql Language & Strings Repository NPM Change SQL Administration com_admin Postgresql Language & Strings Templates (admin) Repository NPM Change
avatar HLeithner
HLeithner - comment - 21 Jul 2023

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

avatar brianteeman
brianteeman - comment - 21 Jul 2023

I just grepped for es5 in all joomla.asset.json files

avatar brianteeman
brianteeman - comment - 21 Jul 2023

Does this need changing as well

"name": "bootstrap.es5",
"type": "script",
"deprecated": true,
"uri": "",
"dependencies": [
"core"
],
"attributes": {
"nomodule": true,
"defer": true
}

avatar HLeithner
HLeithner - comment - 21 Jul 2023

Does this need changing as well

"name": "bootstrap.es5",
"type": "script",
"deprecated": true,
"uri": "",
"dependencies": [
"core"
],
"attributes": {
"nomodule": true,
"defer": true
}

I'm not sure I have seen it, @Fedik @dgrammatiko ?

avatar Fedik
Fedik - comment - 21 Jul 2023

All what in settings will be in vendor/assets.json
When you added it in to the legacy.json, then remove it from settings.

avatar HLeithner
HLeithner - comment - 21 Jul 2023

thanks moved it to the plugin

avatar HLeithner HLeithner - change - 22 Jul 2023
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2023-07-22 09:35:20
Closed_By HLeithner
avatar HLeithner HLeithner - close - 22 Jul 2023
avatar HLeithner HLeithner - merge - 22 Jul 2023
avatar HLeithner
HLeithner - comment - 22 Jul 2023

Hopefully this solve all the JPlugin issues, merging it to get it into alpha3

Add a Comment

Login with GitHub to post a comment