arel_extensions 1.1.8 → 1.1.9

data/lib/arel_extensions/visitors/oracle.rb

@@ -1,7 +1,7 @@
 #require 'oracle_visitor'
 module ArelExtensions
   module Visitors
-    Arel::Visitors::Oracle.class_eval do    
+    Arel::Visitors::Oracle.class_eval do
 
       SPECIAL_CHARS = {"\t" => 'CHR(9)', "\n" => 'CHR(10)', "\r" => 'CHR(13)'}
       Arel::Visitors::Oracle::DATE_MAPPING = {'d' => 'DAY', 'm' => 'MONTH', 'w' => 'IW', 'y' => 'YEAR', 'wd' => 'D', 'h' => 'HOUR', 'mn' => 'MINUTE', 's' => 'SECOND'}
@@ -13,25 +13,25 @@ module ArelExtensions
       }
       Arel::Visitors::Oracle::NUMBER_COMMA_MAPPING = { 'en_US' => '.,', 'fr_FR' => ',', 'sv_SE' => ', ' }
 
-    def visit_ArelExtensions_Nodes_Log10 o, collector
-        collector << "LOG("
-        o.expressions.each_with_index { |arg, i|
-          collector << Arel::Visitors::ToSql::COMMA unless i == 0
-          collector = visit arg, collector
-        }
-        collector << ",10)"
-        collector
-    end
+      def visit_ArelExtensions_Nodes_Log10 o, collector
+          collector << "LOG("
+          o.expressions.each_with_index { |arg, i|
+            collector << Arel::Visitors::ToSql::COMMA unless i == 0
+            collector = visit arg, collector
+          }
+          collector << ",10)"
+          collector
+      end
 
-  def visit_ArelExtensions_Nodes_Power o, collector
-        collector << "POWER("
-        o.expressions.each_with_index { |arg, i|
-          collector << Arel::Visitors::ToSql::COMMA unless i == 0
-          collector = visit arg, collector
-        }
-        collector << ")"
-        collector
-  end
+      def visit_ArelExtensions_Nodes_Power o, collector
+            collector << "POWER("
+            o.expressions.each_with_index { |arg, i|
+              collector << Arel::Visitors::ToSql::COMMA unless i == 0
+              collector = visit arg, collector
+            }
+            collector << ")"
+            collector
+      end
 
       def visit_ArelExtensions_Nodes_Concat o, collector
         collector << '('
@@ -56,7 +56,7 @@ module ArelExtensions
           collector
         end
       end
-      
+
       def visit_ArelExtensions_Nodes_AiMatches o, collector
         collector = visit o.left.ai_collate, collector
         collector << ' LIKE '
@@ -68,7 +68,7 @@ module ArelExtensions
           collector
         end
       end
-      
+
       def visit_ArelExtensions_Nodes_AiIMatches o, collector
         collector = visit o.left.ai_collate, collector
         collector << ' LIKE '
@@ -80,7 +80,7 @@ module ArelExtensions
           collector
         end
       end
-      
+
       def visit_ArelExtensions_Nodes_SMatches o, collector
         collector = visit o.left, collector
         collector << ' LIKE '
@@ -91,8 +91,8 @@ module ArelExtensions
         else
           collector
         end
-      end  
-      
+      end
+
 
       def visit_ArelExtensions_Nodes_IDoesNotMatch o, collector
         collector << 'LOWER('
@@ -112,13 +112,13 @@ module ArelExtensions
       if o.ai
         collector << "NLSSORT("
         collector = visit o.expressions.first, collector
-        collector << Arel::Visitors::Oracle::COMMA 
+        collector << Arel::Visitors::Oracle::COMMA
         collector << "'NLS_SORT = BINARY_AI NLS_COMP = LINGUISTIC'"
         collector << ")"
       elsif o.ci
         collector << "NLSSORT("
         collector = visit o.expressions.first, collector
-        collector << Arel::Visitors::Oracle::COMMA 
+        collector << Arel::Visitors::Oracle::COMMA
         collector << "'NLS_SORT = BINARY_CI NLS_COMP = LINGUISTIC'"
         collector << ")"
       else
@@ -126,7 +126,6 @@ module ArelExtensions
       end
         collector
       end
-      
 
       def visit_ArelExtensions_Nodes_GroupConcat o, collector
         collector << "(LISTAGG("
