ChiliProject is not maintained anymore. Please be advised that there will be no more updates.

We do not recommend that you setup new ChiliProject instances and we urge all existing users to migrate their data to a maintained system, e.g. Redmine. We will provide a migration script later. In the meantime, you can use the instructions by Christian Daehn.

Forums » Develop »

[Decided] - Add helpers :all to ApplicationController?

Added by Eric Davis at 2011-02-27 10:29 pm

Any reason we shouldn't add all helpers to ApplicationController?

If we do, then we won't have to worry about views getting loaded from other controllers and missing helper methods (very common bugs). It would also let plugins add a helper automatically and not have to patch ApplicationController to use it in existing controllers.

Eric Davis


Replies (3)

RE: Add helpers :all to ApplicationController? - Added by Felix Schäfer at 2011-03-07 08:01 pm

Eric Davis wrote:

Any reason we shouldn't add all helpers to ApplicationController?

Well, I don't know the whole story, but I don't know of the advantages or disadvantages either way. Maybe performance or memory issues?

RE: Add helpers :all to ApplicationController? - Added by Eric Davis at 2011-03-07 11:13 pm

helper :all will include all helpers so it might use a bit more memory (ie. more methods on each class) but I'm not sure about how the Ruby VM handles this (does it copy the module methods or point to a single module???).

There are also potential issues based on how the helpers are loaded, especially since some helpers include other helpers (issues_helper.rb pulls in application_helper.rb). But I think they can be resolved with a little work and some testing.

Eric Davis

(1-3/3)