? Success

User tests: Successful: Unsuccessful:

avatar wilsonge
wilsonge
2 May 2014

Note this requires testing of #3548 before this can be merged for changes in the libraries folder of Joomla to change the method to call statically for JApplication(Site/Administrator)::getRouter() instead of non-statically (it's not a b/c break as the method is already static - we're not calling it as such however)

avatar wilsonge wilsonge - open - 2 May 2014
avatar wilsonge wilsonge - change - 2 May 2014
Labels Added: ? ?
avatar infograf768
infograf768 - comment - 5 May 2014

Further explanation:
This PR contains #3548
Therefore #3548 is only for testing

avatar wilsonge wilsonge - change - 5 May 2014
The description was changed
Description <p>Note this requires testing of <a href="https://github.com/joomla/joomla-cms/pull/3548" class="issue-link" title="[#33688] JApplicationCms::getRouter is a static method">#3548</a></p> <p>Note this requires testing of <a href="https://github.com/joomla/joomla-cms/pull/3548" class="issue-link" title="[#33688] JApplicationCms::getRouter is a static method">#3548</a> before this can be merged for changes in the libraries folder of Joomla to change the method to call statically for JApplication(Site/Administrator)::getRouter() instead of non-statically (it's not a b/c break as the method is already static - we're not calling it as such however)</p>
avatar dbhurley
dbhurley - comment - 5 May 2014

:+1: Looks good, lets fix those tests! :)

avatar Bakual Bakual - change - 5 May 2014
Status New Closed
Closed_Date 0000-00-00 00:00:00 2014-05-05 17:52:46
avatar Bakual Bakual - close - 5 May 2014
avatar Bakual Bakual - close - 5 May 2014
avatar wilsonge wilsonge - head_ref_deleted - 5 May 2014
avatar Bakual Bakual - reference | 5a6ff0b - 12 May 14

Add a Comment

Login with GitHub to post a comment