config

[heartcombo]

config

[marcandre]

Actually, ||= would do since the default is false, no need to check for nil especially.

[josevalim]

If you change this value after Rails is initialized, then they won't propagate. I am not really a fan of depending on Rails.application.config.responders, but here is what we could do:

1. Move the configuration options to Responders, so we would have Responders.namespace_lookup
2. Make Rails.application.config.responders = Responders

If you want to change that, a patch is welcome. :) Thanks a lot for your contributions!

[nashby]

@josevalim is it still make sense? If so could you explain a bit how it'll help with current issue? :) Not sure I understood it right.

[carlosantoniodasilva]

@nashby by doing Rails.application.config.responders = Responders, we'll be able to use Rails.application.config.responders.namespace_lookup = true in an initializer as @marcandre expects, because the end result would be the same as setting Responders.namespace_lookup = true.

Right now we use an initializer hook to do that, which means the configuration will be already propagated at the time Rails requires config/initializers, so setting the config there won't work.

@josevalim it's just going to be a bit weird to use Responders.namespace_lookup instead of Responders::FlashResponder.namespace_lookup since the configs are only related to the flash one. A quick way of doing less changes is setting config.responders to the FlashResponder, since it's the only one that actually have configs :D. But, I think we can live with that anyway.