[Catalyst-dev] Re: [Catalyst-Plugin-FillInForm PATCH] documentation fix - short-circuit view rather than fillform

[Peter Collingbourne]

On Mon, Mar 09, 2009 at 10:40:45PM +0100, Aristotle Pagaltzis wrote:
> * Peter Collingbourne <peter at pcc.me.uk> [2009-03-09 22:25]:
> > -        $c->forward('render');
> > -        $c->fillform($c->req->params) unless $c->res->output;
> > +        $c->forward('render') unless $c->res->output;
> > +        $c->fillform($c->req->params);
> 
> Is there a reason you used two separate conditionals?

There is only one conditional -- the patch removes the conditional on
fillform and adds it to forward.

Thanks,
-- 
Peter