diff options
| -rw-r--r-- | app/models/issue_relation.rb | 10 | ||||
| -rw-r--r-- | test/unit/issue_relation_test.rb | 14 |
2 files changed, 1 insertions, 23 deletions
diff --git a/app/models/issue_relation.rb b/app/models/issue_relation.rb index 0a47b1341..44c7c8dba 100644 --- a/app/models/issue_relation.rb +++ b/app/models/issue_relation.rb @@ -207,19 +207,13 @@ class IssueRelation < ActiveRecord::Base # Reverses the relation if needed so that it gets stored in the proper way # Should not be reversed before validation so that it can be displayed back - # as entered on new relation form. - # - # Orders relates relations by ID, so that uniqueness index in DB is triggered - # on concurrent access. + # as entered on new relation form def reverse_if_needed if TYPES.has_key?(relation_type) && TYPES[relation_type][:reverse] issue_tmp = issue_to self.issue_to = issue_from self.issue_from = issue_tmp self.relation_type = TYPES[relation_type][:reverse] - - elsif relation_type == TYPE_RELATES && issue_from_id > issue_to_id - self.issue_to, self.issue_from = issue_from, issue_to end end @@ -234,8 +228,6 @@ class IssueRelation < ActiveRecord::Base issue_from.blocks? issue_to when 'blocks' issue_to.blocks? issue_from - when 'relates' - self.class.where(issue_from_id: issue_to, issue_to_id: issue_from).present? else false end diff --git a/test/unit/issue_relation_test.rb b/test/unit/issue_relation_test.rb index 585a8293e..e6d126dcb 100644 --- a/test/unit/issue_relation_test.rb +++ b/test/unit/issue_relation_test.rb @@ -65,20 +65,6 @@ class IssueRelationTest < ActiveSupport::TestCase assert_equal from, relation.issue_to end - def test_cannot_create_inverse_relates_relations - from = Issue.find(1) - to = Issue.find(2) - - relation1 = IssueRelation.new :issue_from => from, :issue_to => to, - :relation_type => IssueRelation::TYPE_RELATES - assert relation1.save - - relation2 = IssueRelation.new :issue_from => to, :issue_to => from, - :relation_type => IssueRelation::TYPE_RELATES - assert !relation2.save - assert_not_equal [], relation2.errors[:base] - end - def test_follows_relation_should_not_be_reversed_if_validation_fails from = Issue.find(1) to = Issue.find(2) |