searchgasm 1.3.5 → 1.4.0

data/CHANGELOG.rdoc

@@ -1,3 +1,14 @@
+== 1.4.0 released 2008-10-08
+
+* Fixed bug when duping or cloning to copy over instance vars instead of method values
+* Fixed bug with older versions of ActiveRecord and creating an alias method chain on find in AssociationCollection
+* Added modifiers: upper, lower, trim, ltrim, rtrim
+* Added not_nil and not_blank conditions
+* Modified conditions so that table name is a variable, this allows table name to be switched on the fly.
+* Modified auto joins to return symbols instead of SQL. This allows AR to properly merge the joins and ultimately result in proper SQL when performing multiple complex scopes.
+* Fixed conflicts between includes and joins, duplicate joins are not added. Includes are prioritized. This also makes ActiveRecord much more flexible. Now you can cherry pick included associations. And not have to worry about conflicting with joins.
+* Modified ActiveRecord's default behavior to use inner joins with the :joins option. Inner joins impose limitations. When providing this as a "convenience", inner joins do not make sense. The don't allow OR conditions to be across associations, nor does it properly order when an ordering by an association attribute when that association is optional.
+
 == 1.3.5 released 2008-10-08
 
 * Since joins are now strings, AR doesn't know how to merge them properly. So searchgasm now checks which joins to skip over to avoid conflicts.

data/Manifest

@@ -22,10 +22,12 @@ lib/searchgasm/condition/less_than_or_equal_to.rb
 lib/searchgasm/condition/like.rb
 lib/searchgasm/condition/nil.rb
 lib/searchgasm/condition/not_begin_with.rb
+lib/searchgasm/condition/not_blank.rb
 lib/searchgasm/condition/not_end_with.rb
 lib/searchgasm/condition/not_equal.rb
 lib/searchgasm/condition/not_have_keywords.rb
 lib/searchgasm/condition/not_like.rb
+lib/searchgasm/condition/not_nil.rb
 lib/searchgasm/condition/sibling_of.rb
 lib/searchgasm/condition/tree.rb
 lib/searchgasm/conditions/base.rb
@@ -60,6 +62,8 @@ lib/searchgasm/modifiers/hour.rb
 lib/searchgasm/modifiers/log.rb
 lib/searchgasm/modifiers/log10.rb
 lib/searchgasm/modifiers/log2.rb
+lib/searchgasm/modifiers/lower.rb
+lib/searchgasm/modifiers/ltrim.rb
 lib/searchgasm/modifiers/md5.rb
 lib/searchgasm/modifiers/microseconds.rb
 lib/searchgasm/modifiers/milliseconds.rb
@@ -68,11 +72,14 @@ lib/searchgasm/modifiers/month.rb
 lib/searchgasm/modifiers/octal.rb
 lib/searchgasm/modifiers/radians.rb
 lib/searchgasm/modifiers/round.rb
+lib/searchgasm/modifiers/rtrim.rb
 lib/searchgasm/modifiers/second.rb
 lib/searchgasm/modifiers/sign.rb
 lib/searchgasm/modifiers/sin.rb
 lib/searchgasm/modifiers/square_root.rb
 lib/searchgasm/modifiers/tan.rb
+lib/searchgasm/modifiers/trim.rb
+lib/searchgasm/modifiers/upper.rb
 lib/searchgasm/modifiers/week.rb
 lib/searchgasm/modifiers/year.rb
 lib/searchgasm/search/base.rb
@@ -90,12 +97,11 @@ MIT-LICENSE
 Rakefile
 README.rdoc
 test/fixtures/accounts.yml
+test/fixtures/cats.yml
+test/fixtures/dogs.yml
 test/fixtures/orders.yml
 test/fixtures/user_groups.yml
 test/fixtures/users.yml
-test/libs/acts_as_tree.rb
-test/libs/ordered_hash.rb
-test/libs/rexml_fix.rb
 test/test_active_record_associations.rb
 test/test_active_record_base.rb
 test/test_condition_base.rb
@@ -109,4 +115,7 @@ test/test_search_conditions.rb
 test/test_search_ordering.rb
 test/test_search_pagination.rb
 test/test_search_protection.rb
+test_libs/acts_as_tree.rb
+test_libs/ordered_hash.rb
+test_libs/rexml_fix.rb
 TODO.rdoc

data/README.rdoc

@@ -305,6 +305,11 @@ Here are all of the available modifiers:
   
   :md5                                                          Converts to a MD5
   :char_length      :length                                     The length of the string (integer)
