> > 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