? Failure
Referenced as Related to: # 5172

User tests: Successful: Unsuccessful:

avatar Hackwar
Hackwar
23 Nov 2014

This fixes #5172. Mea culpa. I missed that & in the Installation router. ?

Testing instructions:
Set PHP error reporting to maximum, try to install Joomla. You will get an error in your Ajax response and the installation does not proceed.
Apply patch and see that this does not happen anymore.

avatar Hackwar Hackwar - open - 23 Nov 2014
avatar jissues-bot jissues-bot - change - 23 Nov 2014
Labels Added: ?
avatar zero-24 zero-24 - change - 23 Nov 2014
Category Installation SEF
avatar zero-24 zero-24 - change - 23 Nov 2014
Priority Medium Critical
avatar zero-24 zero-24 - change - 23 Nov 2014
Easy No Yes
avatar Hackwar
Hackwar - comment - 23 Nov 2014

Funny thing... That bug was in Joomla since August 2013. Phew, not my fault. Dodged that bullet. :wink:

avatar mbabker
mbabker - comment - 23 Nov 2014

Merged on review

avatar mbabker mbabker - close - 23 Nov 2014
avatar mbabker mbabker - change - 23 Nov 2014
Status Pending Fixed in Code Base
avatar mbabker mbabker - close - 23 Nov 2014
avatar mbabker mbabker - change - 23 Nov 2014
Closed_Date 0000-00-00 00:00:00 2014-11-23 15:16:43
avatar infograf768
infograf768 - comment - 25 Nov 2014

[25-Nov-2014 09:47:40 UTC] PHP Strict standards: Declaration of JRouterInstallation::build() should be compatible with JRouter::build($url) in /Applications/MAMP/htdocs/testnew/trunkgitnew/installation/application/router.php on line 47

avatar infograf768
infograf768 - comment - 25 Nov 2014

Looks like you corrected parse, but broke build

avatar infograf768 infograf768 - reference | 15c5d65 - 25 Nov 14
avatar infograf768
infograf768 - comment - 25 Nov 2014

Please test
#5198

avatar Bakual Bakual - reference | 70eb1c7 - 25 Nov 14
avatar pmorrisarctg pmorrisarctg - reference | bafc49a - 26 Nov 14
avatar Hackwar Hackwar - head_ref_deleted - 6 Jan 2016

Add a Comment

Login with GitHub to post a comment