#687 Don't default to reloading routes in production - Hobo

Type	To find
responsible:me	tickets assigned to you
tagged:"@high"	tickets tagged @high
milestone:next	tickets in the upcoming milestone
state:invalid	tickets with the state invalid
created:"last week"	tickets created last week
sort:number, importance, updated	tickets sorted by #, importance or updated
Combine keywords for powerful searching.
Use advanced searching »

#687 ✓resolved

Don't default to reloading routes in production

Reported by Betelgeuse | April 7th, 2010 @ 10:09 AM

Currently the default hobo initializer is:

betelgeuse@mac ~/hobo $ cat hobo/rails_generators/hobo/templates/initializer.rb
Hobo::ModelRouter.reload_routes_on_every_request = true
# Dryml.precompile_taglibs if File.basename($0) != "rake" && Rails.env.production?

I think we should append unless Rails.env.production? to the first line

Comments and changes to this ticket

Betelgeuse April 7th, 2010 @ 10:17 AM

I guess I should have actually tried disabling the loading. It seems to be break routes:

ActionController::ActionControllerError (Cannot redirect to nil!):
  hobo (1.0.0) lib/hobo/controller.rb:63:in `redirect_to'
  app/controllers/front_controller.rb:7:in `index'
  hobo (1.0.0) lib/hobo/controller.rb:23:in `call'
  hobo (1.0.0) lib/hobo/controller.rb:23:in `included_in_class'
  /Library/Ruby/Gems/1.8/gems/hoptoad_notifier-2.2.2/lib/hoptoad_notifier/rack.rb:27:in `call'

Matt Jones September 14th, 2011 @ 10:07 PM
- State changed from “new” to “resolved”
- Milestone order changed from “0” to “0”
This mechanism is gone completely in 1.3. It might be handy to keep the precompile_taglibs bit around someplace, though.

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.