Pending

User tests: Successful: Unsuccessful:

avatar phproberto
phproberto
21 Mar 2013

Fix already merged into the platform:
joomla/joomla-platform#1842

avatar phproberto phproberto - open - 21 Mar 2013
avatar mahagr
mahagr - comment - 21 Mar 2013

+1 for this one. But aren't installer going to be moved into CMS?

avatar phproberto
phproberto - comment - 21 Mar 2013

It's already moved and this patch is applied after

avatar mahagr
mahagr - comment - 21 Mar 2013

I meant if the pull request should have been done against CMS repository instead of platform.

avatar phproberto
phproberto - comment - 21 Mar 2013

This PR is against the CMS :P

In the platform was already merged

avatar mahagr
mahagr - comment - 21 Mar 2013

Oops. I mixed the pull requests. :) Sorry.

We're currently refactoring the installer, if you're interested to look into it, please check https://github.com/mbabker/joomla-cms/commits/installAbstract

avatar nicksavov
nicksavov - comment - 22 Mar 2013

Thanks for coding this, Roberto!

For anyone interested, here's the tracker item on JoomlaCode:
http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=30205

avatar dextercowley
dextercowley - comment - 29 Mar 2013

Fixed in master. Thanks!

Add a Comment

Login with GitHub to post a comment