Yeah that, redirect really wasn't ready and shouldn't have been in
the patch.
if a raise'd version is ever left in it should be with a "!"
def redirect!(url)
...
end
since it would stop execution, and that is not how you always want it.
everyone get your +1's on #188 ;)
Regards
Farms.
Design & Dev
Oxygen.
http://www.oxdi.eu
On 21 Sep 2007, at 11:46, ry dahl wrote:
I love Chris' controller exceptions rewrite, but I don't think
redirects should be handled with exceptions. It used to be very
simple:
def redirect(url)
MERB_LOGGER.info("Redirecting to: #{url}")
set_status(302)
headers.merge!({'Location'=> url})
return ''
end
On 9/21/07, Adam Roth <[EMAIL PROTECTED]> wrote:
I had redirects working just the other day... before I upgraded to
the
latest release. Can somebody please chime in as to why this simple
redirect
is throwing an error?
Controller:
-----------------------
def do
redirect "http://www.ebay.com";
end
Error Output:
-------------------------
Internal server error 500
uninitialized constant
Merb::ControllerMixin::MovedTemporarily
in /lib/merb/mixins/controller.rb:
def redirect( url )
MERB_LOGGER.info("Redirecting to: #{url}")
raise MovedTemporarily, url
end
Thanks
Adam
_______________________________________________
Merb-devel mailing list
[email protected]
http://rubyforge.org/mailman/listinfo/merb-devel
_______________________________________________
Merb-devel mailing list
[email protected]
http://rubyforge.org/mailman/listinfo/merb-devel
_______________________________________________
Merb-devel mailing list
[email protected]
http://rubyforge.org/mailman/listinfo/merb-devel
