? Success

User tests: Successful: Unsuccessful:

avatar dgt41
dgt41
29 Nov 2014

Restore mootools in administrator/com_installer/views/install/tmpl/default.php

Ensures B/C (#5239)

avatar dgt41 dgt41 - open - 29 Nov 2014
avatar jissues-bot jissues-bot - change - 29 Nov 2014
Labels Added: ?
avatar smanzi
smanzi - comment - 29 Nov 2014

@test success
With this patch applied I could complete the installation of pkg_kunena_v3.0.6_2014-07-28.zip without issues.

avatar smanzi smanzi - test_item - 29 Nov 2014 - Tested successfully
avatar 810
810 - comment - 29 Nov 2014

+1

avatar mbabker
mbabker - comment - 29 Nov 2014

To whomever commits this, I'd also add a comment above the MooTools call and make sure Hathor is loading it in this view. A comment like this should do:

// MooTools is loaded for B/C for extensions generating JavaScript in their install scripts, this call will be removed at 4.0

avatar smanzi
smanzi - comment - 29 Nov 2014

@mbabker 4.0 or 3.5?

avatar 810
810 - comment - 29 Nov 2014

i tested Hathor, and kunena can install. So RTM

avatar mbabker
mbabker - comment - 29 Nov 2014

4.0, our B/C policy does not allow us to break B/C at any release but a major (X.0) release.

avatar smanzi
smanzi - comment - 29 Nov 2014

well... ok, although this is a very special case (we are not breaking the API), but again... :christmas_tree:

avatar mbabker
mbabker - comment - 29 Nov 2014

It technically isn't an API but it can be argued it's a B/C breaking change. It's one of those gray areas that you could argue every possible decision for weeks on end and get nowhere, at least this way a decision is being made and is clearly stated somewhere (even if it's buried in a code comment right now).

avatar smanzi
smanzi - comment - 29 Nov 2014

actually if we commit to keep it till 4.0, the comment is probably redundant: lots of things will break at 4.0, I suppose...

avatar mbabker
mbabker - comment - 29 Nov 2014

It's about the same as our @deprecated 4.0 tags in doc blocks. Redundant? Probably. Clear that we're pulling it out? Absolutely.

avatar dgt41
dgt41 - comment - 29 Nov 2014

And also preventing that nobody will try to remove it before 4.0

avatar smanzi
smanzi - comment - 29 Nov 2014

Point taken!

avatar Bakual
Bakual - comment - 29 Nov 2014

I can live with it if that's the only place MooTools will still be loaded.
However it should be clear that this doesn't become a precedence for other places. Our goal since 3.0 is to get rid of MooTools. By now, every developer should know that it will be removed eventually and that they can't trust on it being present.

avatar Bakual Bakual - change - 29 Nov 2014
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2014-11-29 20:45:48
avatar Bakual Bakual - close - 29 Nov 2014
avatar infograf768
infograf768 - comment - 30 Nov 2014

Don't we need it also for discover and update?

avatar Bakual
Bakual - comment - 30 Nov 2014

Try updating/discovering Kunena and you know :)

avatar mahagr
mahagr - comment - 1 Dec 2014

Thanks guys. We will look into removing last Mootools bits from our installer.

This is kind of special case as Kunena uses enqueued message to inject our installer code into the page and we have no control on what JS has been loaded to the page. :) In the future it would be great if installers had a bit more freedom to run their AJAX installers as Kunena upgrades can take minutes of time if there are millions of posts in the forum. So basically we needed to split those SQL updates into smaller 15-25 second long pieces and to use AJAX as many times as needed.

Add a Comment

Login with GitHub to post a comment