@@ -134,7 +133,7 @@ module ArelExtensions
         collector << Arel::Visitors::Oracle::COMMA
         if o.right  && o.right != 'NULL'
           collector = visit o.right, collector
-        else          
+        else
           collector = visit Arel::Nodes.build_quoted(','), collector
         end
         collector << ") WITHIN GROUP (ORDER BY "
@@ -220,20 +219,20 @@ module ArelExtensions
         collector
       end
 
-      def visit_ArelExtensions_Nodes_Cast o, collector	
+      def visit_ArelExtensions_Nodes_Cast o, collector
         case o.as_attr
         when :string
           as_attr = Arel::Nodes::SqlLiteral.new('varchar(255)')
         when :time
-          left = Arel::Nodes::NamedFunction.new('TO_CHAR',[o.left,Arel::Nodes.build_quoted('HH24:MI:SS')])			
+          left = Arel::Nodes::NamedFunction.new('TO_CHAR',[o.left,Arel::Nodes.build_quoted('HH24:MI:SS')])
           collector = visit left, collector
           return collector
-        when :number 
+        when :number
           as_attr = Arel::Nodes::SqlLiteral.new('int')
-        when :datetime 
+        when :datetime
           as_attr = Arel::Nodes::SqlLiteral.new('timestamp')
-        when :binary			
-          as_attr = Arel::Nodes::SqlLiteral.new('binary')		
+        when :binary
+          as_attr = Arel::Nodes::SqlLiteral.new('binary')
         else
           as_attr = Arel::Nodes::SqlLiteral.new(o.as_attr.to_s)
         end
@@ -245,8 +244,6 @@ module ArelExtensions
         collector
       end
 
-
-
       def visit_ArelExtensions_Nodes_Length o, collector
         collector << "LENGTH("
         collector = visit o.expr, collector
@@ -254,13 +251,12 @@ module ArelExtensions
         collector
       end
 
-
       def visit_ArelExtensions_Nodes_IsNull o, collector
         collector = visit o.left, collector
         collector << ' IS NULL'
         collector
       end
-      
+
       def visit_ArelExtensions_Nodes_IsNotNull o, collector
         collector = visit o.left, collector
         collector << ' IS NOT NULL'
@@ -397,7 +393,7 @@ module ArelExtensions
         collector << ")"
         collector
       end
-      
+
       def visit_ArelExtensions_Nodes_Repeat o, collector
         collector << "LPAD("
         collector = visit o.expressions[0], collector #can't put empty string, otherwise it wouldn't work
@@ -407,214 +403,228 @@ module ArelExtensions
         collector = visit o.expressions[0], collector
         collector << ")"
         collector
-      end  
-
-      # add primary_key if not present, avoid zip
-    if Arel::VERSION.to_i < 7
-      def visit_ArelExtensions_InsertManager_BulkValues o, collector
-        collector << "("
-        o.left.each_with_index do |row, idx| # values
-        collector << " UNION ALL " if idx != 0
-          collector << "(SELECT "
-          v = Arel::Nodes::Values.new(row, o.cols)
-          len = v.expressions.length - 1
-          v.expressions.each_with_index { |value, i|
+      end
+
+        # add primary_key if not present, avoid zip
+      if Arel::VERSION.to_i < 7
+        def visit_ArelExtensions_InsertManager_BulkValues o, collector
+          collector << "("
+          o.left.each_with_index do |row, idx| # values
+          collector << " UNION ALL " if idx != 0
+            collector << "(SELECT "
+            v = Arel::Nodes::Values.new(row, o.cols)
+            len = v.expressions.length - 1
+            v.expressions.each_with_index { |value, i|
+                case value
+                when Arel::Nodes::SqlLiteral, Arel::Nodes::BindParam
+                  collector = visit value, collector
+                else
+                  attr = v.columns[i]
+                  collector << quote(value, attr && column_for(attr)).to_s
+                end
+                collector << Arel::Visitors::Oracle::COMMA unless i == len
+            }
+            collector << ' FROM DUAL)'
+          end
+          collector << ")"
+          collector
+        end
+      else
+        def visit_ArelExtensions_InsertManager_BulkValues o, collector
+          collector << "("
+          o.left.each_with_index do |row, idx|
+            collector << " UNION ALL " if idx != 0
+            collector << "(SELECT "
+            v = Arel::Nodes::Values.new(row, o.cols)
+            len = v.expressions.length - 1
+            v.expressions.zip(v.columns).each_with_index { |(value, attr), i|
               case value
               when Arel::Nodes::SqlLiteral, Arel::Nodes::BindParam
                 collector = visit value, collector
               else
-                attr = v.columns[i]
-                collector << quote(value, attr && column_for(attr)).to_s
+                collector << (attr && attr.able_to_type_cast? ? quote(attr.type_cast_for_database(value)) : quote(value).to_s)
               end
               collector << Arel::Visitors::Oracle::COMMA unless i == len
-          }
-          collector << ' FROM DUAL)'
+            }
+            collector << ' FROM DUAL)'
+          end
+          collector << ")"
+          collector
         end
