Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: Catalyst: Dev
[Catalyst-Plugin-FillInForm PATCH] documentation fix - short-circuit view rather than fillform
 

Index | Next | Previous | View Flat


peter at pcc

Mar 9, 2009, 2:24 PM


Views: 753
Permalink
[Catalyst-Plugin-FillInForm PATCH] documentation fix - short-circuit view rather than fillform

---
lib/Catalyst/Plugin/FillInForm.pm | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/Catalyst/Plugin/FillInForm.pm b/lib/Catalyst/Plugin/FillInForm.pm
index 21405b3..5f6c96e 100644
--- a/lib/Catalyst/Plugin/FillInForm.pm
+++ b/lib/Catalyst/Plugin/FillInForm.pm
@@ -130,8 +130,8 @@ Then you'll need to change it to something like this:

sub end : Private {
my ($self, $c) = @_;
- $c->forward('render');
- $c->fillform($c->req->params) unless $c->res->output;
+ $c->forward('render') unless $c->res->output;
+ $c->fillform($c->req->params);
}

sub render : ActionClass('RenderView') { }
--
1.5.6.5


--
Peter
Attachments: signature.asc (0.19 KB)

Subject User Time
[Catalyst-Plugin-FillInForm PATCH] documentation fix - short-circuit view rather than fillform peter at pcc Mar 9, 2009, 2:24 PM
    Re: [Catalyst-Plugin-FillInForm PATCH] documentation fix - short-circuit view rather than fillform pagaltzis at gmx Mar 9, 2009, 2:40 PM
        Re: Re: [Catalyst-Plugin-FillInForm PATCH] documentation fix - short-circuit view rather than fillform peter at pcc Mar 10, 2009, 7:30 AM

  Index | Next | Previous | View Flat
 
 


Interested in having your list archived? Contact lists@gossamer-threads.com
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.