From: Jean-Philippe Lang <jp_lang@yahoo.fr>
Date: Sun, 13 Mar 2011 13:00:55 +0000 (+0000)
Subject: Prevent SystemStackError on Issue#all_dependent_issues with mutiple circular dependen... 
X-Git-Tag: 1.2.0~739
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=9375f1c3a4ea42a7175febf75da404dbf59aa7b0;p=redmine.git

Prevent SystemStackError on Issue#all_dependent_issues with mutiple circular dependencies (#7320).

git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@5104 e93f8b46-1217-0410-a6f0-8f06a7374b81
---

diff --git a/app/models/issue.rb b/app/models/issue.rb
index 94b4fc0c8..de31aa636 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 8dcee3847..f4dcf6625 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