squeel 1.0.16 → 1.0.17

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 1.0.17 (2013-02-28)
+
+* Revert MySQL hacks, since AR did too.
+
 ## 1.0.16 (2013-02-12)
 
 * Port workaround for MySQL's "helpful" casting behavior from Rails 3.2.12

data/lib/squeel/adapters/active_record/3.0/compat.rb

@@ -142,8 +142,6 @@ module Arel
         table   = attr.relation.table_name
 
         column_cache[table][name]
-      rescue ActiveRecord::StatementInvalid # non-existent tables
-        nil
       end
 
       def column_cache

data/lib/squeel/version.rb

@@ -1,3 +1,3 @@
 module Squeel
-  VERSION = "1.0.16"
+  VERSION = "1.0.17"
 end

data/lib/squeel/visitors/predicate_visitation.rb

@@ -40,7 +40,7 @@ module Squeel
           value = visit(value, parent) if can_visit?(value)
         end
 
-        value = quote_for_node(value, o.expr, parent)
+        value = quote_for_node(o.expr, value)
 
         attribute = case o.expr
         when Nodes::Stub, Nodes::Function, Nodes::Literal, Nodes::Grouping
@@ -68,7 +68,6 @@ module Squeel
           value = visit(value.select(value.klass.arel_table[value.klass.primary_key]), parent)
         else
           value = can_visit?(value) ? visit(value, parent) : value
-          value = quote_for_attribute(value, attribute)
         end
 
         case value
@@ -109,48 +108,14 @@ module Squeel
       # can end up with annoyances like having "joe" quoted to 0, if the
       # last visited column was of an integer type.
       #
-      # @param v The value to (possibly) quote
       # @param node The node we (might) be quoting for
-      # @param parent The parent of the node being quoted for
-      def quote_for_node(v, node, parent)
+      # @param v The value to (possibly) quote
+      def quote_for_node(node, v)
         case node
         when Nodes::Function, Nodes::Literal
           quote(v)
         when Nodes::Predicate
-          quote_for_node(v, node.expr, parent)
-        when Symbol, Nodes::Stub # MySQL hates freedom
-          quote_for_attribute v, visit(node, parent)
-        else
-          v
-        end
-      end
-
-      # Because MySQL hates doing sane things, we are forced to try to quote
-      # certain values for a specific column type. Otherwise, MySQL might
-      # "helpfully" cast the column we're checking to the type we're comparing
-      # it to, resulting in such wonderful queries as...
-      #
-      #   SELECT * FROM table WHERE str_column = 0
-      #
-      # ...returning every record in the table that doesn't have a number in
-      # str_column.
-      #
-      # Everything about this method is awful. 2 x private method calls to ARel,
-      # wrapping a pre-quoted value in an SqlLiteral... Everything. My only
-      # solace is that I think we can fix it in ARel in the longer term.
-      def quote_for_attribute(v, attr)
-        case v
-        when Array
-          v.map { |v| quote_for_attribute(v, attr) }
-        when Range
-          Range.new(
-            quote_for_attribute(v.begin, attr),
-            quote_for_attribute(v.end, attr),
-            v.exclude_end?
-          )
-        when Bignum, Fixnum, Integer, ActiveSupport::Duration
-          column = arel_visitor.send(:column_for, attr)
-          Arel::Nodes::SqlLiteral.new arel_visitor.send(:quote, v, column)
+          quote_for_node(node.expr, v)
         else
           v
         end

data/lib/squeel/visitors/predicate_visitor.rb

@@ -56,11 +56,11 @@ module Squeel
 
         case k
         when Nodes::Predicate
-          visit(k % quote_for_node(v, k.expr, parent), parent)
+          visit(k % quote_for_node(k.expr, v), parent)
         when Nodes::Function, Nodes::Literal
           arel_predicate_for(visit(k, parent), quote(v), parent)
         when Nodes::KeyPath
-          visit(k % quote_for_node(v, k.endpoint, parent), parent)
+          visit(k % quote_for_node(k.endpoint, v), parent)
         else
           attr_name = k.to_s
           attribute = if !hash_context_shifted? && attr_name.include?('.')

data/spec/squeel/visitors/predicate_visitor_spec.rb

@@ -23,44 +23,6 @@ module Squeel
         node.right.should be_a Arel::Nodes::SelectStatement
       end
 
-      it 'quotes the value of a predicate based on its key' do
-        predicate = Nodes::Predicate.new(Nodes::Stub.new(:id), :eq, 1)
-        node = @v.accept(predicate)
-        node.to_sql.should be_like '"people"."id" = 1'
-        predicate = Nodes::Predicate.new(Nodes::Stub.new(:name), :eq, 1)
-        node = @v.accept(predicate)
-        node.to_sql.should be_like '"people"."name" = \'1\''
-      end
-
-      it 'quotes the value of a hash based on its key' do
-        node = @v.accept(:id => 1)
-        node.to_sql.should be_like '"people"."id" = 1'
-        node = @v.accept(:name => 1)
-        node.to_sql.should be_like '"people"."name" = \'1\''
-      end
-
-      it 'quotes array values for a Predicate based on its key' do
-        predicate = Nodes::Predicate.new(Nodes::Stub.new(:name), :in, [1, 2])
-        node = @v.accept(predicate)
-        node.to_sql.should be_like '"people"."name" IN (\'1\', \'2\')'
-      end
-
-      it 'quotes array values for a hash based on their key' do
-        node = @v.accept(:name => [1, 2])
-        node.to_sql.should be_like '"people"."name" IN (\'1\', \'2\')'
-      end
-
-      it 'quotes range values for a predicate based on its key' do
-        predicate = Nodes::Predicate.new(Nodes::Stub.new(:name), :in, 1..2)
-        node = @v.accept(predicate)
-        node.to_sql.should be_like '"people"."name" BETWEEN \'1\' AND \'2\''
-      end
-
-      it 'quotes range values for a hash based on its key' do
-        node = @v.accept(:name => 1..2)
-        node.to_sql.should be_like '"people"."name" BETWEEN \'1\' AND \'2\''
-      end
-
       it 'creates Equality nodes for simple hashes' do
         predicate = @v.accept(:name => 'Joe')
         predicate.should be_a Arel::Nodes::Equality
@@ -188,13 +150,15 @@ module Squeel
       it 'converts ActiveRecord::Base objects to their id' do
         predicate = @v.accept(:id => Person.first)
         predicate.should be_a Arel::Nodes::Equality
-        predicate.to_sql.should be_like '"people"."id" = 1'
+        predicate.left.name.to_s.should eq 'id'
+        predicate.right.should eq 1
       end
 
       it 'converts arrays of ActiveRecord::Base objects to their ids' do
         predicate = @v.accept(:id => [Person.first, Person.last])
         predicate.should be_a Arel::Nodes::In
-        predicate.to_sql.should be_like '"people"."id" IN (1, 332)'
+        predicate.left.name.to_s.should eq 'id'
+        predicate.right.should eq [1, 332]
       end
 
       it 'creates the node against the proper table for nested hashes' do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: squeel
 version: !ruby/object:Gem::Version
-  version: 1.0.16
+  version: 1.0.17
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-02-12 00:00:00.000000000 Z
+date: 2013-02-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -255,7 +255,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -3974244035788001518
+      hash: -2874076671290138869
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -264,7 +264,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -3974244035788001518
+      hash: -2874076671290138869
 requirements: []
 rubyforge_project: squeel
 rubygems_version: 1.8.24