| @@ -17,7 +17,7 | |||
|
|
17 | 17 | |
|
|
18 | 18 | class IssueStatus < ActiveRecord::Base |
|
|
19 | 19 | before_destroy :check_integrity |
|
|
20 | has_many :workflows, :foreign_key => "old_status_id" | |
|
|
20 | has_many :workflows, :foreign_key => "old_status_id", :dependent => :delete_all | |
|
|
21 | 21 | acts_as_list |
|
|
22 | 22 | |
|
|
23 | 23 | validates_presence_of :name |
| @@ -47,7 +47,7 class IssueStatus < ActiveRecord::Base | |||
|
|
47 | 47 | def find_new_statuses_allowed_to(role, tracker) |
|
|
48 | 48 | new_statuses = workflows.find(:all, |
|
|
49 | 49 | :include => :new_status, |
|
|
50 | :conditions => ["role_id=? and tracker_id=?", role.id, tracker.id]).collect{ |w| w.new_status } if role && tracker | |
|
|
50 | :conditions => ["role_id=? and tracker_id=?", role.id, tracker.id]).collect{ |w| w.new_status }.compact if role && tracker | |
|
|
51 | 51 | new_statuses ? new_statuses.sort{|x, y| x.position <=> y.position } : [] |
|
|
52 | 52 | end |
|
|
53 | 53 | |
General Comments 0
You need to be logged in to leave comments.
Login now