diff options
author | Sean Griffin <sean@thoughtbot.com> | 2014-06-11 14:32:00 -0600 |
---|---|---|
committer | Sean Griffin <sean@thoughtbot.com> | 2014-06-11 15:12:36 -0600 |
commit | fef044cd2bd167201462861385ebf38bbf3158d7 (patch) | |
tree | d173e9b6834bd0bc2c795e6ce5f5131f82b4f6bf /activerecord/test | |
parent | 1b14bff81a9ab6654badf52b552f90c8fc3ab0e3 (diff) | |
download | rails-fef044cd2bd167201462861385ebf38bbf3158d7.tar.gz rails-fef044cd2bd167201462861385ebf38bbf3158d7.tar.bz2 rails-fef044cd2bd167201462861385ebf38bbf3158d7.zip |
Pluck should work with columns of the same name from different tables
The column name given by the adapter doesn't include the table
namespace, so going through the hashed version of the result set causes
overridden keys.
Fixes #15649
Diffstat (limited to 'activerecord/test')
-rw-r--r-- | activerecord/test/cases/calculations_test.rb | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/activerecord/test/cases/calculations_test.rb b/activerecord/test/cases/calculations_test.rb index dfc6a7ec67..222b1505a8 100644 --- a/activerecord/test/cases/calculations_test.rb +++ b/activerecord/test/cases/calculations_test.rb @@ -610,4 +610,11 @@ class CalculationsTest < ActiveRecord::TestCase assert_equal [1,2,3,4,5], taks_relation.pluck(:id) assert_equal [false, true, true, true, true], taks_relation.pluck(:approved) end + + def test_pluck_columns_with_same_name + expected = [["The First Topic", "The Second Topic of the day"], ["The Third Topic of the day", "The Fourth Topic of the day"]] + actual = Topic.joins(:replies) + .pluck('topics.title', 'replies_topics.title') + assert_equal expected, actual + end end |