[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