Success

User tests: Successful: Unsuccessful:

avatar mbabker
mbabker
18 Jun 2016

Summary of Changes

Converts the view/rendering API to use the Framework v2 renderer and view packages and upgrades the filp/whoops package to the current version. Converts the renderer configuration logic to a service provider and moved out of the base controller.

Testing Instructions

Double check everything still renders as expected.

What Doesn't Work Anymore?

The pure PHP renderer doesn't work anymore. Considering those templates are long abandoned I don't see the issue with that. Mustache and Twig integrations should still be fine.

avatar mbabker mbabker - open - 18 Jun 2016
avatar elkuku
elkuku - comment - 19 Jun 2016

I tried to triple check and everything still seems to render as it should.

One small exception: The clean cache command does not work anymore because of a "3pd" silently requiring the joomla/filesystem package that has been removed by this PR.
Updating the elkuku/g11n package will solve this issue ?

Since it removes > 1k lines of code I can't resist to push that merge button ?

avatar elkuku elkuku - change - 19 Jun 2016
Status New Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-06-19 15:38:28
Closed_By elkuku
avatar elkuku elkuku - close - 19 Jun 2016
avatar elkuku elkuku - merge - 19 Jun 2016
avatar elkuku elkuku - reference | 1f608ae - 19 Jun 16
avatar elkuku elkuku - merge - 19 Jun 2016
avatar elkuku elkuku - close - 19 Jun 2016
avatar elkuku elkuku - head_ref_deleted - 19 Jun 2016
avatar elkuku
elkuku - comment - 19 Jun 2016

The pure PHP renderer doesn't work anymore.

No issue with that ?

avatar mbabker
mbabker - comment - 19 Jun 2016

because of a "3pd" silently requiring the joomla/filesystem package that has been removed by this PR

This is why you declare your dependencies ?

Add a Comment

Login with GitHub to post a comment