diff options
| author | Ryuta Kamizono <kamipo@gmail.com> | 2017-12-08 02:37:02 +0900 |
|---|---|---|
| committer | Ryuta Kamizono <kamipo@gmail.com> | 2017-12-08 03:42:04 +0900 |
| commit | 6a8ce7416d6615a13ee5c4b9f6bcd91cc5adef4d (patch) | |
| tree | d6885c7eddc3232aa81dc72ebb12574b5bbe0b7c /activerecord/test/cases/relation_test.rb | |
| parent | bbacd60048a8efa1777a01292a9392e146a7d885 (diff) | |
| download | rails-6a8ce7416d6615a13ee5c4b9f6bcd91cc5adef4d.tar.gz rails-6a8ce7416d6615a13ee5c4b9f6bcd91cc5adef4d.tar.bz2 rails-6a8ce7416d6615a13ee5c4b9f6bcd91cc5adef4d.zip | |
Fix `scope_for_create` to do not lose polymorphic associations
This regression was caused at 213796fb due to polymorphic predicates are
combined by `Arel::Nodes::And`. But I'd like to keep that combined
because it would help inverting polymorphic predicates correctly
(e9ba12f7), and we can collect equality nodes regardless of combined by
`Arel::Nodes::And` (`a AND (b AND c) AND d` == `a AND b AND c AND d`).
This change fixes the regression to collect equality nodes in
`Arel::Nodes::And` as well.
Fixes #31338.
Diffstat (limited to 'activerecord/test/cases/relation_test.rb')
| -rw-r--r-- | activerecord/test/cases/relation_test.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/activerecord/test/cases/relation_test.rb b/activerecord/test/cases/relation_test.rb index a71d8de521..b424ca91de 100644 --- a/activerecord/test/cases/relation_test.rb +++ b/activerecord/test/cases/relation_test.rb @@ -68,7 +68,7 @@ module ActiveRecord relation = Relation.new(Post, Post.arel_table, Post.predicate_builder) left = relation.table[:id].eq(10) right = relation.table[:id].eq(10) - combine = left.and right + combine = left.or(right) relation.where! combine assert_equal({}, relation.where_values_hash) end |