? ? Pending

User tests: Successful: Unsuccessful:

avatar wilsonge
wilsonge
28 Jun 2017

PR for issue #16698

This PR hopefully fixes the unit tests which are failing because they try to get a URI instance (either directly in Joomla\Cms\View\HtmlView::__construct() or in the case of the form fields, where they are getting URLs when including javascript files.

In 4.0 we now have stricter validation on URLs which is why we haven't seen this issue until now. Although there is no real reason to not port this back to staging as it is presumably an issue there - just one we ignore.

avatar wilsonge wilsonge - open - 28 Jun 2017
avatar wilsonge wilsonge - change - 28 Jun 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 28 Jun 2017
Category Unit Tests
avatar wilsonge wilsonge - change - 28 Jun 2017
The description was changed
avatar wilsonge wilsonge - edited - 28 Jun 2017
avatar zero-24
zero-24 - comment - 29 Jun 2017

Looks like we have still the same error. On appveyor :(

avatar wilsonge
wilsonge - comment - 29 Jun 2017

Partly? The 15 JViewLegacy tests are now passing but the two form fields arent. It's weird, I'll keep digging

avatar wilsonge wilsonge - change - 29 Jun 2017
Labels Added: ? ?
avatar wilsonge
wilsonge - comment - 29 Jun 2017

Well I mean this now passes. If anyone wants to explain why REQUEST_URI works for JViewLegacy but SCRIPT_NAME is required for the form fields be my guest :/

avatar wilsonge wilsonge - change - 29 Jun 2017
The description was changed
avatar wilsonge wilsonge - edited - 29 Jun 2017
avatar wilsonge wilsonge - change - 29 Jun 2017
The description was changed
avatar wilsonge wilsonge - edited - 29 Jun 2017
avatar wilsonge wilsonge - change - 29 Jun 2017
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-06-29 10:37:14
Closed_By wilsonge
avatar wilsonge wilsonge - close - 29 Jun 2017
avatar wilsonge wilsonge - merge - 29 Jun 2017

Add a Comment

Login with GitHub to post a comment