##// END OF EJS Templates
Prevents useless COUNT queries....
Jean-Philippe Lang -
r13132:05462a145f33
parent child
Show More
@@ -541,11 +541,11 class Issue < ActiveRecord::Base
541 return @workflow_rule_by_attribute if @workflow_rule_by_attribute && user.nil?
541 return @workflow_rule_by_attribute if @workflow_rule_by_attribute && user.nil?
542
542
543 user_real = user || User.current
543 user_real = user || User.current
544 roles = user_real.admin ? Role.all : user_real.roles_for_project(project)
544 roles = user_real.admin ? Role.all.to_a : user_real.roles_for_project(project)
545 return {} if roles.empty?
545 return {} if roles.empty?
546
546
547 result = {}
547 result = {}
548 workflow_permissions = WorkflowPermission.where(:tracker_id => tracker_id, :old_status_id => status_id, :role_id => roles.map(&:id))
548 workflow_permissions = WorkflowPermission.where(:tracker_id => tracker_id, :old_status_id => status_id, :role_id => roles.map(&:id)).to_a
549 if workflow_permissions.any?
549 if workflow_permissions.any?
550 workflow_rules = workflow_permissions.inject({}) do |h, wp|
550 workflow_rules = workflow_permissions.inject({}) do |h, wp|
551 h[wp.field_name] ||= []
551 h[wp.field_name] ||= []
General Comments 0
You need to be logged in to leave comments. Login now