diff options
author | Sean Griffin <sean@seantheprogrammer.com> | 2016-01-14 13:43:04 -0700 |
---|---|---|
committer | Sean Griffin <sean@seantheprogrammer.com> | 2016-01-14 13:43:04 -0700 |
commit | 434ec2371498e99d4669453b66a4b9fa08f7a9a0 (patch) | |
tree | 2b1bca6c151f8c65a44e4fb71bc6f1ad6fc612cb /activerecord/lib | |
parent | f138bea5da463509c0dc51283dd35e803796a77a (diff) | |
parent | a557ab00f51e28d5609837f02fcc66d519615d3a (diff) | |
download | rails-434ec2371498e99d4669453b66a4b9fa08f7a9a0.tar.gz rails-434ec2371498e99d4669453b66a4b9fa08f7a9a0.tar.bz2 rails-434ec2371498e99d4669453b66a4b9fa08f7a9a0.zip |
Merge pull request #23048 from kamipo/substitute_at_is_no_longer_used
`substitute_at` is no longer used
Diffstat (limited to 'activerecord/lib')
3 files changed, 2 insertions, 9 deletions
diff --git a/activerecord/lib/active_record/associations/join_dependency/join_association.rb b/activerecord/lib/active_record/associations/join_dependency/join_association.rb index a6ad09a38a..be65cf318c 100644 --- a/activerecord/lib/active_record/associations/join_dependency/join_association.rb +++ b/activerecord/lib/active_record/associations/join_dependency/join_association.rb @@ -74,9 +74,8 @@ module ActiveRecord value = foreign_klass.base_class.name column = klass.columns_hash[reflection.type.to_s] - substitute = klass.connection.substitute_at(column) binds << Relation::QueryAttribute.new(column.name, value, klass.type_for_attribute(column.name)) - constraint = constraint.and table[reflection.type].eq substitute + constraint = constraint.and table[reflection.type].eq(Arel::Nodes::BindParam.new) end joins << table.create_join(table, table.create_on(constraint), join_type) diff --git a/activerecord/lib/active_record/connection_adapters/abstract_adapter.rb b/activerecord/lib/active_record/connection_adapters/abstract_adapter.rb index dce34a208f..d9b42d4283 100644 --- a/activerecord/lib/active_record/connection_adapters/abstract_adapter.rb +++ b/activerecord/lib/active_record/connection_adapters/abstract_adapter.rb @@ -311,12 +311,6 @@ module ActiveRecord {} end - # Returns a bind substitution value given a bind +column+ - # NOTE: The column param is currently being used by the sqlserver-adapter - def substitute_at(column, _unused = 0) - Arel::Nodes::BindParam.new - end - # REFERENTIAL INTEGRITY ==================================== # Override to turn off referential integrity while executing <tt>&block</tt>. diff --git a/activerecord/lib/active_record/relation.rb b/activerecord/lib/active_record/relation.rb index 6aa490908b..032b8d4c5d 100644 --- a/activerecord/lib/active_record/relation.rb +++ b/activerecord/lib/active_record/relation.rb @@ -99,7 +99,7 @@ module ActiveRecord end substitutes = values.map do |(arel_attr, _)| - [arel_attr, connection.substitute_at(klass.columns_hash[arel_attr.name])] + [arel_attr, Arel::Nodes::BindParam.new] end [substitutes, binds] |