+  :lower            :downcase, :lcase                           Converts the string to all lower case characters
+  :ltrin            :lstrip                                     Strips off spaces from the beginning of the string
+  :trim             :strip                                      Strips off spaces from the beginning and end of the string
+  :rtrim            :rstrip                                     Strips off spaces from the end of the string
+  :upper            :upcase, :ucase                             Converts the string to all upper case character
   
   :absolute         :abs                                        The absolute value (-1 => 1)
   :acos                                                         The arc cosine
@@ -344,6 +349,12 @@ Here's the cool part. Chaining modifiers:
   
 As long as the modifier chain makes sense the possibilities are endless.
 
+=== Modifiers are not indexed
+
+Depending on your database you can create complex indexes. But chances are you probably didn't or don't plan to. So keep in mind that once you use a modifier it will not be using an index, meaning the query will be slower. One of the ways to get the best of both worlds is to cache virtual attributes in the database. Checkout my tutorial:
+
+http://www.binarylogic.com/2008/10/5/tutorial-caching-virtual-attributes-in-the-database
+
 == Roll your own conditions & modifiers
 
 For more information on this please see Searchgasm::Conditions::Base

data/TODO.rdoc

@@ -1,3 +1,6 @@
 = To Do
 
-1. Perform "more efficient" checks: year_of_created_at = 2008 and month_of_created_at = 8. Should result in a "BETWEEN" statement utilizing the column indexes. Thanks Georg for letting me know about this.
+1. Perform "more efficient" checks: year_of_created_at = 2008 and month_of_created_at = 8. Should result in a "BETWEEN" statement utilizing the column indexes. Thanks Georg for letting me know about this.
+2. Solve conflicts between scope joins and joins in the search (for old versions of AR). Also solve conflicts between joins and includes.
+3. Add configuration to change the "english" words
+4. Re-add the distinct option

data/lib/searchgasm/active_record/associations.rb

@@ -13,7 +13,7 @@ module Searchgasm
         end
       end
       
-      module Shared
+      module HasManyAssociation
         def count_with_searchgasm(*args)
           options = args.extract_options!
           args << filter_options_with_searchgasm(options)
@@ -24,18 +24,29 @@ module Searchgasm
   end
 end
 
-module ActiveRecord
-  module Associations
-    class AssociationCollection
-      include Searchgasm::ActiveRecord::Associations::AssociationCollection
-      
-      alias_method_chain :find, :searchgasm
-    end
-    
-    class HasManyAssociation
-      include Searchgasm::ActiveRecord::Associations::Shared
-      
-      alias_method_chain :count, :searchgasm
-    end
+ActiveRecord::Associations::AssociationCollection.class_eval do
+  if respond_to?(:find)
+    include Searchgasm::ActiveRecord::Associations::AssociationCollection
+    alias_method_chain :find, :searchgasm
   end
+end
+
+ActiveRecord::Associations::HasManyAssociation.class_eval do
+  include Searchgasm::ActiveRecord::Associations::HasManyAssociation
+  alias_method_chain :count, :searchgasm
+  
+  # Older versions of AR have find in here, not in AssociationCollection
+  include Searchgasm::ActiveRecord::Associations::AssociationCollection
+  alias_method_chain :find, :searchgasm
+end
+
+ActiveRecord::Associations::ClassMethods::InnerJoinDependency::InnerJoinAssociation.class_eval do
+  private
+    # Inner joins impose limitations on queries. They can be quicker but you can't do OR conditions when conditions
+    # overlap from the base model to any of its associations. Also, inner joins won't allow you to order by an association
+    # attribute. What if the association is optional? All of those records are ommitted. It just doesn't make sense to default
+    # to inner joins when providing this as a "convenience" when searching. So let's change it.
+    def join_type
+      "LEFT OUTER JOIN"
+    end
 end

data/lib/searchgasm/active_record/base.rb

@@ -165,6 +165,55 @@ module ActiveRecord #:nodoc: all
       def valid_calculations_options
         Calculations::CALCULATIONS_OPTIONS
       end
