? ? Pending

User tests: Successful: Unsuccessful:

avatar PhilETaylor
PhilETaylor
8 May 2021

Closes #33663

Summary of Changes

I have no idea what any of this means, I just did as I was told by @dgrammatiko haha

Testing Instructions

git checkout 4.0-dev
run the standard jinstall command (as aliased in the docs to be:

alias jclean="rm -rf administrator/templates/atum/css; rm -rf templates/cassiopeia/css; rm -rf administrator/templates/system/css; rm -rf templates/system/css; rm -rf media/; rm -rf node_modules/; rm -rf libraries/vendor/;rm -f administrator/cache/autoload_psr4.php;rm -rf installation/template/css"
alias jinstall="jclean; composer install; npm ci"

Actual result BEFORE applying this Pull Request

image

Expected result AFTER applying this Pull Request

No changes after running command.

Documentation Changes Required

avatar PhilETaylor PhilETaylor - open - 8 May 2021
avatar PhilETaylor PhilETaylor - change - 8 May 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 8 May 2021
Category JavaScript Repository
avatar PhilETaylor PhilETaylor - change - 8 May 2021
The description was changed
avatar PhilETaylor PhilETaylor - edited - 8 May 2021
avatar dgrammatiko dgrammatiko - test_item - 8 May 2021 - Tested successfully
avatar dgrammatiko
dgrammatiko - comment - 8 May 2021

I have tested this item successfully on deddb71


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

avatar PhilETaylor PhilETaylor - change - 8 May 2021
The description was changed
avatar PhilETaylor PhilETaylor - edited - 8 May 2021
avatar PhilETaylor
PhilETaylor - comment - 8 May 2021

This did not work :(

Screenshot 2021-05-08 at 22 51 05

Screenshot 2021-05-08 at 22 51 02

avatar dgrammatiko
dgrammatiko - comment - 8 May 2021

This did not work :(

Wrong expectations?
Let me clarify what is happening here. Before #32300 the languages were ordered ascending. After the refactoring the order was random (mostly en-US first and the en-GB but really dependant on the filesystem). Applying this PR the order becomes again asc (en-GB first, en-US second)

avatar PhilETaylor
PhilETaylor - comment - 8 May 2021

ah so you want me to commit these two modified files too? then they will stay like this...

avatar dgrammatiko
dgrammatiko - comment - 8 May 2021

ah so you want me to commit these two modified files too? then they will stay like this...

Yup, that should fix it for everybody and set the ordering to something that is more predictable

avatar PhilETaylor PhilETaylor - change - 8 May 2021
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 8 May 2021
Category JavaScript Repository JavaScript Repository Front End Templates (site)
avatar PhilETaylor
PhilETaylor - comment - 8 May 2021

Yup, that should fix it for everybody

ok added those files after building them with the new JS

avatar YatharthVyas YatharthVyas - test_item - 9 May 2021 - Tested successfully
avatar YatharthVyas
YatharthVyas - comment - 9 May 2021

I have tested this item successfully on 5e8b84a

I was scratching my head over this issue yesterday. Thanks for fixing it!
I tracked the initial files before and after a clean install, they weren't modified.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/33674.
avatar dgrammatiko dgrammatiko - test_item - 9 May 2021 - Tested successfully
avatar dgrammatiko
dgrammatiko - comment - 9 May 2021

I have tested this item successfully on 5e8b84a


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

avatar alikon alikon - change - 9 May 2021
Status Pending Ready to Commit
avatar alikon
alikon - comment - 9 May 2021

RTC


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

avatar richard67 richard67 - close - 9 May 2021
avatar richard67 richard67 - merge - 9 May 2021
avatar richard67 richard67 - change - 9 May 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-05-09 12:17:33
Closed_By richard67
Labels Added: ?
avatar richard67
richard67 - comment - 9 May 2021

Thanks all!

Add a Comment

Login with GitHub to post a comment