[Catalyst] Patch for Catalyst::View::Mason
Matt S Trout
dbix-class at trout.me.uk
Tue Mar 11 02:52:10 GMT 2008
On Thu, Mar 06, 2008 at 11:11:51PM +0100, David Jack Wange Olrik wrote:
> Hi Florian,
>
> Here is a small patch for Catalyst::View::Mason.
>
> When setting $c->stash->{template}, template_extension was not added
> to the component path. In my opinion this should always be added, so
> one can easily change the view without having to provide extra logic
> that sets the template extension according to the current view.
Tough. That's not how $c->stash->{template} works.
We can't break compat for everybody else because you want it to do something
different.
On the other hand, you do have a point.
So why not do it as a patch that allows a config option?
--
Matt S Trout Need help with your Catalyst or DBIx::Class project?
Technical Director http://www.shadowcat.co.uk/catalyst/
Shadowcat Systems Ltd. Want a managed development or deployment platform?
http://chainsawblues.vox.com/ http://www.shadowcat.co.uk/servers/
More information about the Catalyst
mailing list