+      
+      private
+        # This is copied over from 2 different versions of ActiveRecord. I have to do this in order to preserve the "auto joins"
+        # as symbols. Keeping them as symbols allows ActiveRecord to merge them properly. The problem is when they conflict with includes.
+        # Includes add joins also, and they add them before joins do. So if they already added them skip them. Now you can do queries like:
+        #
+        # User.all(:joins => {:orders => :line_items}, :include => :orders)
+        #
+        # Where as before, the only way to get the above query to work would be to include line_items also, which is not neccessarily what you want.
+        def add_joins!(sql, options_or_joins, scope = :auto) # :nodoc:
+          code_type = array_of_strings?([""]) && :array_of_strings rescue nil
+          code_type ||= merge_joins("", "") && :merge_joins rescue nil
+
+          case code_type
+          when :array_of_strings, :merge_joins
+            joins = options_or_joins
+            scope = scope(:find) if :auto == scope
+            merged_joins = scope && scope[:joins] && joins ? merge_joins(scope[:joins], joins) : (joins || scope && scope[:joins])
+            case merged_joins
+            when Symbol, Hash, Array
+              if code_type == :array_of_strings && array_of_strings?(merged_joins)
+                merged_joins.each { |merged_join| sql << " #{merged_join} " unless sql.include?(merged_join) }
+              else
+                join_dependency = ActiveRecord::Associations::ClassMethods::InnerJoinDependency.new(self, merged_joins, nil)
+                join_dependency.join_associations.each do |assoc|
+                  join_sql = assoc.association_join
+                  sql << " #{join_sql} " unless sql.include?(join_sql)
+                end
+              end
+            when String
+              sql << " #{merged_joins} " if merged_joins && !sql.include?(merged_joins)
+            end
+          else
+            options = options_or_joins
+            scope = scope(:find) if :auto == scope
+            [(scope && scope[:joins]), options[:joins]].each do |join|
+              case join
+              when Symbol, Hash, Array
+                join_dependency = ActiveRecord::Associations::ClassMethods::InnerJoinDependency.new(self, join, nil)
+                join_dependency.join_associations.each do |assoc|
+                  join_sql = assoc.association_join
+                  sql << " #{join_sql} " unless sql.include?(join_sql)
+                end
+              else
+                sql << " #{join} " if join && !sql.include?(join)
+              end
+            end
+          end
+        end
     end
   end
 end

data/lib/searchgasm/active_record/connection_adapters/mysql_adapter.rb

@@ -52,10 +52,30 @@ module Searchgasm
           "CHAR_LENGTH(#{column_name})"
         end
         
+        def lower_sql(column_name)
+          "LOWER(#{column_name})"
+        end
+        
+        def ltrim_sql(column_name)
+          "LTRIM(#{column_name})"
+        end
+        
         def md5_sql(column_name)
           "MD5(#{column_name})"
         end
         
+        def rtrim_sql(column_name)
+          "RTRIM(#{column_name})"
+        end
+        
+        def trim_sql(column_name)
+          "TRIM(#{column_name})"
+        end
+        
+        def upper_sql(column_name)
+          "UPPER(#{column_name})"
+        end
+        
         # Number functions
         def absolute_sql(column_name)
           "ABS(#{column_name})"

data/lib/searchgasm/active_record/connection_adapters/postgresql_adapter.rb

@@ -52,10 +52,30 @@ module Searchgasm
           "length(#{column_name})"
         end
         
+        def lower_sql(column_name)
+          "lower(#{column_name})"
+        end
+        
+        def ltrim_sql(column_name)
+          "ltrim(#{column_name})"
+        end
+        
         def md5_sql(column_name)
           "md5(#{column_name})"
         end
         
+        def rtrim_sql(column_name)
+          "rtrim(#{column_name})"
+        end
+        
+        def trim_sql(column_name)
+          "trim(#{column_name})"
+        end
+        
+        def upper_sql(column_name)
+          "upper(#{column_name})"
+        end
+        
         # Number functions
         def absolute_sql(column_name)
           "abs(#{column_name})"

data/lib/searchgasm/active_record/connection_adapters/sqlite_adapter.rb

@@ -46,6 +46,27 @@ module Searchgasm
         def year_sql(column_name)
           "strftime('%Y', #{column_name})"
         end
+        
+        # String functions
+        def lower_sql(column_name)
+          "lower(#{column_name})"
+        end
+        
+        def ltrim_sql(column_name)
+          "ltrim(#{column_name})"
+        end
+        
+        def rtrim_sql(column_name)
+          "rtrim(#{column_name})"
+        end
+        
+        def trim_sql(column_name)
+          "trim(#{column_name})"
+        end
+        
+        def upper_sql(column_name)
+          "upper(#{column_name})"
+        end
       end
     end
   end

data/lib/searchgasm/condition/base.rb

