Document why relation is reversed after validation.

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

app/models/issue_relation.rb

@@ -115,6 +115,8 @@ class IssueRelation < ActiveRecord::Base
   private
   
   # 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
   def reverse_if_needed
     if TYPES.has_key?(relation_type) && TYPES[relation_type][:reverse]
       issue_tmp = issue_to

test/unit/issue_relation_test.rb

@@ -50,9 +50,6 @@ class IssueRelationTest < ActiveSupport::TestCase
     assert_equal from, relation.issue_to
   end
   
-  # TODO : document why it shouldn't be reversed if validation fails : having
-  # relations reversed before the validation would allow simpler code for the
-  # validation
   def test_follows_relation_should_not_be_reversed_if_validation_fails
     from = Issue.find(1)
     to = Issue.find(2)