]> source.dussan.org Git - redmine.git/commitdiff
Fixed: default status not showing in new issue
authorJean-Philippe Lang <jp_lang@yahoo.fr>
Tue, 24 Apr 2007 15:19:49 +0000 (15:19 +0000)
committerJean-Philippe Lang <jp_lang@yahoo.fr>
Tue, 24 Apr 2007 15:19:49 +0000 (15:19 +0000)
git-svn-id: http://redmine.rubyforge.org/svn/trunk@475 e93f8b46-1217-0410-a6f0-8f06a7374b81

app/controllers/projects_controller.rb
app/models/issue_status.rb

index 494c231395190d065b482f59a1dce60d1d678007..19f93de9698a833ac23628df03b66f22dcc85fb4 100644 (file)
@@ -216,7 +216,7 @@ class ProjectsController < ApplicationController
     default_status = IssueStatus.default
     @issue = Issue.new(:project => @project, :tracker => @tracker)    
     @issue.status = default_status
-    @allowed_statuses = (default_status.find_new_statuses_allowed_to(logged_in_user.role_for_project(@project), @issue.tracker) || [default_status])if logged_in_user
+    @allowed_statuses = ([default_status] + default_status.find_new_statuses_allowed_to(logged_in_user.role_for_project(@project), @issue.tracker))if logged_in_user
     if request.get?
       @issue.start_date = Date.today
       @custom_values = @project.custom_fields_for_issues(@tracker).collect { |x| CustomValue.new(:custom_field => x, :customized => @issue) }
index 30a96f4ec5c086259e07b6414f2c84da46702f48..cea77132e28591e094257ea32366ad48acd5be3f 100644 (file)
@@ -39,7 +39,7 @@ class IssueStatus < ActiveRecord::Base
   # Uses association cache when called more than one time
   def new_statuses_allowed_to(role, tracker)
     new_statuses = workflows.select {|w| w.role_id == role.id && w.tracker_id == tracker.id}.collect{|w| w.new_status} if role && tracker
-    new_statuses.sort{|x, y| x.position <=> y.position } if new_statuses
+    new_statuses ? new_statuses.sort{|x, y| x.position <=> y.position } : []
   end
   
   # Same thing as above but uses a database query
@@ -48,7 +48,7 @@ class IssueStatus < ActiveRecord::Base
     new_statuses = workflows.find(:all, 
                                    :include => :new_status,
                                    :conditions => ["role_id=? and tracker_id=?", role.id, tracker.id]).collect{ |w| w.new_status }  if role && tracker
-    new_statuses.sort{|x, y| x.position <=> y.position } if new_statuses
+    new_statuses ? new_statuses.sort{|x, y| x.position <=> y.position } : []
   end
   
 private