? Pending

User tests: Successful: Unsuccessful:

avatar dgrammatiko
dgrammatiko
16 Aug 2020

Pull Request supplementary for Issue #30384 .

Summary of Changes

This PR ensures that all the render() have the appropriate params (in particular templateInherits that acts as a switch for child templates)

Testing Instructions

Error pages render correctly (even for child templates, the previous PR was covering only the legacy mode)

Actual result BEFORE applying this Pull Request

Expected result AFTER applying this Pull Request

Documentation Changes Required

@wilsonge

avatar dgrammatiko dgrammatiko - open - 16 Aug 2020
avatar dgrammatiko dgrammatiko - change - 16 Aug 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 16 Aug 2020
Category Libraries
avatar richard67
richard67 - comment - 17 Aug 2020

API tests failing in Drone for this PR. Not sure, but it could be related to the PR. See https://ci.joomla.org/joomla/joomla-cms/34749/1/30

<!-- Argument 1 passed to Joomla\CMS\WebAsset\WebAssetRegistry::addTemplateRegistryFile() must be of the type string, null given, called in /tests/www/test-install/libraries/src/Error/Renderer/HtmlRenderer.php on line 55 (500 Whoops, looks like something went wrong.) -->
avatar dgrammatiko dgrammatiko - change - 17 Aug 2020
Labels Added: ?
avatar dgrammatiko
dgrammatiko - comment - 17 Aug 2020

@richard67 if I got this right then this was a problem in the API Application and 15b510e should fix it

avatar joomla-cms-bot joomla-cms-bot - change - 18 Aug 2020
Category Libraries Installation Libraries
avatar wilsonge wilsonge - change - 4 Sep 2020
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-09-04 23:04:45
Closed_By wilsonge
avatar wilsonge wilsonge - close - 4 Sep 2020
avatar wilsonge wilsonge - merge - 4 Sep 2020
avatar wilsonge
wilsonge - comment - 4 Sep 2020

Thanks!

Add a Comment

Login with GitHub to post a comment