-        collector << ")"
+      end
+
+
+      def get_time_converted element
+        if element.is_a?(Time)
+          ArelExtensions::Nodes::Format.new [element, '%H:%M:%S']
+        elsif element.is_a?(Arel::Attributes::Attribute)
+          col = Arel::Table.engine.connection.schema_cache.columns_hash(element.relation.table_name)[element.name.to_s]
+          if col && (col.type == :time)
+            ArelExtensions::Nodes::Format.new [element, '%H:%M:%S']
+          else
+            element
+          end
+        else
+          element
+        end
+      end
+
+
+      remove_method(:visit_Arel_Nodes_GreaterThanOrEqual) rescue nil
+      def visit_Arel_Nodes_GreaterThanOrEqual o, collector
+        collector = visit get_time_converted(o.left), collector
+        collector << " >= "
+        collector = visit get_time_converted(o.right), collector
         collector
       end
-    else
-      def visit_ArelExtensions_InsertManager_BulkValues o, collector
-        collector << "("
-        o.left.each_with_index do |row, idx|
-          collector << " UNION ALL " if idx != 0
-          collector << "(SELECT "
-          v = Arel::Nodes::Values.new(row, o.cols)
-          len = v.expressions.length - 1
-          v.expressions.zip(v.columns).each_with_index { |(value, attr), i|
-            case value
-            when Arel::Nodes::SqlLiteral, Arel::Nodes::BindParam
-              collector = visit value, collector
-            else
-              collector << (attr && attr.able_to_type_cast? ? quote(attr.type_cast_for_database(value)) : quote(value).to_s)
-            end
-            collector << Arel::Visitors::Oracle::COMMA unless i == len
-          }
-          collector << ' FROM DUAL)'
+
+      remove_method(:visit_Arel_Nodes_GreaterThan) rescue nil
+      def visit_Arel_Nodes_GreaterThan o, collector
+        collector = visit get_time_converted(o.left), collector
+        collector << " > "
+        collector = visit get_time_converted(o.right), collector
+        collector
+      end
+
+      remove_method(:visit_Arel_Nodes_LessThanOrEqual) rescue nil
+      def visit_Arel_Nodes_LessThanOrEqual o, collector
+        collector = visit get_time_converted(o.left), collector
+        collector << " <= "
+        collector = visit get_time_converted(o.right), collector
+        collector
+      end
+
+      remove_method(:visit_Arel_Nodes_LessThan) rescue nil
+      def visit_Arel_Nodes_LessThan o, collector
+        collector = visit get_time_converted(o.left), collector
+        collector << " < "
+        collector = visit get_time_converted(o.right), collector
+        collector
+      end
+
+
+      alias_method :old_visit_Arel_Nodes_SelectStatement, :visit_Arel_Nodes_SelectStatement
+      def visit_Arel_Nodes_SelectStatement o, collector
+        if !(collector.value.blank? || (collector.value.is_a?(Array) && collector.value[0].blank?)) && o.limit.blank? && o.offset.blank?
+          o = o.dup
+          o.orders = []
         end
-        collector << ")"
+        old_visit_Arel_Nodes_SelectStatement(o,collector)
+      end
+
+      alias_method :old_visit_Arel_Nodes_TableAlias, :visit_Arel_Nodes_TableAlias
+      def visit_Arel_Nodes_TableAlias o, collector
+        if o.name.length > 30
+          o = Arel::Table.new(o.table_name).alias(Base64.urlsafe_encode64(Digest::MD5.new.digest(o.name)).tr('=', '').tr('-', '_'))
+        end
+        old_visit_Arel_Nodes_TableAlias(o,collector)
+      end
+
+      alias_method :old_visit_Arel_Nodes_As, :visit_Arel_Nodes_As
+      def visit_Arel_Nodes_As o, collector
+        if o.left.is_a?(Arel::Nodes::Binary)
+          collector << '('
+          collector = visit o.left, collector
+          collector << ')'
+        else
+          collector = visit o.left, collector
+        end
+        quote = o.right.to_s =~ /[^a-zA-Z_]/ ? '"' : ''
+        collector << " AS #{quote}"
+        collector = visit o.right, collector
+        collector << "#{quote}"
         collector
       end
