?
avatar Fedik
Fedik
18 Nov 2017

flying-focus.js source and minified versions are different,
that why we still have that flying-focus.js error on every page.

I guess it is build failure.
Can be that other scrips/styles also has difference.

avatar Fedik Fedik - open - 18 Nov 2017
avatar joomla-cms-bot joomla-cms-bot - change - 18 Nov 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 18 Nov 2017
avatar Fedik Fedik - change - 18 Nov 2017
The description was changed
avatar Fedik Fedik - edited - 18 Nov 2017
avatar franz-wohlkoenig franz-wohlkoenig - change - 18 Nov 2017
Category JavaScript
avatar infograf768
infograf768 - comment - 18 Nov 2017

I confirm that when in debug mode I do not get the error in #18642


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

avatar infograf768 infograf768 - change - 18 Nov 2017
Build staging 4.0-dev
avatar franz-wohlkoenig franz-wohlkoenig - change - 18 Nov 2017
Status New Discussion
avatar C-Lodder
C-Lodder - comment - 19 Nov 2017

Hmm, I cannot seem to replicate this console log error. Next time you see if, please load the uncompressed JS file and report the error back.

avatar infograf768
infograf768 - comment - 19 Nov 2017

That is exactly what I have done by using debug mode

avatar Fedik
Fedik - comment - 19 Nov 2017

@C-Lodder that what we meant, uncompressed works fine, but minified are broken ?

just compare the files from the 4.0-Alfa archive, they are different version of "flying-focus" js,
same in 4.0-dev branch

avatar C-Lodder
C-Lodder - comment - 19 Nov 2017

@Fedik I've tried about 10-15 views in the backend with debug mode enabled and disabled. I still cannot see any error

avatar C-Lodder
C-Lodder - comment - 19 Nov 2017

@Fedik @infograf768 can you try this file and let me know if it works?

flying-focus.min.zip

avatar infograf768
infograf768 - comment - 19 Nov 2017

My test was done before the release of alpha and recent PRs in the 4.0-dev branch.
I can't reproduce the issue right now. Will test with this file.

avatar Fedik
Fedik - comment - 19 Nov 2017

I see,
it was fixed in c50e945 by @wilsonge while fixing another issue ?

I am closing here

avatar Fedik Fedik - change - 19 Nov 2017
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2017-11-19 11:32:28
Closed_By Fedik
avatar Fedik Fedik - close - 19 Nov 2017

Add a Comment

Login with GitHub to post a comment