@@ -7,7 +7,7 @@ module Searchgasm
     class Base
       include Shared::Utilities
       
-      attr_accessor :column, :column_for_type_cast, :column_sql, :klass
+      attr_accessor :column, :column_for_type_cast, :column_sql, :column_sql_format, :klass, :table_name
       class_inheritable_accessor :handle_array_value, :ignore_meaningless_value, :value_type
       self.ignore_meaningless_value = true
     
@@ -16,7 +16,7 @@ module Searchgasm
         def condition_type_name
           name.split("::").last.underscore
         end
-        
+                
         def handle_array_value?
           handle_array_value == true
         end
@@ -39,16 +39,29 @@ module Searchgasm
         end
       end
     
-      def initialize(klass, column_obj = nil, column_type = nil, column_sql = nil)
+      def initialize(klass, options = {})
         self.klass = klass
-
-        if column_obj
-          self.column = column_obj.class < ::ActiveRecord::ConnectionAdapters::Column ? column_obj : klass.columns_hash[column_obj.to_s]
-          type = (!self.class.value_type.blank? && self.class.value_type.to_s) || (!column_type.blank? && column_type.to_s) || column.sql_type
-          self.column_for_type_cast = column.class.new(column.name, column.default.to_s, type, column.null)
-          self.column_sql = column_sql || "#{klass.connection.quote_table_name(klass.table_name)}.#{klass.connection.quote_column_name(column.name)}"
+        self.table_name = options[:table_name] || klass.table_name
+        
+        if options[:column]
+          self.column = options[:column].class < ::ActiveRecord::ConnectionAdapters::Column ? options[:column] : klass.columns_hash[options[:column].to_s]
+          
+          if options[:column_for_type_cast]
+            self.column_for_type_cast = options[:column_for_type_cast]
+          else
+            type = (!self.class.value_type.blank? && self.class.value_type.to_s) || (!options[:column_type].blank? && options[:column_type].to_s) || column.sql_type
+            self.column_for_type_cast = column.class.new(column.name, column.default.to_s, type, column.null)
+          end
+          
+          self.column_sql_format = options[:column_sql_format] || "{table}.{column}"
         end
       end
+      
+      # Substitutes string vars with table and column name. Allows us to switch the column and table on the fly and have the condition update appropriately.
+      # The table name could be variable depending on the condition. Take STI and more than one child model is used in the condition, the first gets the parent table name, the rest get aliases.
+      def column_sql
+        column_sql_format.gsub("{table}", quoted_table_name).gsub("{column}", quoted_column_name)
+      end
     
       # Allows nils to be meaninful values
       def explicitly_set_value=(value)
@@ -60,6 +73,10 @@ module Searchgasm
         @explicitly_set_value == true
       end
       
+      def options
+        {:table_name => table_name, :column => column, :column_for_type_cast => column_for_type_cast, :column_sql_format => column_sql_format}
+      end
+      
       # You should refrain from overwriting this method, it performs various tasks before callign your to_conditions method, allowing you to keep to_conditions simple.
       def sanitize(alt_value = nil) # :nodoc:
         return if value_is_meaningless?
@@ -107,12 +124,16 @@ module Searchgasm
           klass.connection.quote_column_name(column_name)
         end
         
+        def quoted_column_name
+          quote_column_name(column.name)
+        end
+        
         def quote_table_name(table_name)
           klass.connection.quote_table_name(table_name)
         end
         
         def quoted_table_name
-          quote_table_name(klass.table_name)
+          quote_table_name(table_name)
         end
         
         def type_cast_value(v)

data/lib/searchgasm/condition/inclusive_descendant_of.rb

@@ -2,7 +2,7 @@ module Searchgasm
   module Condition
     class InclusiveDescendantOf < Tree
       def to_conditions(value)
-        condition = DescendantOf.new(klass, column)
+        condition = DescendantOf.new(klass, options)
         condition.value = value
         merge_conditions(["#{quoted_table_name}.#{quote_column_name(klass.primary_key)} = ?", (value.is_a?(klass) ? value.send(klass.primary_key) : value)], condition.sanitize, :any => true)
       end

data/lib/searchgasm/condition/not_begin_with.rb

@@ -8,7 +8,7 @@ module Searchgasm
       end
       
       def to_conditions(value)
-        begin_with = BeginWith.new
+        begin_with = BeginWith.new(klass, options)
         begin_with.value = value
         conditions = being_with.to_conditions
         return conditions if conditions.blank?

