Skip to content
This repository

association_preload.rb:336: warning: Object#type is deprecated #157

Closed
wants to merge 3 commits into from

3 participants

ServerCrunch Jeremy Kemper bluetrans-deploy
ServerCrunch

No description provided.

Jeremy Kemper
Owner

Thanks!

Tom Stuart tomstuart referenced this pull request from a commit in econsultancy/rails
Ivan Torres mexpolk [PATCH] Update guides after Jeremy Kemper's changes on fieldWithErrors
to field_with_errors [#157 state:resolved]
2d2bde9
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Showing 3 unique commits by 1 author.

Jan 06, 2011
bluetrans-deploy bluetrans-deploy merge passed options f18cd30
Jan 07, 2011
bluetrans-deploy bluetrans-deploy Revert "merge passed options"
This reverts commit f18cd3045b2d7e7463df78d264de2f9dc138122e.
989ffe4
bluetrans-deploy bluetrans-deploy use Object#class instead of Object#type 0ee9e07
This page is out of date. Refresh to see the latest.
2  activerecord/lib/active_record/association_preload.rb
@@ -333,7 +333,7 @@ def preload_belongs_to_association(records, reflection, preload_options={})
333 333
334 334 table_name = klass.quoted_table_name
335 335 primary_key = reflection.options[:primary_key] || klass.primary_key
336   - column_type = klass.columns.detect{|c| c.name == primary_key}.type
  336 + column_type = klass.columns.detect{|c| c.name == primary_key}.class
337 337 ids = id_map.keys.map do |id|
338 338 if column_type == :integer
339 339 id.to_i

Tip: You can add notes to lines in a file. Hover to the left of a line to make a note

Something went wrong with that request. Please try again.