Skip to content

Commit

Permalink
Merge pull request #5 from fortytools/fix_type_cast_for_rails_4_2_0
Browse files Browse the repository at this point in the history
Fix usage of type_cast in preparation of rails 4.2.0 upgrade
  • Loading branch information
axelerator committed Feb 27, 2015
2 parents 6d327f3 + dd12624 commit cac1e1a
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion lib/forty_facets/filter/facet_filter_definition.rb
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ class AttributeFilter < FacetFilter
def selected
entity = definition.origin_class
column = entity.columns_hash[definition.attribute.to_s]
values.map{|v| column.type_cast(v)}
values.map{|v| entity.connection.type_cast(v, column)}
end

def build_scope
Expand Down Expand Up @@ -169,6 +169,8 @@ def build_filter(search_instance, param_value)
BelongsToFilter.new(self, search_instance, param_value)
elsif association.macro == :has_many
HasManyFilter.new(self, search_instance, param_value)
elsif association.macro == :has_and_belongs_to_many
HasManyFilter.new(self, search_instance, param_value)
else
raise "Unsupported association type: #{association.macro}"
end
Expand Down

0 comments on commit cac1e1a

Please sign in to comment.