Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#29660] - [4.0] Impossible to add image in editor from media manager

?
avatar PhilETaylor
PhilETaylor
16 Jun 2020

Steps to reproduce the issue

Joomla 4.0.0 beta 1
Go to edit an article
Click the CMS Content button in the WYSIWYG Editor

NOTE RIGHT HERE YOU GET YOUR FIRST console.log error message

Error: <path> attribute d: Expected number, "…4.99-12 10zM2224h-18v-12h3.934c0…" <truncated>

Select an image
Click insert image

Expected result

Image inserted

Actual result

No image inserted

Console.log Errors...

Screenshot 2020-06-17 at 00 09 18

[Error] Error: Problem parsing d="M8 20c0 0 1.838-6 12-6v6l12-8-12-8v6c-8 0-12 4.99-12 10zM2224h-18v-12h3.934c0.315-0.372 0.654-0.729 1.015-1.068 1.374-1.287 3.018-2.27 4.879-2.932h-13.827v20h26v-8.395l-4 2.667v1.728z"
	each (theme.min.js:9:2066)
	Li (theme.min.js:9:34652)
	Xi (theme.min.js:9:36488)
	(anonymous function) (theme.min.js:9:37749)
	V (theme.min.js:9:4239)
	Xi (theme.min.js:9:36322)
	(anonymous function) (theme.min.js:9:37749)
	V (theme.min.js:9:4239)
	Xi (theme.min.js:9:36322)
	(anonymous function) (theme.min.js:9:37749)
	V (theme.min.js:9:4239)
	Xi (theme.min.js:9:36322)
	(anonymous function) (theme.min.js:9:37749)
	(anonymous function) (theme.min.js:9:102570)
	(anonymous function) (theme.min.js:9:5867)
	(anonymous function) (theme.min.js:9:5936)
	ft (theme.min.js:9:5799)
	mt (theme.min.js:9:5912)
	e (theme.min.js:9:102316)
	(anonymous function) (theme.min.js:9:105323)
	run (theme.min.js:9:24417)
	(anonymous function) (theme.min.js:9:30225)
	(anonymous function) (theme.min.js:9:154454)
	Ab (theme.min.js:9:154801)
	triggerEvent (theme.min.js:9:157449)
	Go (theme.min.js:9:23940)
	Lo (theme.min.js:9:23746)
	As (theme.min.js:9:65531)
	Vs (theme.min.js:9:66050)
	Rs (theme.min.js:9:65947)
	Us (theme.min.js:9:66725)
	Ws (theme.min.js:9:66772)
	(anonymous function) (theme.min.js:9:173099)
	(anonymous function) (theme.min.js:9:173029)
	promiseReactionJob
[Error] TypeError: Joomla.getImage is not a function. (In 'Joomla.getImage(Joomla.selectedFile, 'jform_articletext')', 'Joomla.getImage' is undefined)
	onclick (index.php:360)

System information (as much as possible)

Reproduced in Safari and Google Chrome on Mac.

avatar PhilETaylor PhilETaylor - open - 16 Jun 2020
avatar joomla-cms-bot joomla-cms-bot - change - 16 Jun 2020
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 16 Jun 2020
avatar PhilETaylor PhilETaylor - change - 16 Jun 2020
The description was changed
avatar PhilETaylor PhilETaylor - edited - 16 Jun 2020
avatar infograf768 infograf768 - change - 17 Jun 2020
Labels Added: ?
avatar infograf768 infograf768 - labeled - 17 Jun 2020
avatar infograf768
infograf768 - comment - 17 Jun 2020

Confirmed. Setting as release blocker.

avatar wilsonge
wilsonge - comment - 17 Jun 2020

I think this was fixed by #29323 already as I can't reproduce this on nightly (using chrome or safari on Mac).

The original JS notice when you click on the menu item isn't fixed (although functionally doesn't cause issues) so removing release blocker tag

avatar wilsonge wilsonge - change - 17 Jun 2020
Labels Removed: ?
avatar wilsonge wilsonge - unlabeled - 17 Jun 2020
avatar PhilETaylor
PhilETaylor - comment - 17 Jun 2020