data/lib/searchgasm/condition/not_blank.rb

@@ -0,0 +1,17 @@
+module Searchgasm
+  module Condition
+    class NotBlank < Base
+      class << self
+        def condition_names_for_column
+          super + ["is_not_blank"]
+        end
+      end
+      
+      def to_conditions(value)
+        blank = Blank.new(klass, options)
+        blank.value = !value
+        blank.to_conditions
+      end
+    end
+  end
+end

data/lib/searchgasm/condition/not_end_with.rb

@@ -8,7 +8,7 @@ module Searchgasm
       end
       
       def to_conditions(value)
-        ends_with = EndsWith.new
+        ends_with = EndsWith.new(klass, options)
         ends_with.value = value
         conditions = ends_with.to_conditions
         return conditions if conditions.blank?

data/lib/searchgasm/condition/not_equal.rb

@@ -12,7 +12,7 @@ module Searchgasm
       
       def to_conditions(value)
         # Delegate to equals and then change
-        condition = Equals.new(klass, column)
+        condition = Equals.new(klass, options)
         condition.value = value
         
         conditions_array = condition.sanitize

data/lib/searchgasm/condition/not_have_keywords.rb

@@ -8,7 +8,7 @@ module Searchgasm
       end
       
       def to_conditions(value)
-        keywords = Keywords.new
+        keywords = Keywords.new(klass, options)
         keywords.value = value
         conditions = keywords.to_conditions
         return conditions if conditions.blank?

data/lib/searchgasm/condition/not_like.rb

@@ -8,7 +8,7 @@ module Searchgasm
       end
       
       def to_conditions(value)
-        like = Like.new
+        like = Like.new(klass, options)
         like.value = value
         conditions = like.to_conditions
         return conditions if conditions.blank?

data/lib/searchgasm/condition/not_nil.rb

@@ -0,0 +1,17 @@
+module Searchgasm
+  module Condition
+    class NotNil < Base
+      class << self
+        def condition_names_for_column
+          super + ["is_not_nil", "is_not_null", "not_null"]
+        end
+      end
+      
+      def to_conditions(value)
+        is_nil = Nil.new(klass, options)
+        is_nil.value = !value
+        is_nil.to_conditions
+      end
+    end
+  end
+end

data/lib/searchgasm/condition/sibling_of.rb

@@ -5,7 +5,7 @@ module Searchgasm
         parent_association = klass.reflect_on_association(:parent)
         foreign_key_name = (parent_association && parent_association.options[:foreign_key]) || "parent_id"
         parent_id = (value.is_a?(klass) ? value : klass.find(value)).send(foreign_key_name)
-        condition = ChildOf.new(klass, column)
+        condition = ChildOf.new(klass, options)
         condition.value = parent_id
         merge_conditions(["#{quoted_table_name}.#{quote_column_name(klass.primary_key)} != ?", (value.is_a?(klass) ? value.send(klass.primary_key) : value)], condition.sanitize)
       end

data/lib/searchgasm/conditions/base.rb

@@ -179,7 +179,7 @@ module Searchgasm
         associations.each do |name, association|
           next if association.conditions.blank?
           association_joins = association.auto_joins
-          j << (association_joins.blank? ? association.relationship_name.to_sym : {association.relationship_name.to_sym => association_joins})
+          j << (association_joins.blank? ? name : {name => association_joins})
         end
         j.blank? ? nil : (j.size == 1 ? j.first : j)
       end
@@ -233,7 +233,7 @@ module Searchgasm
           if object.class < Searchgasm::Conditions::Base
             relationship_conditions = object.conditions
             next if relationship_conditions.blank?
-            conditions_hash[object.relationship_name.to_sym] = relationship_conditions
+            conditions_hash[name] = relationship_conditions
           else
             next if object.value_is_meaningless?
             conditions_hash[name] = object.value
@@ -367,14 +367,14 @@ module Searchgasm
                 column_type = modifier_klasses.first.return_type
               
                 # Build the column sql
-                column_sql = "#{klass.connection.quote_table_name(klass.table_name)}.#{klass.connection.quote_column_name(column_detail[:column].name)}"
+                column_sql = "{table}.{column}"
                 modifier_klasses.each do |modifier_klass|
                   next unless klass.connection.respond_to?(modifier_klass.adapter_method_name)
                   column_sql = klass.connection.send(modifier_klass.adapter_method_name, column_sql)
                 end
               end
             
