scrivito_sdk 1.5.5 → 1.6.0.rc1

data/lib/scrivito/basic_widget.rb

@@ -142,7 +142,8 @@ class BasicWidget
   # @option context [Scrivito::User] :scrivito_user current visitor
   # @return [Widget] the newly created {Scrivito::BasicWidget Widget}
   #
-  # @see Scrivito::BasicWidget.new
+  # @raise [ArgumentError] if called on +Widget+ without +_obj_class+ attribute.
+  #
   # @see Scrivito::AttributeContent::ClassMethods#default_for
   #
   def self.new(attributes = {}, context = {})
@@ -319,7 +320,8 @@ class BasicWidget
     end
   end
 
-  # Returns the entity ({Scrivito::BasicObj} or {Scrivito::BasicWidget}) that references this widget.
+  # Returns the entity ({Scrivito::BasicObj} or {Scrivito::BasicWidget})
+  # that references this widget.
   # @api public
   #
   # @return [Scrivito::AttributeContent] the entity that references this widget.
@@ -365,6 +367,10 @@ class BasicWidget
     end
   end
 
+  # Returns the {Scrivito::BasicObj Obj} to which this widget belongs.
+  # @api public
+  #
+  # @return [Scrivito::BasicObj] - the Obj this widget belongs to.
   def obj
     if @obj
       @obj

data/lib/scrivito/binary.rb

@@ -107,7 +107,18 @@ class Binary
   # accessible hours or days after they have been generated.
   # @return [String] the URL at which this content is available.
   def url
-    find_url('get')
+    blob_data = CmsBackend.find_blob_data(id, access_type, 'get',
+      transformation_definition: transformation_definition)
+    blob_data['url']
+  rescue ClientError => e
+    case e.backend_code
+    when /\Abinary\.unprocessable\.image\.transform\.source\./
+      raise TransformationSourceError.new(e.message, e.backend_code)
+    when 'binary.unprocessable.image.transform.config.invalid'
+      raise TransformationDefinitionError.new(e.message, e.backend_code)
+    else
+      raise e
+    end
   end
 
   def no_cache_url
@@ -311,33 +322,10 @@ class Binary
 
   private
 
-  def public_content?
-    !!@public_content
-  end
-
   def raise_field_not_available(field_name)
     raise ScrivitoError, "#{field_name} is not available for transformed images"
   end
 
-  def headers
-    CmsBackend.find_blob_metadata(id, find_url('head'))
-  end
-
-  def find_url(verb)
-    blob_data = CmsBackend.find_blob_data(id, access_type, verb,
-      transformation_definition: transformation_definition)
-    blob_data['url']
-  rescue ClientError => e
-    case e.backend_code
-    when /\Abinary\.unprocessable\.image\.transform\.source\./
-      raise TransformationSourceError.new(e.message, e.backend_code)
-    when 'binary.unprocessable.image.transform.config.invalid'
-      raise TransformationDefinitionError.new(e.message, e.backend_code)
-    else
-      raise e
-    end
-  end
-
   def access_type
     public? ? 'public_access' : 'private_access'
   end

data/lib/scrivito/binary_rewrite.rb

@@ -24,7 +24,7 @@ module Scrivito
     # @example
     #   Scrivito::BinaryRewrite.enable_for(request) do |binary_url|
     #     binary_url.sub(
-    #       /https:\/\/.+\.scrvt\.com\//,
+    #       "https://scrivito-public-cdn.s3-eu-west-1.amazonaws.com/",
     #       "https://my.reverse.proxy.com/")
     #   end
     #

data/lib/scrivito/binary_routing.rb

@@ -6,7 +6,7 @@ class BinaryRouting < Struct.new(:request, :scrivito_engine)
   end
 
   def binary_obj_url(obj, binary, image_options = {})
-    binary = if image_options.has_key?(:transform)
+    binary = if image_options.key?(:transform)
       apply_custom_transformation(binary, image_options[:transform])
     else
       apply_default_transformation(binary, obj)

data/lib/scrivito/cms_env.rb

@@ -31,7 +31,8 @@ module Scrivito
         if params[:id]
           Obj.find(params[:id])
         elsif params[:permalink].present?
-          self.class.find_permalink_by_param(params[:permalink])
+          permalink =  Array(params[:permalink]).join("/")
+          Obj.find_by_permalink!(permalink)
         else
           Scrivito::RequestHomepage.call(env)
         end
