? ? Success

User tests: Successful: Unsuccessful:

avatar wilsonge
wilsonge
28 Jul 2016

Pull Request for Issue #10835 .

Summary of Changes

Updates phpmailer, joomla registry, symfony polyfill (php 5.6), symfony yaml to their latest versions

Testing Instructions

Registry is used everywhere. So navigate around and check for errors. Yaml isn't used by core. If possible test in PHP < 5.6 to test the polyfill and check emails still send for phpmailer.

avatar wilsonge wilsonge - open - 28 Jul 2016
avatar wilsonge wilsonge - change - 28 Jul 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 28 Jul 2016
Category External Library Libraries
avatar joomla-cms-bot joomla-cms-bot - change - 28 Jul 2016
Labels Added: ? ?
avatar brianteeman brianteeman - test_item - 28 Jul 2016 - Tested successfully
avatar brianteeman
brianteeman - comment - 28 Jul 2016

I have tested this item successfully on e716894

Seems ok - sendtestmail still works. No errors navigating around. Not sure what polyfil is used for


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

avatar wilsonge
wilsonge - comment - 28 Jul 2016

The 5.6 polyfill is used to make the hash_equals function available on PHP 5.3, 5.4 and 5.5. So check logging in with passwords still works properly on those php versions

avatar brianteeman brianteeman - test_item - 28 Jul 2016 - Tested unsuccessfully
avatar brianteeman
brianteeman - comment - 28 Jul 2016

I have tested this item ? unsuccessfully on e716894

OK something went wrong.
Admin is fine but front end fails to load with this in the logs

Call to a member function getDefault() on a non-object in libraries/cms/router/site.php on line 266


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

avatar brianteeman
brianteeman - comment - 28 Jul 2016

Under php7 I get

Error displaying the error page: Call to a member function getActive() on null: Call to a member function getDefault() on null


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

avatar wilsonge
wilsonge - comment - 30 Jul 2016

Woohoo so Joomla's been shipping sample data with invalid JSON since 2012 - we just never had error handling :P Fixed for existing data sets and dealt with the invalid JSON

avatar brianteeman
brianteeman - comment - 30 Jul 2016

The error on the homepage has gone now

avatar brianteeman brianteeman - test_item - 30 Jul 2016 - Tested successfully
avatar brianteeman
brianteeman - comment - 30 Jul 2016

I have tested this item successfully on 4c258c5

Tested creating users and logging in under multiple php versions 5.4, 5.5, 5.6, 7
Tested sending mail using the testmail function in global config


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

avatar viocassel viocassel - test_item - 3 Aug 2016 - Tested successfully
avatar viocassel
viocassel - comment - 3 Aug 2016

I have tested this item successfully on 4c258c5


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

avatar brianteeman brianteeman - change - 3 Aug 2016
Status Pending Ready to Commit
Labels
avatar brianteeman
brianteeman - comment - 3 Aug 2016

RTC


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

avatar joomla-cms-bot joomla-cms-bot - change - 3 Aug 2016
Labels Added: ?
avatar mbabker
mbabker - comment - 8 Aug 2016

@wilsonge What version of Composer do you have? Current Composer versions are generating a autoload_static.php file which isn't added in this PR.

avatar wilsonge
wilsonge - comment - 8 Aug 2016
Georges-MacBook-Pro:~ george$ composer --version
Composer version 1.0.2 2016-04-21 12:30:18

Just updated to 1.2.0

avatar mbabker
mbabker - comment - 8 Aug 2016

About time! ?

avatar rdeutz rdeutz - change - 13 Aug 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-08-13 17:37:36
Closed_By rdeutz
avatar brianteeman brianteeman - change - 13 Aug 2016
Labels Removed: ?

Add a Comment

Login with GitHub to post a comment