diff options
| author | Sean Griffin <sean@thoughtbot.com> | 2015-01-27 12:07:06 -0700 |
|---|---|---|
| committer | Sean Griffin <sean@thoughtbot.com> | 2015-01-27 12:07:06 -0700 |
| commit | 6c235dd95812a7c4e8123b6d34d6f3433805ee1c (patch) | |
| tree | 9c148b9d42c13a88fe646494529d9f756e391405 /activerecord/lib/active_record/relation/from_clause.rb | |
| parent | 102a5272c5944a6f715da8332d18e7e0380727d1 (diff) | |
| download | rails-6c235dd95812a7c4e8123b6d34d6f3433805ee1c.tar.gz rails-6c235dd95812a7c4e8123b6d34d6f3433805ee1c.tar.bz2 rails-6c235dd95812a7c4e8123b6d34d6f3433805ee1c.zip | |
Use an `Attribute` object to represent a bind value
The column is primarily used for type casting, which we're trying to
separate from the idea of a column. Since what we really need is the
combination of a name, type, and value, let's use the object that we
already have to represent that concept, rather than this tuple. No
consumers of the bind values have been changed, only the producers
(outside of tests which care too much about internals). This is
*finally* possible since the bind values are now produced from a
reasonable number of lcoations.
Diffstat (limited to 'activerecord/lib/active_record/relation/from_clause.rb')
| -rw-r--r-- | activerecord/lib/active_record/relation/from_clause.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/activerecord/lib/active_record/relation/from_clause.rb b/activerecord/lib/active_record/relation/from_clause.rb index 8022901524..a93952fa30 100644 --- a/activerecord/lib/active_record/relation/from_clause.rb +++ b/activerecord/lib/active_record/relation/from_clause.rb @@ -10,7 +10,7 @@ module ActiveRecord def binds if value.is_a?(Relation) - value.bind_values + value.bound_attributes else [] end |