@@ -42,10 +43,5 @@ module Scrivito
     def params
       env["action_dispatch.request.path_parameters"]
     end
-
-    def self.find_permalink_by_param(permalink_param)
-      permalink =  Array(permalink_param).join("/")
-      Obj.find_by_permalink!(permalink)
-    end
   end
 end

data/lib/scrivito/cms_field_tag.rb

@@ -75,6 +75,10 @@ class CmsFieldTag < Struct.new(:view, :tag_name, :obj_or_widget, :editing_option
       options["private-field-widget-inner-tag"] = inner_tag
       options['private-field-widget-allowed-classes'] = build_valid_widget_classes.to_json
       options['private-field-widget-template'] = widget_render_context
+
+      if editing_options[:disable_margins]
+        options['private-field-widget-disable-margins'] = true
+      end
     end
 
     options

data/lib/scrivito/cms_rest_api.rb

@@ -151,7 +151,6 @@ module Scrivito
 
             if response.code.start_with?('4')
               backend_code = error_body['code']
-              message = "'#{specific_output}' on '#{resource_path}'"
               raise ClientError.new(specific_output,
                   http_code: http_code, backend_code: backend_code)
             elsif response.code == '500' && specific_output
@@ -181,6 +180,7 @@ module Scrivito
 
       def upload_file(file, content_type, upload_permission)
         uri = URI.parse(upload_permission['url'])
+        uri.normalize!
 
         File.open(file) do |open_file|
           upload_io = UploadIO.new(open_file, content_type, File.basename(file))

data/lib/scrivito/configuration.rb

@@ -120,7 +120,8 @@ module Scrivito
       #  @example Use Memcached as the second level cache (https://rubygems.org/gems/dalli)
       #
       #   Scrivito.configure do |config|
-      #     config.second_level_cache = ActiveSupport::Cache::DalliStore.new("localhost", "server-downstairs.localnetwork:8229")
+      #     config.second_level_cache = ActiveSupport::Cache::DalliStore.new("localhost",
+      #         "server-downstairs.localnetwork:8229")
       #   end
       #
       def second_level_cache=(cache_store)
@@ -170,7 +171,9 @@ module Scrivito
       # e.g. Content Read Service.
       # @api public
       def ca_file=(path)
-        File.read(path) if path # Try to read the given file and fail if it doesn't exist or is not readable.
+        if path # Try to read the given file and fail if it doesn't exist or is not readable.
+          File.read(path)
+        end
         @ca_file = path
       end
 
@@ -285,8 +288,8 @@ Please use the new scrivito_route api to replicate the legacy routes:
       # {Scrivito::BasicObj#apply_image_transformation? Obj#apply_image_transformation?} returns
       # +true+.
       #
-      # If not changed the default image transformation is an empty transformation (an empty +Hash+).
-      # Set it to +false+ to disabled the default image transformation completely.
+      # If not changed the default image transformation is an empty transformation (an empty
+      # +Hash+). Set it to +false+ to disabled the default image transformation completely.
       #
       # @param [Hash] value the {Scrivito::Binary#transform transformation definition}
       #

data/lib/scrivito/connection_manager.rb

@@ -152,7 +152,7 @@ module Scrivito
       rescue IO::WaitReadable, IO::WaitWritable
         # these "errors" indicate the connection is still usable
         false
-      rescue => e
+      rescue
         # other errors indicate the connection has a problem
         true
       end

data/lib/scrivito/editing_context.rb

@@ -2,10 +2,15 @@ module Scrivito
 
 class EditingContext
   # +options+:
-  #   +:selected_workspace_id+: The id of the selected workspace. If empty +published+ workspace is used.
-  #   +:display_mode+: Can be +view+, +editing+, +added+ or +deleted+. If empty +view+ is used.
-  #   +:editor+: A block, which is lazy evaluated on first access, to determine the editor (details see {Scrivito::Configuration.editing_auth})
-  #   +:workspace_changed+: A boolean, determining if the workspace has been changed with this request.
+  #   +:selected_workspace_id+:
+  #     The id of the selected workspace. If empty +published+ workspace is used.
+  #   +:display_mode+:
+  #     Can be +view+, +editing+, +added+ or +deleted+. If empty +view+ is used.
+  #   +:editor+:
+  #     A block, which is lazy evaluated on first access, to determine the editor
+  #     (details see {Scrivito::Configuration.editing_auth})
+  #   +:workspace_changed+:
+  #     A boolean, determining if the workspace has been changed with this request.
   # @param [Hash] options
   def initialize(options = {})
     @selected_workspace_id = options[:selected_workspace_id]

data/lib/scrivito/float_conversion.rb

@@ -1,15 +1,14 @@
 module Scrivito
   module FloatConversion
     FLOAT_STRING_REGEX = /\A-?\d+(\.\d+)?\z/.freeze
-    DEFAULT_VALUE = 0.0
 
     def self.deserialize_from_backend(backend_value)
       case backend_value
       when Integer, Float, FLOAT_STRING_REGEX
         value = backend_value.to_f
-        value.finite? ? value : DEFAULT_VALUE
+        value.finite? ? value : nil
       when nil, String
-        DEFAULT_VALUE
+        nil
       else
         raise Scrivito::InternalError,
             "invalid value #{backend_value} (class: #{backend_value.class})"

data/lib/scrivito/integer_conversion.rb

@@ -1,16 +1,15 @@
 module Scrivito
   module IntegerConversion
     INTEGER_STRING_REGEX = /\A-?\d+\z/.freeze
-    DEFAULT_VALUE = 0
     RANGE = Range.new(-(2**53 - 1), (2**53 - 1)).freeze
 
     def self.deserialize_from_backend(backend_value)
       case backend_value
       when Integer, Float, INTEGER_STRING_REGEX
         value = backend_value.to_i
-        RANGE.include?(value) ? value : DEFAULT_VALUE
+        RANGE.include?(value) ? value : nil
       when nil, String
-        DEFAULT_VALUE
+        nil
       else
         raise Scrivito::InternalError,
             "invalid value #{backend_value} (class: #{backend_value.class})"

data/lib/scrivito/link.rb

@@ -2,8 +2,8 @@ require 'active_model/naming'
 
 module Scrivito
   # This class represents individual external or internal links Scrivito is able to identify and
-  # to process.
-  # To format a link for rendering on an HTML page, use the +scrivito_path+ or +scrivito_url+ methods.
+  # to process. To format a link for rendering on an HTML page, use the +scrivito_path+ or
+  # +scrivito_url+ methods.
   # @api public
   class Link
 

data/lib/scrivito/link_parser.rb

@@ -30,7 +30,7 @@ module Scrivito
         link_params[:url] = uri.to_s
       end
 
-      link_params[:query] = remove_editing_context_params(uri.query)
+      link_params[:query] = remove_editing_context_params(uri.query_values)
       link_params[:fragment] = uri.fragment
 
       Link.new(link_params)
@@ -96,12 +96,12 @@ module Scrivito
     rescue ActionController::RoutingError
     end
 
-    def remove_editing_context_params(query)
-      if query
-        params = Rack::Utils.parse_nested_query(query)
-        params.delete('_scrivito_display_mode')
-        params.delete('_scrivito_workspace_id')
-        params.to_query
+    def remove_editing_context_params(query_values)
+      if query_values
+        query_values.delete('_scrivito_display_mode')
+        query_values.delete('_scrivito_workspace_id')
+        uri = Addressable::URI.new(query_values: query_values)
+        uri.query
       end
     end
   end

data/lib/scrivito/meta_data_collection.rb

@@ -5,7 +5,7 @@ module Scrivito
 #
 # == Available meta data attributes
 # All binaries have the meta attributes +content_length+ and +content_type+, in addition, PDFs
-# and images have attributes specfic to their type.
+# and images have attributes specific to their type.
 #
 # If not specified otherwise, the attributes contain +string+ values.
 #
@@ -59,7 +59,8 @@ class MetaDataCollection
   # Find value of a meta data attribute.
   #
   # @param name [Symbol, String] the name of the meta data attribute.
-  # @return [String, Array, Fixnum, Date, nil] meta data attribute value if found or +nil+ otherwise.
+  # @return [String, Array, Fixnum, Date, nil] meta data attribute value if
+  #     found or +nil+ otherwise.
   def [](name)
     @attributes[name]
   end

data/lib/scrivito/migrations/workspace_lock.rb

@@ -24,7 +24,7 @@ module Scrivito
       end
 
       def exists?
-        File.exists?(filename)
+        File.exist?(filename)
       end
 
       private

data/lib/scrivito/obj_collection.rb

@@ -63,9 +63,11 @@ module Scrivito
       find_by(:permalink, [permalink]).first.min_by(&:id)
     end
 
-    # Returns an {ObjSearchEnumerator} with the given initial subquery consisting of the four arguments.
+    # Returns an {ObjSearchEnumerator} with the given initial subquery consisting of the
+    # four arguments.
     #
-    # Note that +field+ and +value+ may also be arrays for searching several fields or searching for several values.
+    # Note that +field+ and +value+ may also be arrays for searching several fields or searching
+    # for several values.
     #
     # {ObjSearchEnumerator}s can be chained using one of the chainable methods
     # (e.g. {ObjSearchEnumerator#and} and {ObjSearchEnumerator#and_not}).

data/lib/scrivito/obj_create_params_parser.rb

@@ -1,5 +1,11 @@
 module Scrivito
-  class ObjCreateParamsParser < ObjParamsParser
+  class ObjCreateParamsParser < Struct.new(:host, :port, :context)
+    def parse(params)
+      params = params.deep_dup
+      convert_params(params)
+      params
+    end
+
     private
 
     def convert_params(params)
@@ -10,5 +16,23 @@ module Scrivito
         raise ArgumentError, 'Missing "_obj_class" param'
       end
     end
+
+    def convert_field_params(params, attribute_definitions)
+      params.each_pair do |key, value|
+        params[key] = case attribute_definitions[key.to_s].try(:type)
+          when 'binary'     then parse_binary_params(value)
+          when 'html'       then ContentConversion.convert_html_links(value, host, port)
+          when 'link'       then ContentConversion.convert_link(value, host, port)
+          when 'linklist'   then ContentConversion.convert_linklist_urls(value, host, port)
+          when 'date'       then DateConversion.deserialize_from_client(value)
+          else value
+          end
+      end
+    end
+
+    def parse_binary_params(params)
+      return unless params
+      UploadedBinary.new(params)
+    end
   end
 end

data/lib/scrivito/obj_search_enumerator.rb

@@ -2,7 +2,7 @@
 module Scrivito
   # Provides an enumerator for iterating over the results of searches for CMS objects to retrieve
   # instances of these objects. This is achieved through the
-  # {http://ruby-doc.org/core/Enumerable.html +Enumerable+ mixin}, which provides methods such
+  # {https://ruby-doc.org/core/Enumerable.html +Enumerable+ mixin}, which provides methods such
   # as +map+, +select+ or +take+.
   #
   # This enumerator is lazy. If, for example, you are looking for {Scrivito::BasicObj Obj}s whose
@@ -77,7 +77,8 @@ module Scrivito
   #
   # [+:equals+]           The +field+ value needs to be identical to the +value+ of this subquery.
   #
-  #                       Applicable to +string+, +stringlist+, +enum+, +multienum+ and +date+ fields.
+  #                       Applicable to +string+, +stringlist+, +enum+, +multienum+,
+  #                       +float+, +integer+ and +date+ fields.
   #
   #                       Example subquery values:
   #
@@ -115,7 +116,7 @@ module Scrivito
   #
   # === +is_less_than+ and +is_greater_than+
   #
-  # These operators are intended for comparing +date+ values or numerical metadata, for example the width of an image.
+  # These operators are intended for comparing +date+, +integer+, or +float+ values.
   # It only considers attributes of {Scrivito::BasicObj Obj}s and _not_ of {Scrivito::BasicWidget Widget}s.
   # Therefore, {Scrivito::BasicWidget Widget} attributes are not searchable using the +is_less_than+ and +is_greater_than+ operators.
   #
@@ -137,50 +138,84 @@ module Scrivito
   #                       ✔ +Time.new(2000,01,01,00,00,01)+ (is greater than)
   #
   #                       ✘ +Time.new(2000,01,01,00,00,00)+ (equal, not greater than)
-  # === +refers_to+
   #
-  # The +refers_to+ operator searches for CMS objects in which specific attributes
-  # reference specific CMS objects. It works with individual +reference+ or +referencelist+
-  # attributes and with a list of attributes of these types.
+  # For +is_less_than+ and +is_greater_than+, the examples are based on the following float value:
+  # +23.42+
   #
-  # So the operator returns the CMS objects in which given attributes of the +reference+ or
-  # +referencelist+ type reference the given CMS objects.
+  # [+:is_less_than+]     Matches if the field value is less than the subquery numeric value.
   #
-  # Using the "*" wildcard for the attributes to search causes all reference and referencelist
-  # attributes of the searched CMS objects to be taken into account.
+  #                       Example subquery values:
+  #
+  #                       ✔ +23.41+ (is less than)
   #
-  # Note that only attributes directly part of the CMS objects are searched.
-  # Currently, attributes contained in widgets are not searched.
+  #                       ✔ +5+ (is less than)
   #
-  # [+:refers_to+]        Searches for CMS objects referencing given objects by means of specific attributes.
+  #                       ✘ +23.42+ (equal, not less than)
+  #
+  # [+:is_greater_than+]  Matches if the field value is greater than the subquery numeric value.
   #
   #                       Example subquery values:
   #
-  #                       ✔ myObj (an instance of {Scrivito::BasicObj Obj})
+  #                       ✔ +23.43+ (is greater than)
   #
-  #                       ✘ "NameString" (not an instance of {Scrivito::BasicObj Obj})
+  #                       ✔ +42+ (is greater than)
   #
-  #                       ✘ +Time.new(2000,01,01,00,00,00)+ (not an instance of {Scrivito::BasicObj Obj})
+  #                       ✘ +23.42+ (equal, not greater than)
   #
   # === +links_to+
   #
-  # The +links_to+ operator searches for CMS objects containing one or more links pointing to
-  # a specific CMS object.
-  # So the operator returns the CMS objects in which an
-  # +html+, +link+, +linklist+, +reference+ or +referencelist+ attribute links to the CMS object.
+  # The +links_to+ operator searches for CMS objects containing one or more attributes linking to
+  # specific CMS objects. So the operator returns the CMS objects in which at least one +html+,
+  # +link+, +linklist+, +reference+ or +referencelist+ attribute links to specific CMS objects.
+  #
+  # The operator can only be applied to all attributes, so the +"*"+ wildcard _must_ be specified
+  # for the attributes to search. If you want to search specific +reference+ or +referencelist+
+  # attributes, please use the +refers_to+ operator.
+  #
+  # Using +nil+ instead of an instance of {Scrivito::BasicObj Obj} raises an error.
+  #
+  # Note that, in contrast to the +refers_to+ operator, the +links_to+ operator searches the
+  # attributes directly part of the CMS objects _as_ _well_ _as_ the attributes contained in
+  # widgets.
+  #
+  # [+:links_to+]         Searches for CMS objects linking to a specific CMS object.
+  #
+  #                       Example subquery values:
+  #
+  #                       ✔ +my_obj+ (an instance of {Scrivito::BasicObj Obj})
+  #
+  #                       ✔ +[my_obj1, my_obj2]+ (an +Array+ of instances of {Scrivito::BasicObj Obj})
   #
-  # The operator can only be applied to all attributes, so the "*" wild card must be specified for
-  # the attribute to search.
+  #                       ✘ +nil+ (not an instance of {Scrivito::BasicObj Obj})
   #
-  # [+:links_to+]         Searches for one or more linked objects.
+  #                       ✘ "some_string" (not an instance of {Scrivito::BasicObj Obj})
+  #
+  # === +refers_to+
+  #
+  # The +refers_to+ operator searches for CMS objects in which at least one of the specified
+  # +reference+ or +referencelist+ attributes refers to specific CMS objects.
+  #
+  # Using the +"*"+ wildcard for the attributes to search causes all +reference+ and +referencelist+
+  # attributes of the searched CMS objects to be taken into account.
+  #
+  # Using +nil+ instead of {Scrivito::BasicObj Objs} searches for all CMS objects in which none of
+  # the specified attributes refer to a CMS object.
+  #
+  # Note that, in contrast to the +links_to+ operator, the +refers_to+ operator only searches
+  # attributes _directly_ _part_ _of_ _the_ _CMS_ _objects_. Currently, attributes contained in
+  # widgets are _not_ searched.
+  #
+  # [+:refers_to+]        Searches for CMS objects in which specific +reference+ or +referencelist+ attributes refer to specific CMS objects.
   #
   #                       Example subquery values:
   #
-  #                       ✔ myObj (an instance of {Scrivito::BasicObj Obj})
+  #                       ✔ +my_obj+ (an instance of {Scrivito::BasicObj Obj})
+  #
+  #                       ✔ +[my_obj1, my_obj2]+ (an +Array+ of instances of {Scrivito::BasicObj Obj})
   #
-  #                       ✘ "NameString" (not an instance of {Scrivito::BasicObj Obj})
+  #                       ✔ +nil+
   #
-  #                       ✘ +Time.new(2000,01,01,00,00,00)+ (not an instance of {Scrivito::BasicObj Obj})
+  #                       ✘ "some_string" (not an instance of {Scrivito::BasicObj Obj})
   #
   # == Matching +multienum+ and +stringlist+
   #
@@ -189,8 +224,15 @@ module Scrivito
   # A search query matches a +multienum+ or +stringlist+, if at least one string in the list matches.
   # Example: A query using the operator +:equals+ and the value +"Eggs"+ matches an Obj containing +["Spam","Eggs"]+ in a +stringlist+ or +multienum+ attribute.
   #
+  # == Limits
+  #
+  # The number of chainable subqueries is limited. The limit depends on the number of values,
+  # fields, and boost parameters requested, as well as the number of words in a free text
+  # search.
+  #
   # @api public
   class ObjSearchEnumerator
+    INVALID_NEGATED_OPERATORS = [:contains, :contains_prefix].freeze
     include Enumerable
 
     attr_reader :workspace
@@ -212,7 +254,8 @@ module Scrivito
     # @param [Symbol, String, Array<Symbol, String>] field Name(s) of the field(s) to be searched.
     #   For arrays, the subquery matches if one or more of these fields meet this criterion.
     # @param [Symbol, String] operator See "Currently available operators" above.
-    # @param [String, Date, Time, Array<String, Date, Time>] value The value(s) to compare with the field value(s) using the
+    # @param [String, Integer, Float, Date, Time, Array<String, Date, Time, Float, Integer>]
+    #   value The value(s) to compare with the field value(s) using the
     #   +operator+ of this subquery. For arrays, the subquery matches if the condition is met for
     #   one or more of the array elements.
     # @param [Hash] boost A hash where the keys are field names and their values are boosting
@@ -222,7 +265,7 @@ module Scrivito
     # @api public
     def and(field, operator, value, boost = nil)
       real_operator = operator_mapping(operator)
-      subquery = {:field => field, :operator => real_operator, :value => convert_value(value)}
+      subquery = { field: field, operator: real_operator, value: convert_value(value) }
       if boost.present?
         valid_boost_operators = [:contains, :contains_prefix]
         if valid_boost_operators.include?(operator.to_sym)
@@ -246,21 +289,20 @@ module Scrivito
     # @param [Symbol, String, Array<Symbol, String>] field Name(s) of the field(s) to be searched.
     #   For arrays, the subquery matches if one or more of these fields meet this criterion.
     # @param [Symbol, String] operator Must be one of: +equals+,
-    #   +starts_with+, +is_greater_than+, +is_less_than+.
+    #   +starts_with+, +is_greater_than+, +is_less_than+, +links_to+, +refers_to+.
     #   (See "Currently available operators" above).
-    # @param [String, Date, Time, Array<String, Date, Time>] value The value(s) to compare with the field value(s) using the
+    # @param [String, Date, Time, Integer, Float Array<String, Date, Time, Integer, Float>] value The value(s) to compare with the field value(s) using the
     #   +operator+ of this subquery. For arrays, the subquery matches if the condition is met for
     #   one or more of the array elements.
     # @return [Scrivito::ObjSearchEnumerator]
     # @api public
     def and_not(field, operator, value)
-      real_operator = operator_mapping(operator)
-      valid_negated_operators = [:equals, :starts_with, :is_greater_than, :is_less_than]
-      unless valid_negated_operators.include?(operator.to_sym)
+      if INVALID_NEGATED_OPERATORS.include?(operator.to_sym)
         raise "Negating operator '#{operator}' is not valid."
       end
-      subquery = {:field => field, :operator => real_operator, :value => convert_value(value),
-          :negate => true}
+      real_operator = operator_mapping(operator)
+      subquery = { field: field, operator: real_operator, value: convert_value(value),
+          negate: true }
       reset_for_changed_query
       @query = (query || []) + [subquery]
 
@@ -269,7 +311,7 @@ module Scrivito
 
     # Orders the results by +field_name+.
     #
-    # Applicable to the attribute types +string+, +enum+ and +date+.
+    # Applicable to the attribute types +string+, +enum+, +integer+, +float+ and +date+.
     #
     # There is a precision limit when sorting string values:
     # Only the first 50 characters of a string are guaranteed to be considered when sorting search results.