Use Arel for scope

bsylvain · Feb 20, 2012 · 3fa6c68 · 3fa6c68
1 parent 9178d2b
commit 3fa6c68
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/core/app/models/spree/preference.rb b/core/app/models/spree/preference.rb
@@ -4,7 +4,7 @@ class Spree::Preference < ActiveRecord::Base
   validates :value, :presence => true
   validates :value_type, :presence => true
 
-  scope :valid, where("key IS NOT NULL").where("value_type IS NOT NULL")
+  scope :valid, where(Spree::Preference.arel_table[:key].not_eq(nil)).where(Spree::Preference.arel_table[:value_type].not_eq(nil))
 
   # The type conversions here should match
   # the ones in spree::preferences::preferrable#convert_preference_value