Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Address an error for test_has_many_through_polymorphic_has_one with Oracle #5655

Merged
merged 1 commit into from

2 participants

@yahonda

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

#5640 has been merged to the 3-2-stable branch.

@spastorino spastorino merged commit 72dc7ae into rails:3-0-stable
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Mar 29, 2012
  1. @yahonda

    Address an error for test_has_many_through_polymorphic_has_one

    yahonda authored
    with Oracle for the 3-0-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
View
2  activerecord/test/cases/associations/join_model_test.rb
@@ -384,7 +384,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).taggings.order(:id)
+ assert_equal Tagging.find(1,2).sort_by { |t| t.id }, authors(:david).taggings.order('taggings.id')
end
def test_has_many_through_polymorphic_has_many
Something went wrong with that request. Please try again.