-    end
 
+      alias_method :old_visit_Arel_Attributes_Attribute, :visit_Arel_Attributes_Attribute
+      def visit_Arel_Attributes_Attribute o, collector
+        join_name = o.relation.table_alias || o.relation.name
+        if join_name.length > 30
+          join_name = Arel.shorten(join_name)
+        end
+        collector << "#{quote_table_name join_name}.#{quote_column_name o.name}"
+      end
 
-	def get_time_converted element
-		if element.is_a?(Time)
-			ArelExtensions::Nodes::Format.new [element, '%H:%M:%S']
-		elsif element.is_a?(Arel::Attributes::Attribute)
-			col = Arel::Table.engine.connection.schema_cache.columns_hash(element.relation.table_name)[element.name.to_s]
-			if col && (col.type == :time)
-				ArelExtensions::Nodes::Format.new [element, '%H:%M:%S']
-			else
-				element
-			end
-		else
-			element
-		end
-	end
-	
-	
-	remove_method(:visit_Arel_Nodes_GreaterThanOrEqual) rescue nil 
-	def visit_Arel_Nodes_GreaterThanOrEqual o, collector
-		collector = visit get_time_converted(o.left), collector
-		collector << " >= "		
-		collector = visit get_time_converted(o.right), collector
-		collector
-	end
-
-	remove_method(:visit_Arel_Nodes_GreaterThan) rescue nil 
-	def visit_Arel_Nodes_GreaterThan o, collector
-		collector = visit get_time_converted(o.left), collector
-		collector << " > "		
-		collector = visit get_time_converted(o.right), collector
-		collector
-	end
-
-	remove_method(:visit_Arel_Nodes_LessThanOrEqual) rescue nil 
-	def visit_Arel_Nodes_LessThanOrEqual o, collector
-		collector = visit get_time_converted(o.left), collector
-		collector << " <= "		
-		collector = visit get_time_converted(o.right), collector
-		collector
-	end
-	
-	remove_method(:visit_Arel_Nodes_LessThan) rescue nil 
-	def visit_Arel_Nodes_LessThan o, collector
-		collector = visit get_time_converted(o.left), collector
-		collector << " < "		
-		collector = visit get_time_converted(o.right), collector
-		collector
-	end
-
-	
-	alias_method :old_visit_Arel_Nodes_SelectStatement, :visit_Arel_Nodes_SelectStatement
-	def visit_Arel_Nodes_SelectStatement o, collector	
-		if !(collector.value.blank? || (collector.value.is_a?(Array) && collector.value[0].blank?)) && o.limit.blank? && o.offset.blank?
-			o = o.dup
-			o.orders = []
-		end
-		old_visit_Arel_Nodes_SelectStatement(o,collector)
-	end	
-	
-	alias_method :old_visit_Arel_Nodes_TableAlias, :visit_Arel_Nodes_TableAlias  
-	def visit_Arel_Nodes_TableAlias o, collector
-		if o.name.length > 30
-			o = Arel::Table.new(o.table_name).alias(Base64.urlsafe_encode64(Digest::MD5.new.digest(o.name)).tr('=', '').tr('-', '_'))
-		end
-		old_visit_Arel_Nodes_TableAlias(o,collector)		
-	end
-	
-	
-	alias_method :old_visit_Arel_Attributes_Attribute, :visit_Arel_Attributes_Attribute  
-	def visit_Arel_Attributes_Attribute o, collector			
-		join_name = o.relation.table_alias || o.relation.name
-		if join_name.length > 30
-			join_name = Arel.shorten(join_name)
-		end
-		collector << "#{quote_table_name join_name}.#{quote_column_name o.name}"
-	end
-	
-
-	def visit_ArelExtensions_Nodes_FormattedNumber o, collector		
-		col = o.left.coalesce(0)
-		comma = Arel::Visitors::Oracle::NUMBER_COMMA_MAPPING[o.locale] || '.,'
-		comma_in_format = o.precision == 0 ? '' : 'D'
-		nines_after = (1..o.precision-1).map{'9'}.join('')+'0'
-		if comma.length == 1
-			options = Arel::Nodes.build_quoted("NLS_NUMERIC_CHARACTERS = '"+comma+" '")
-			nines_before = ("999"*4+"990")
-		else
-			options = Arel::Nodes.build_quoted("NLS_NUMERIC_CHARACTERS = '"+comma+"'")
-			nines_before = ("999G"*4+"990")
-		end
-		sign = ArelExtensions::Nodes::Case.new.when(col<0).
-							then('-').
-							else(o.flags.include?('+') ? '+' : (o.flags.include?(' ') ? ' ' : ''))
-		sign_length = o.flags.include?('+') || o.flags.include?(' ') ? 
-						Arel::Nodes.build_quoted(1) : 
-						ArelExtensions::Nodes::Case.new.when(col<0).then(1).else(0)
-		
-		if o.scientific_notation 
-			number = Arel::Nodes::NamedFunction.new('TO_CHAR',[
-						Arel::Nodes.build_quoted(col.abs),
-						Arel::Nodes.build_quoted('FM'+nines_before+comma_in_format+nines_after+'EEEE'),
-						options
-					])
-			if o.type == 'e'
-				number = number.replace('E','e')
-			end
-		else
-			number = Arel::Nodes::NamedFunction.new('TO_CHAR',[
-						Arel::Nodes.build_quoted(col.abs),
-						Arel::Nodes.build_quoted('FM'+nines_before+comma_in_format+nines_after),
-						options
-					])
-		end
-		
-		repeated_char = (o.width == 0) ? Arel::Nodes.build_quoted('') : ArelExtensions::Nodes::Case.new().
-			when(Arel::Nodes.build_quoted(o.width).abs-(number.length+sign_length)>0).
-			then(Arel::Nodes.build_quoted(
-					o.flags.include?('-') ? ' ' : (o.flags.include?('0') ? '0' : ' ')
-				).repeat(Arel::Nodes.build_quoted(o.width).abs-(number.length+sign_length))
-			).
-			else('')
-		before = (!o.flags.include?('0'))&&(!o.flags.include?('-')) ? repeated_char : ''
-		middle = (o.flags.include?('0'))&&(!o.flags.include?('-'))  ? repeated_char : ''
-		after  = o.flags.include?('-') ? repeated_char : ''
-		full_number = ArelExtensions::Nodes::Concat.new([
-				before,
-				sign,
-				middle,
-				number,
-				after
-			])
-		collector = visit ArelExtensions::Nodes::Concat.new([Arel::Nodes.build_quoted(o.prefix),full_number,Arel::Nodes.build_quoted(o.suffix)]), collector
-		collector		
-	end
-
-		def visit_ArelExtensions_Nodes_Std o, collector		
-			collector << (o.unbiased_estimator ? "STDDEV_SAMP(" : "STDDEV_POP(")
-			visit o.left, collector
-			collector << ")"
-			collector
-		end
-		
-		def visit_ArelExtensions_Nodes_Variance o, collector	
-			collector << (o.unbiased_estimator ? "VAR_SAMP(" : "VAR_POP(")
-			visit o.left, collector
-			collector << ")"
-			collector
-		end
-		
-		def visit_ArelExtensions_Nodes_LevenshteinDistance o, collector
-			collector << "UTL_MATCH.edit_distance("
-			collector = visit o.left, collector
-			collector << Arel::Visitors::ToSql::COMMA
-			collector = visit o.right, collector
-			collector << ')'
-			collector
-		end
 
