? ? Pending

User tests: Successful: Unsuccessful:

avatar dgrammatiko
dgrammatiko
23 Oct 2021

Pull Request alternative to #35878

Summary of Changes

Load either minified or non minified language files depending on the JDEBUG constant (same as the rest scripts/styles in the core)

Testing Instructions

Same as #35878

Actual result BEFORE applying this Pull Request

Broken

Expected result AFTER applying this Pull Request

Fixed
Screenshot 2021-10-23 at 20 30 22

Screenshot 2021-10-23 at 20 33 31

Documentation Changes Required

No bug fix

@brianteeman

avatar dgrammatiko dgrammatiko - open - 23 Oct 2021
avatar dgrammatiko dgrammatiko - change - 23 Oct 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 23 Oct 2021
Category Front End Plugins
avatar dgrammatiko dgrammatiko - change - 24 Oct 2021
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2021-10-24 10:25:15
Closed_By dgrammatiko
Labels Added: ?
avatar dgrammatiko dgrammatiko - close - 24 Oct 2021
avatar dgrammatiko dgrammatiko - change - 25 Oct 2021
Status Closed New
Closed_Date 2021-10-24 10:25:15
Closed_By dgrammatiko
avatar dgrammatiko dgrammatiko - change - 25 Oct 2021
Status New Pending
avatar dgrammatiko dgrammatiko - reopen - 25 Oct 2021
avatar brianteeman
brianteeman - comment - 26 Oct 2021

similar problem with the editor.min.css

avatar dgrammatiko
dgrammatiko - comment - 26 Oct 2021

similar problem with the editor.min.css

There's a PR for the editor.css that needs some refactoring to handle also the .min.css case: #31785

avatar brianteeman
brianteeman - comment - 26 Oct 2021

yes but this pr is for 4.0 and that pr is for 4.whenever

avatar chmst
chmst - comment - 23 Nov 2021

If testing instructions are the same as for #35878, the PR fails. Both min.js and .js are present after npm.

avatar dgrammatiko
dgrammatiko - comment - 23 Nov 2021

@chmst the 3rd option from #35878 is not valid with this pr, we do have both min and unminified files. I think you have to check if the list of languages doesn’t have any files with extension and if you peek one then check that is correctly loaded. The images I shared in the description should help. Oh and enabling debug should load the non mini field language script

avatar laoneo
laoneo - comment - 10 Jan 2022

I have tested this item successfully on 8180880


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

avatar laoneo laoneo - test_item - 10 Jan 2022 - Tested successfully
avatar Quy
Quy - comment - 12 Jan 2022

I am getting the following when in edit mode of an article. Please update your branch to rebuilt the package that I can test instead with patch tester. Thanks.

Notice: Trying to get property '0' of non-object in \plugins\editors\tinymce\tinymce.php on line 200

Notice: Trying to get property '2' of non-object in \plugins\editors\tinymce\tinymce.php on line 200
avatar dgrammatiko
dgrammatiko - comment - 12 Jan 2022

@Quy should be synced now

avatar Quy
Quy - comment - 12 Jan 2022

I have tested this item successfully on 8180880


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

avatar Quy Quy - test_item - 12 Jan 2022 - Tested successfully
avatar Quy Quy - change - 12 Jan 2022
Status Pending Ready to Commit
avatar Quy
Quy - comment - 12 Jan 2022

RTC


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

avatar bembelimen bembelimen - change - 21 Jan 2022
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2022-01-21 13:16:29
Closed_By bembelimen
Labels Added: ?
avatar bembelimen bembelimen - close - 21 Jan 2022
avatar bembelimen bembelimen - merge - 21 Jan 2022
avatar bembelimen
bembelimen - comment - 21 Jan 2022

Thx

avatar dgrammatiko
dgrammatiko - comment - 21 Jan 2022

Thank you, Ben, just be careful with this one when you push it to 4.1 (in 4.1 the tinyMCE is refactored so you might have to do a bit of copy pasting)

Add a Comment

Login with GitHub to post a comment