diff --git a/app/models/issue.rb b/app/models/issue.rb index 94b4fc0..de31aa6 100644 --- a/app/models/issue.rb +++ b/app/models/issue.rb @@ -455,11 +455,11 @@ class Issue < ActiveRecord::Base (relations_from + relations_to).sort end - def all_dependent_issues(except=nil) - except ||= self + def all_dependent_issues(except=[]) + except << self dependencies = [] relations_from.each do |relation| - if relation.issue_to && relation.issue_to != except + if relation.issue_to && !except.include?(relation.issue_to) dependencies << relation.issue_to dependencies += relation.issue_to.all_dependent_issues(except) end diff --git a/test/unit/issue_test.rb b/test/unit/issue_test.rb index 8dcee38..f4dcf66 100644 --- a/test/unit/issue_test.rb +++ b/test/unit/issue_test.rb @@ -686,6 +686,18 @@ class IssueTest < ActiveSupport::TestCase assert_equal [2, 3], Issue.find(1).all_dependent_issues.collect(&:id).sort end + + def test_all_dependent_issues_with_persistent_multiple_circular_dependencies + IssueRelation.delete_all + assert IssueRelation.create!(:issue_from => Issue.find(1), :issue_to => Issue.find(2), :relation_type => IssueRelation::TYPE_RELATES) + assert IssueRelation.create!(:issue_from => Issue.find(2), :issue_to => Issue.find(3), :relation_type => IssueRelation::TYPE_RELATES) + assert IssueRelation.create!(:issue_from => Issue.find(3), :issue_to => Issue.find(8), :relation_type => IssueRelation::TYPE_RELATES) + # Validation skipping + assert IssueRelation.new(:issue_from => Issue.find(8), :issue_to => Issue.find(2), :relation_type => IssueRelation::TYPE_RELATES).save(false) + assert IssueRelation.new(:issue_from => Issue.find(3), :issue_to => Issue.find(1), :relation_type => IssueRelation::TYPE_RELATES).save(false) + + assert_equal [2, 3, 8], Issue.find(1).all_dependent_issues.collect(&:id).sort + end context "#done_ratio" do setup do