+      def visit_ArelExtensions_Nodes_FormattedNumber o, collector
+        col = o.left.coalesce(0)
+        comma = Arel::Visitors::Oracle::NUMBER_COMMA_MAPPING[o.locale] || '.,'
+        comma_in_format = o.precision == 0 ? '' : 'D'
+        nines_after = (1..o.precision-1).map{'9'}.join('')+'0'
+        if comma.length == 1
+          options = Arel::Nodes.build_quoted("NLS_NUMERIC_CHARACTERS = '"+comma+" '")
+          nines_before = ("999"*4+"990")
+        else
+          options = Arel::Nodes.build_quoted("NLS_NUMERIC_CHARACTERS = '"+comma+"'")
+          nines_before = ("999G"*4+"990")
+        end
+        sign = ArelExtensions::Nodes::Case.new.when(col<0).
+                  then('-').
+                  else(o.flags.include?('+') ? '+' : (o.flags.include?(' ') ? ' ' : ''))
+        sign_length = o.flags.include?('+') || o.flags.include?(' ') ?
+                Arel::Nodes.build_quoted(1) :
+                ArelExtensions::Nodes::Case.new.when(col<0).then(1).else(0)
+
+        if o.scientific_notation
+          number = Arel::Nodes::NamedFunction.new('TO_CHAR',[
+                Arel::Nodes.build_quoted(col.abs),
+                Arel::Nodes.build_quoted('FM'+nines_before+comma_in_format+nines_after+'EEEE'),
+                options
+              ])
+          if o.type == 'e'
+            number = number.replace('E','e')
+          end
+        else
+          number = Arel::Nodes::NamedFunction.new('TO_CHAR',[
+                Arel::Nodes.build_quoted(col.abs),
+                Arel::Nodes.build_quoted('FM'+nines_before+comma_in_format+nines_after),
+                options
+              ])
+        end
+
+        repeated_char = (o.width == 0) ? Arel::Nodes.build_quoted('') : ArelExtensions::Nodes::Case.new().
+          when(Arel::Nodes.build_quoted(o.width).abs-(number.length+sign_length)>0).
+          then(Arel::Nodes.build_quoted(
+              o.flags.include?('-') ? ' ' : (o.flags.include?('0') ? '0' : ' ')
+            ).repeat(Arel::Nodes.build_quoted(o.width).abs-(number.length+sign_length))
+          ).
+          else('')
+        before = (!o.flags.include?('0'))&&(!o.flags.include?('-')) ? repeated_char : ''
+        middle = (o.flags.include?('0'))&&(!o.flags.include?('-'))  ? repeated_char : ''
+        after  = o.flags.include?('-') ? repeated_char : ''
+        full_number = ArelExtensions::Nodes::Concat.new([
+            before,
+            sign,
+            middle,
+            number,
+            after
+          ])
+        collector = visit ArelExtensions::Nodes::Concat.new([Arel::Nodes.build_quoted(o.prefix),full_number,Arel::Nodes.build_quoted(o.suffix)]), collector
+        collector
+      end
+
+      def visit_ArelExtensions_Nodes_Std o, collector
+        collector << (o.unbiased_estimator ? "STDDEV_SAMP(" : "STDDEV_POP(")
+        visit o.left, collector
+        collector << ")"
+        collector
+      end
+
+      def visit_ArelExtensions_Nodes_Variance o, collector
+        collector << (o.unbiased_estimator ? "VAR_SAMP(" : "VAR_POP(")
+        visit o.left, collector
+        collector << ")"
+        collector
+      end
+
+      def visit_ArelExtensions_Nodes_LevenshteinDistance o, collector
+        collector << "UTL_MATCH.edit_distance("
+        collector = visit o.left, collector
+        collector << Arel::Visitors::ToSql::COMMA
+        collector = visit o.right, collector
+        collector << ')'
+        collector
+      end
 
     end
   end

