##// END OF EJS Templates
Code cleanup....
Jean-Philippe Lang -
r13133:f33dfe92436e
parent child
Show More
@@ -779,25 +779,30 class Issue < ActiveRecord::Base
779 initial_status = nil
779 initial_status = nil
780 if new_record?
780 if new_record?
781 initial_status = IssueStatus.default
781 initial_status = IssueStatus.default
782 elsif status_id_was
782 else
783 initial_status = IssueStatus.find_by_id(status_id_was)
783 initial_status = status_was
784 end
784 end
785 initial_status ||= status
786
785
787 initial_assigned_to_id = assigned_to_id_changed? ? assigned_to_id_was : assigned_to_id
786 initial_assigned_to_id = assigned_to_id_changed? ? assigned_to_id_was : assigned_to_id
788 assignee_transitions_allowed = initial_assigned_to_id.present? &&
787 assignee_transitions_allowed = initial_assigned_to_id.present? &&
789 (user.id == initial_assigned_to_id || user.group_ids.include?(initial_assigned_to_id))
788 (user.id == initial_assigned_to_id || user.group_ids.include?(initial_assigned_to_id))
790
789
791 statuses = initial_status.find_new_statuses_allowed_to(
790 statuses = []
792 user.admin ? Role.all : user.roles_for_project(project),
791 if initial_status
793 tracker,
792 statuses += initial_status.find_new_statuses_allowed_to(
794 author == user,
793 user.admin ? Role.all.to_a : user.roles_for_project(project),
795 assignee_transitions_allowed
794 tracker,
796 )
795 author == user,
796 assignee_transitions_allowed
797 )
798 end
797 statuses << initial_status unless statuses.empty?
799 statuses << initial_status unless statuses.empty?
798 statuses << IssueStatus.default if include_default
800 statuses << IssueStatus.default if include_default
799 statuses = statuses.compact.uniq.sort
801 statuses = statuses.compact.uniq.sort
800 blocked? ? statuses.reject {|s| s.is_closed?} : statuses
802 if blocked?
803 statuses.reject!(&:is_closed?)
804 end
805 statuses
801 end
806 end
802 end
807 end
803
808
General Comments 0
You need to be logged in to leave comments. Login now