? ? Pending

User tests: Successful: Unsuccessful:

avatar laoneo
laoneo
2 Sep 2017

Pull Request for Issue #17804.

Summary of Changes

As class names are not case sensitive in PHP, the alias support in JLoader must also be case insensitive. This pr fixes it.

Testing Instructions

In the file administrator/components/com_content/content.php change line 11 to JHTML::_('behavior.tabstate');

Expected result

No error is thrown.

Actual result

Class 'JHTML' not found error is thrown.

avatar joomla-cms-bot joomla-cms-bot - change - 2 Sep 2017
Category Libraries
avatar laoneo laoneo - open - 2 Sep 2017
avatar laoneo laoneo - change - 2 Sep 2017
Status New Pending
avatar joeforjoomla
joeforjoomla - comment - 2 Sep 2017

I have tested this item successfully

avatar joomla-cms-bot joomla-cms-bot - change - 2 Sep 2017
Category Libraries Libraries Unit Tests
avatar laoneo laoneo - change - 2 Sep 2017
Labels Added: ?
avatar laoneo
laoneo - comment - 2 Sep 2017

@joeforjoomla I guess you need to test again as I changed something in the code. Thanks for the help!

avatar joeforjoomla
joeforjoomla - comment - 2 Sep 2017

@laoneo I've tested it again just now... looks still working successfully! My pleasure to help you

avatar ggppdk ggppdk - test_item - 3 Sep 2017 - Tested successfully
avatar ggppdk
ggppdk - comment - 3 Sep 2017

I have tested this item successfully on e214d1f


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

avatar Quy
Quy - comment - 3 Sep 2017
avatar franz-wohlkoenig franz-wohlkoenig - alter_testresult - 3 Sep 2017 - joeforjoomla: Tested successfully
avatar franz-wohlkoenig franz-wohlkoenig - change - 3 Sep 2017
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 3 Sep 2017

RTC after two successful tests.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 3 Sep 2017

Thanks for Hint, @Quy

avatar mbabker mbabker - change - 3 Sep 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-09-03 16:33:23
Closed_By mbabker
Labels Added: ?
avatar mbabker mbabker - close - 3 Sep 2017
avatar mbabker mbabker - merge - 3 Sep 2017

Add a Comment

Login with GitHub to post a comment