Maybe its time for another beta then if people are being asked to test the beta but most of the issues are resolved in git only.... #justsaying

avatar infograf768
infograf768 - comment - 17 Jun 2020

@wilsonge
My test are always in 4.0-dev. not beta

avatar brianteeman
brianteeman - comment - 17 Jun 2020

thats why we have nightly releases

avatar PhilETaylor
PhilETaylor - comment - 17 Jun 2020

I can replicate this in 4.0-dev @ 00a94b6 so no this is not fixed.

avatar PhilETaylor
PhilETaylor - comment - 17 Jun 2020

Correction: The Image can be added now, but the other console log error message still happens.

Screenshot 2020-06-17 at 11 22 50

avatar infograf768
infograf768 - comment - 17 Jun 2020

Screen Shot 2020-06-17 at 12 20 55

Image can be inserted btw after patching with #29675
To test, make sure btw that safari cache is cleared

avatar Quy
Quy - comment - 3 Jul 2020

@PhilETaylor Can you still replicate this? Thanks.

avatar PhilETaylor
PhilETaylor - comment - 3 Jul 2020

As said before - its now possible to insert an image - however console.log errors still occur.

Its no longer a release blocker but is still an issue.

[Error] Error: Problem parsing d="M8 20c0 0 1.838-6 12-6v6l12-8-12-8v6c-8 0-12 4.99-12 10zM2224h-18v-12h3.934c0.315-0.372 0.654-0.729 1.015-1.068 1.374-1.287 3.018-2.27 4.879-2.932h-13.827v20h26v-8.395l-4 2.667v1.728z"
	each (theme.min.js:9:2066)
	Li (theme.min.js:9:34652)
	Xi (theme.min.js:9:36488)
	(anonymous function) (theme.min.js:9:37749)
	V (theme.min.js:9:4239)
	Xi (theme.min.js:9:36322)
	(anonymous function) (theme.min.js:9:37749)
	V (theme.min.js:9:4239)
	Xi (theme.min.js:9:36322)
	(anonymous function) (theme.min.js:9:37749)
	V (theme.min.js:9:4239)
	Xi (theme.min.js:9:36322)
	(anonymous function) (theme.min.js:9:37749)
	(anonymous function) (theme.min.js:9:102570)
	(anonymous function) (theme.min.js:9:5867)
	(anonymous function) (theme.min.js:9:5936)
	ft (theme.min.js:9:5799)
	mt (theme.min.js:9:5912)
	e (theme.min.js:9:102316)
	(anonymous function) (theme.min.js:9:105323)
	run (theme.min.js:9:24417)
	(anonymous function) (theme.min.js:9:30225)
	(anonymous function) (theme.min.js:9:154454)
	Ab (theme.min.js:9:154801)
	triggerEvent (theme.min.js:9:157449)
	Go (theme.min.js:9:23940)
	Lo (theme.min.js:9:23746)
	As (theme.min.js:9:65531)
	Vs (theme.min.js:9:66050)
	Rs (theme.min.js:9:65947)
	Us (theme.min.js:9:66725)
	Ws (theme.min.js:9:66772)
	(anonymous function) (theme.min.js:9:173099)
	(anonymous function) (theme.min.js:9:173029)
	promiseReactionJob
avatar Quy
Quy - comment - 3 Jul 2020

Reproducible in Chrome but not Firefox.

avatar PhilETaylor
PhilETaylor - comment - 3 Jul 2020

Dont panic, its only been broken for 15 months :) I have the fix :-)

avatar PhilETaylor
PhilETaylor - comment - 3 Jul 2020

PR #29935

avatar alikon
alikon - comment - 3 Jul 2020

please test #29935

avatar alikon alikon - change - 3 Jul 2020
Status New ⇒ Closed
Closed_Date 0000-00-00 00:00:00 ⇒ 2020-07-03 16:23:36
Closed_By ⇒ alikon
avatar alikon alikon - close - 3 Jul 2020

Add a Comment

Login with GitHub to post a comment