? Pending

User tests: Successful: Unsuccessful:

avatar zero-24
zero-24
30 Aug 2019

Pull Request for Issue #26087

Summary of Changes

Make sure passing an boolean does silently work like before cc @HLeithner #25225

Testing Instructions

echo JRoute::_('index.php', false, true)."\n";  //returns http:
echo JRoute::_('index.php', false, false)."\n"; //returns relative
echo JRoute::_('index.php', false, 1)."\n"; //return https:
echo JRoute::_('index.php', false, '1')."\n"; //returns https:
echo JRoute::_('index.php', false, JROUTE::TLS_DISABLE)."\n"; debug_zval_dump(JROUTE::TLS_DISABLE); //returns http: 
echo JRoute::_('index.php', false, JROUTE::TLS_FORCE)."\n"; debug_zval_dump(JROUTE::TLS_FORCE);  //returns https:
echo JRoute::_('index.php', false, JROUTE::TLS_IGNORE)."\n"; debug_zval_dump(JROUTE::TLS_IGNORE);  //returns relative

Expected result

echo JRoute::_('index.php', false, true)."\n";  //returns https:
echo JRoute::_('index.php', false, -1)."\n"; //return http:

Actual result

echo JRoute::_('index.php', false, true)."\n";  //returns http:
echo JRoute::_('index.php', false, -1)."\n"; //return http:

Documentation Changes Required

none.

avatar zero-24 zero-24 - open - 30 Aug 2019
avatar zero-24 zero-24 - change - 30 Aug 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 30 Aug 2019
Category Libraries
avatar zero-24 zero-24 - change - 30 Aug 2019
The description was changed
avatar zero-24 zero-24 - edited - 30 Aug 2019
avatar zero-24 zero-24 - change - 3 Sep 2019
Labels Added: ?
avatar wilsonge
wilsonge - comment - 13 Sep 2019

@HLeithner you need to review this as it's a 3.x regression

avatar HLeithner HLeithner - change - 17 Sep 2019
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-09-17 13:02:34
Closed_By HLeithner
avatar HLeithner HLeithner - close - 17 Sep 2019
avatar HLeithner HLeithner - merge - 17 Sep 2019
avatar HLeithner
HLeithner - comment - 17 Sep 2019

Thanks for fix this b/c break.

Add a Comment

Login with GitHub to post a comment