? Error

User tests: Successful: Unsuccessful:

avatar mbabker
mbabker
5 Jun 2015

In the install app, the code to boot the session is triggered twice. This PR fixes that.

Testing Instructions

So, the easiest way I have to confirm this is fixed is putting a echo 'loadSession started'; line at the beginning of InstallationApplicationWeb::loadSession() as it isn't something you'll notice in normal code execution. Pre-patch, the message should show twice; post-patch only once. Otherwise, this is basically just make sure the CMS installs correctly.

Or, you can read through the code and see how the method is called twice.

avatar mbabker mbabker - open - 5 Jun 2015
avatar wilsonge wilsonge - change - 5 Jun 2015
Status New Closed
Closed_Date 0000-00-00 00:00:00 2015-06-05 23:16:44
Closed_By wilsonge
avatar wilsonge wilsonge - close - 5 Jun 2015
avatar joomla-cms-bot joomla-cms-bot - close - 5 Jun 2015
avatar wilsonge wilsonge - reference | 6d94152 - 5 Jun 15
avatar wilsonge wilsonge - merge - 5 Jun 2015
avatar wilsonge wilsonge - close - 5 Jun 2015
avatar joomla-cms-bot joomla-cms-bot - change - 5 Jun 2015
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 5 Jun 2015
Labels Added: ?
avatar wilsonge
wilsonge - comment - 5 Jun 2015

We are stupid sometimes. Merged

avatar wilsonge wilsonge - change - 5 Jun 2015
Milestone Added:
avatar mbabker mbabker - head_ref_deleted - 5 Jun 2015

Add a Comment

Login with GitHub to post a comment