-              add_condition!(condition_klass, method_name, column_detail[:column], column_type, column_sql)
+              add_condition!(condition_klass, method_name, :column => column_detail[:column], :column_type => column_type, :column_sql_format => column_sql)
             
               ([column_detail[:column].name] + column_detail[:aliases]).each do |column_name|
                 condition_klass.condition_names_for_column.each do |condition_name|
@@ -393,13 +393,15 @@ module Searchgasm
           false
         end
         
-        def add_condition!(condition, name, column = nil, column_type = nil, column_sql = nil)
+        def add_condition!(condition, name, options = {})
           self.class.condition_names << name
+          options[:column] = options[:column].name if options[:column].class < ::ActiveRecord::ConnectionAdapters::Column
           
           self.class.class_eval <<-"end_eval", __FILE__, __LINE__
             def #{name}_object
               if objects[:#{name}].nil?
-                objects[:#{name}] = #{condition.name}.new(klass, #{column.blank? ? "nil" : "klass.columns_hash['#{column.name}']"}, #{column_type.blank? ? "nil" : "\"#{column_type}\""}, #{column_sql.blank? ? "nil" : "\"#{column_sql.gsub('"', '\"')}\""})
+                options = {}
+                objects[:#{name}] = #{condition.name}.new(klass, #{options.inspect})
               end
               objects[:#{name}]
             end
@@ -465,7 +467,7 @@ module Searchgasm
         end
         
         def reset_objects!
-          objects.each { |name, object| object.class < ::Searchgasm::Conditions::Base ? eval("@#{object.relationship_name} = nil") : eval("@#{name} = nil") }
+          objects.each { |name, object| eval("@#{name} = nil") }
           objects.clear
         end
         

data/lib/searchgasm/conditions/protection.rb

@@ -23,7 +23,7 @@ module Searchgasm
       end
       
       def protect=(value)
-        associations.each { |association| association.protect = value }
+        associations.each { |name, obj| obj.protect = value }
         @protect = value
       end
       

data/lib/searchgasm/helpers/control_types/select.rb

@@ -68,7 +68,7 @@ module Searchgasm
             else
               options[:html][:onchange] += remote_function(:url => url, :method => :get).gsub(/\\'\+this.value\+\\'/, "'+this.value+'")
             end
-            options[:html][:id] ||= ""
+            options[:html][:id] ||= "#{option}_select"
             options
           end
       end

data/lib/searchgasm/helpers/utilities.rb

@@ -105,7 +105,7 @@ module Searchgasm
       # === Options
       # * <tt>:params_scope</tt> -- default: :search, this is the scope in which your search params will be preserved (params[:search]). If you don't want a scope and want your options to be at base leve in params such as params[:page], params[:per_page], etc, then set this to nil.
       # * <tt>:search_obj</tt> -- default: @#{params_scope}, this is your search object, everything revolves around this. It will try to infer the name from your params_scope. If your params_scope is :search it will try to get @search, etc. If it can not be inferred by this, you need to pass the object itself.
-      def searchgasm_state(options)
+      def searchgasm_state(options = {})
         return "" if @added_searchgasm_state
         add_searchgasm_defaults!(options)
         html = ""

data/lib/searchgasm/modifiers/lower.rb

@@ -0,0 +1,15 @@
+module Searchgasm
+  module Modifiers
+    class Lower < Base
+      class << self
+        def modifier_names
+          super + ["downcase", "lcase"]
+        end
+        
+        def return_type
+          :string
+        end
+      end
+    end
+  end
+end

data/lib/searchgasm/modifiers/ltrim.rb

@@ -0,0 +1,15 @@
+module Searchgasm
+  module Modifiers
+    class Ltrim < Base
+      class << self
+        def modifier_names
+          super + ["lstrip"]
+        end
+        
+        def return_type
+          :string
+        end
+      end
+    end
+  end
+end

data/lib/searchgasm/modifiers/rtrim.rb

@@ -0,0 +1,15 @@
+module Searchgasm
+  module Modifiers
+    class Rtrim < Base
+      class << self
+        def modifier_names
+          super + ["rstrip"]
+        end
+        
+        def return_type
+          :string
+        end
+      end
+    end
+  end
+end

data/lib/searchgasm/modifiers/trim.rb

@@ -0,0 +1,15 @@
+module Searchgasm
+  module Modifiers
+    class Trim < Base
+      class << self
+        def modifier_names
+          super + ["strip"]
+        end
+        
+        def return_type
+          :string
+        end
+      end
+    end
+  end
+end