Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Address an error for test_has_many_through_polymorphic_has_one with Oracle #5658

Merged
merged 1 commit into from

2 participants

Yasuo Honda Santiago Pastorino
Yasuo Honda

This pull request address the issue #5636 to the 3-1-stable branch.

#5640 has been merged to the 3-2-stable branch.
Opened #5655 for the 3-0-stable branch.

Santiago Pastorino spastorino merged commit 5cbb20d into from
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Mar 29, 2012
  1. Yasuo Honda

    Address an error for test_has_many_through_polymorphic_has_one

    yahonda authored
    with Oracle for the 3-1-stable branch
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  activerecord/test/cases/associations/join_model_test.rb
2  activerecord/test/cases/associations/join_model_test.rb
View
@@ -403,7 +403,7 @@ def test_has_many_through_has_many_find_by_id
end
def test_has_many_through_polymorphic_has_one
- assert_equal Tagging.find(1,2).sort_by { |t| t.id }, authors(:david).tagging.order(:id)
+ assert_equal Tagging.find(1,2).sort_by { |t| t.id }, authors(:david).tagging.order('taggings.id')
end
def test_has_many_through_polymorphic_has_many
Something went wrong with that request. Please try again.