Optimize: Only select the name when getting the module names.

This has speed up TimelogController#details by 100% with larger databases (demo.redmine went from 4077ms down to 2085ms). git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@3484 e93f8b46-1217-0410-a6f0-8f06a7374b81

Optimize: Only select the name when getting the module names.
This has speed up TimelogController#details by 100% with larger databases (demo.redmine went from 4077ms down to 2085ms). git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@3484 e93f8b46-1217-0410-a6f0-8f06a7374b81
39c97279 · Eric Davis · f7bd0801 · 39c97279
Commit 39c97279 authored Feb 25, 2010 by Eric Davis
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

project.rb app/models/project.rb +1 -1

No files found.
--- a/app/models/project.rb
+++ b/app/models/project.rb
@@ -639,7 +639,7 @@ class Project < ActiveRecord::Base
  
  def allowed_permissions
    @allowed_permissions ||= begin
-      module_names = enabled_modules.collect {|m| m.name}
+      module_names = enabled_modules.all(:select => :name).collect {|m| m.name}
      Redmine::AccessControl.modules_permissions(module_names).collect {|p| p.name}
    end
  end