changing application.rb to application_controller.rb

4 views
Skip to first unread message

Jay

unread,
Dec 19, 2007, 7:34:29 PM12/19/07
to Ruby on Rails: Core
I submitted a patch that changes application.rb to
application_controller.rb. The patch fixes the problem where a
controller referenced before 'require "application"' will cause an
exception.

The patch causes no new test failures and I've also manually tested it
by creating a sample application and running it, running script/
console and running script/performance/request.

ticket: http://dev.rubyonrails.org/ticket/10570

Mislav Marohnić

unread,
Dec 20, 2007, 4:32:58 AM12/20/07
to rubyonra...@googlegroups.com
+1 for the idea if backwards compatibility is preserved

This has been thrown around before (proof: http://project.ioni.st/post/1215#irc_quote_1215)

Tammer Saleh

unread,
Dec 20, 2007, 7:51:00 AM12/20/07
to Ruby on Rails: Core
+1

Nicolas Pouillard

unread,
Dec 20, 2007, 7:52:23 AM12/20/07
to Ruby on Rails: Core
+1

Excerpts from Tammer Saleh's message of Thu Dec 20 13:51:00 +0100 2007:

--
Nicolas Pouillard aka Ertai

Vladislav Gorodetskiy

unread,
Feb 6, 2008, 2:49:58 AM2/6/08
to Ruby on Rails: Core
+1
Reply all
Reply to author
Forward
0 new messages