data/lib/arel_extensions/visitors/oracle12.rb

@@ -59,7 +59,45 @@ module ArelExtensions
 		  collector << "DECODE("
 		  collector = visit [o.left, o.right, 0, 1], collector
 		  collector << ")"
-		end	 
+		end
+      def visit_ArelExtensions_Nodes_Json o,collector
+        case o.hash
+        when Array
+          collector << 'json_array('
+          o.hash.each.with_index do |v,i|
+            if i != 0
+              collector << Arel::Visitors::MySQL::COMMA
+            end
+            collector  = visit v, collector
+          end
+          collector << ')'
+        when Hash
+          collector << 'json__object('
+          o.hash.each.with_index do |(k,v),i|
+            if i != 0
+              collector << Arel::Visitors::MySQL::COMMA
+            end
+            collector << 'KEY '
+            collector = visit k, collector
+            collector << ' IS '
+            collector = visit v, collector
+            collector << 'FORMAT JSON'
+          end
+          collector << ')'
+        when String,Numeric,TrueClass,FalseClass
+          collector = visit Arel::Nodes.build_quoted("#{o.hash}"), collector
+          collector << ' FORMAT JSON'
+        when NilClass
+          collector  << %Q['null' FORMAT JSON]
+        when Arel::Attributes::Attribute
+          collector = visit o.hash.cast('JSON'), collector
+        else
+          collector = visit o.hash, collector
+          collector << ' FORMAT JSON'
+        end
+        collector
+      end
+
 	end
   end
 end