]> source.dussan.org Git - redmine.git/commitdiff
Code cleanup.
authorJean-Philippe Lang <jp_lang@yahoo.fr>
Thu, 9 Feb 2012 19:51:38 +0000 (19:51 +0000)
committerJean-Philippe Lang <jp_lang@yahoo.fr>
Thu, 9 Feb 2012 19:51:38 +0000 (19:51 +0000)
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@8826 e93f8b46-1217-0410-a6f0-8f06a7374b81

app/controllers/context_menus_controller.rb
app/models/issue.rb

index 46eba1abf6fcc82037749455fa53f99a50e72b70..fa7caf4e03e7e5dbac87f58523a340a22fc23c6a 100644 (file)
@@ -4,17 +4,11 @@ class ContextMenusController < ApplicationController
 
   def issues
     @issues = Issue.visible.all(:conditions => {:id => params[:ids]}, :include => :project)
-
     if (@issues.size == 1)
       @issue = @issues.first
-      @allowed_statuses = @issue.new_statuses_allowed_to(User.current)
-    else
-      @allowed_statuses = @issues.map do |i|
-        i.new_statuses_allowed_to(User.current)
-      end.inject do |memo,s|
-        memo & s
-      end
     end
+
+    @allowed_statuses = @issues.map(&:new_statuses_allowed_to).inject{|memo,a| memo & a}
     @projects = @issues.collect(&:project).compact.uniq
     @project = @projects.first if @projects.size == 1
 
index 098113bdfa71d63abcb34b80cdfa0bfd7e2d0ff3..64a99e6be97b56883f9f494ef8b6f41248c79d61 100644 (file)
@@ -510,7 +510,7 @@ class Issue < ActiveRecord::Base
   end
 
   # Returns an array of status that user is able to apply
-  def new_statuses_allowed_to(user, include_default=false)
+  def new_statuses_allowed_to(user=User.current, include_default=false)
     statuses = status.find_new_statuses_allowed_to(
       user.admin ? Role.all : user.roles_for_project(project),
       tracker,