> > I'm not suggesting we solve the redirecting back to self issue; that
> > is a whole different can of worms. My intial patch simply added an
> > exception so if someone called redirect_to :back with a nil
> > request["HTTP_REFERER"] it would display a helpful error instead of
> > the other error (I don't remember at this point, my code is patched)
> > which didn't really say what the problem was. I agree with David that
> > anything more complex should be left out of core, but do like good
> > error messages.

Sure, I'll buy an exception patch to guard against nil.
--
David Heinemeier Hansson
http://www.loudthinking.com -- Broadcasting Brain
http://www.basecamphq.com   -- Online project management
http://www.backpackit.com   -- Personal information manager
http://www.rubyonrails.com  -- Web-application framework
_______________________________________________
Rails-core mailing list
Rails-core@lists.rubyonrails.org
http://lists.rubyonrails.org/mailman/listinfo/rails-core

Reply via email to