mongoid 3.1.6 → 4.0.2

data/lib/mongoid/extensions/big_decimal.rb

@@ -66,4 +66,4 @@ module Mongoid
 end
 
 ::BigDecimal.__send__(:include, Mongoid::Extensions::BigDecimal)
-::BigDecimal.__send__(:extend, Mongoid::Extensions::BigDecimal::ClassMethods)
+::BigDecimal.extend(Mongoid::Extensions::BigDecimal::ClassMethods)

data/lib/mongoid/extensions/boolean.rb

@@ -1,24 +1,22 @@
 # encoding: utf-8
 module Mongoid
-  module Extensions
-    module Boolean
-      module ClassMethods
+  class Boolean
 
-        # Turn the object from the ruby type we deal with to a Mongo friendly
-        # type.
-        #
-        # @example Mongoize the object.
-        #   Boolean.mongoize("123.11")
-        #
-        # @return [ String ] The object mongoized.
-        #
-        # @since 3.0.0
-        def mongoize(object)
-          evolve(object)
-        end
+    class << self
+
+      # Turn the object from the ruby type we deal with to a Mongo friendly
+      # type.
+      #
+      # @example Mongoize the object.
+      #   Boolean.mongoize("123.11")
+      #
+      # @return [ String ] The object mongoized.
+      #
+      # @since 3.0.0
+      def mongoize(object)
+        ::Boolean.evolve(object)
       end
+      alias :evolve :mongoize
     end
   end
 end
-
-::Boolean.__send__(:extend, Mongoid::Extensions::Boolean::ClassMethods)

data/lib/mongoid/extensions/date.rb

@@ -74,4 +74,4 @@ module Mongoid
 end
 
 ::Date.__send__(:include, Mongoid::Extensions::Date)
-::Date.__send__(:extend, Mongoid::Extensions::Date::ClassMethods)
+::Date.extend(Mongoid::Extensions::Date::ClassMethods)

data/lib/mongoid/extensions/date_time.rb

@@ -12,9 +12,9 @@ module Mongoid
       #
       # @since 3.0.0
       def __mongoize_time__
-        return self if utc? && Mongoid.use_utc?
+        return to_time if utc? && Mongoid.use_utc?
         if Mongoid.use_activesupport_time_zone?
-          in_time_zone(::Time.zone)
+          in_time_zone(::Time.zone).to_time
         else
           time = to_time
           time.respond_to?(:getlocal) ? time.getlocal : time
@@ -70,4 +70,4 @@ module Mongoid
 end
 
 ::DateTime.__send__(:include, Mongoid::Extensions::DateTime)
-::DateTime.__send__(:extend, Mongoid::Extensions::DateTime::ClassMethods)
+::DateTime.extend(Mongoid::Extensions::DateTime::ClassMethods)

data/lib/mongoid/extensions/float.rb

@@ -47,10 +47,11 @@ module Mongoid
             nil
           end
         end
+        alias :demongoize :mongoize
       end
     end
   end
 end
 
 ::Float.__send__(:include, Mongoid::Extensions::Float)
-::Float.__send__(:extend, Mongoid::Extensions::Float::ClassMethods)
+::Float.extend(Mongoid::Extensions::Float::ClassMethods)

data/lib/mongoid/extensions/hash.rb

@@ -40,7 +40,7 @@ module Mongoid
         each_pair do |key, value|
           if key =~ /\$/
             value.each_pair do |_key, _value|
-              value[_key] = mongoize_for(key, klass, _key, _value)
+              value[_key] = (key == "$rename") ? _value.to_s : mongoize_for(key, klass, _key, _value)
             end
             (consolidated[key] ||= {}).merge!(value)
           else
@@ -170,7 +170,7 @@ module Mongoid
         field = klass.fields[key.to_s]
         if field
           val = field.mongoize(value)
-          if Mongoid::Persistence::LIST_OPERATIONS.include?(operator) && field.resizable?
+          if Mongoid::Persistable::LIST_OPERATIONS.include?(operator) && field.resizable?
             val = val.first if !value.is_a?(Array)
           end
           val
@@ -214,4 +214,4 @@ module Mongoid
 end
 
 ::Hash.__send__(:include, Mongoid::Extensions::Hash)
-::Hash.__send__(:extend, Mongoid::Extensions::Hash::ClassMethods)
+::Hash.extend(Mongoid::Extensions::Hash::ClassMethods)

data/lib/mongoid/extensions/integer.rb

@@ -64,4 +64,4 @@ module Mongoid
 end
 
 ::Integer.__send__(:include, Mongoid::Extensions::Integer)
-::Integer.__send__(:extend, Mongoid::Extensions::Integer::ClassMethods)
+::Integer.extend(Mongoid::Extensions::Integer::ClassMethods)

data/lib/mongoid/extensions/object.rb

@@ -104,19 +104,19 @@ module Mongoid
         send(name, *args) if name && respond_to?(name)
       end
 
-      # Get the value for an instance variable or nil if it doesn't exist.
+      # Get the value for an instance variable or false if it doesn't exist.
       #
       # @example Get the value for an instance var.
       #   document.ivar("person")
       #
       # @param [ String ] name The name of the variable.
       #
-      # @return [ Object, nil ] The value or nil.
+      # @return [ Object, false ] The value or false.
       #
       # @since 2.0.0.rc.1
       def ivar(name)
-        if instance_variable_defined?("@#{name}")
-          return instance_variable_get("@#{name}")
+        if instance_variable_defined?("@_#{name}")
+          return instance_variable_get("@_#{name}")
         else
           false
         end
@@ -170,8 +170,8 @@ module Mongoid
       #
       # @since 2.1.0
       def remove_ivar(name)
-        if instance_variable_defined?("@#{name}")
-          return remove_instance_variable("@#{name}")
+        if instance_variable_defined?("@_#{name}")
+          return remove_instance_variable("@_#{name}")
         else
           false
         end
@@ -271,4 +271,4 @@ module Mongoid
 end
 
 ::Object.__send__(:include, Mongoid::Extensions::Object)
-::Object.__send__(:extend, Mongoid::Extensions::Object::ClassMethods)
+::Object.extend(Mongoid::Extensions::Object::ClassMethods)

data/lib/mongoid/extensions/object_id.rb

@@ -8,7 +8,7 @@ module Mongoid
       # @example Evolve the object id.
       #   object_id.__evolve_object_id__
       #
-      # @return [ Moped::BSON::ObjectId ] self.
+      # @return [ BSON::ObjectId ] self.
       #
       # @since 3.0.0
       def __evolve_object_id__
@@ -25,7 +25,7 @@ module Mongoid
         #
         # @param [ Object ] object The object to evolve.
         #
-        # @return [ Moped::BSON::ObjectId ] The object id.
+        # @return [ BSON::ObjectId ] The object id.
         #
         # @since 3.0.0
         def evolve(object)
@@ -39,7 +39,7 @@ module Mongoid
         #
         # @param [ Object ] object The object to convert.
         #
-        # @return [ Moped::BSON::ObjectId ] The object id.
+        # @return [ BSON::ObjectId ] The object id.
         #
         # @since 3.0.0
         def mongoize(object)
@@ -50,5 +50,5 @@ module Mongoid
   end
 end
 
-Moped::BSON::ObjectId.__send__(:include, Mongoid::Extensions::ObjectId)
-Moped::BSON::ObjectId.__send__(:extend, Mongoid::Extensions::ObjectId::ClassMethods)
+BSON::ObjectId.__send__(:include, Mongoid::Extensions::ObjectId)
+BSON::ObjectId.extend(Mongoid::Extensions::ObjectId::ClassMethods)

data/lib/mongoid/extensions/range.rb

@@ -53,7 +53,7 @@ module Mongoid
         #
         # @since 3.0.0
         def demongoize(object)
-          object.nil? ? nil : ::Range.new(object["min"], object["max"])
+          object.nil? ? nil : ::Range.new(object["min"], object["max"], object["exclude_end"])
         end
 
         # Turn the object from the ruby type we deal with to a Mongo friendly
@@ -68,7 +68,13 @@ module Mongoid
         #
         # @since 3.0.0
         def mongoize(object)
-          object.nil? ? nil : { "min" => object.first, "max" => object.last }
+          return nil if object.nil?
+          return object if object.is_a?(::Hash)
+          hash = { "min" => object.first, "max" => object.last }
+          if object.respond_to?(:exclude_end?) && object.exclude_end?
+            hash.merge!("exclude_end" => true)
+          end
+          hash
         end
       end
     end
@@ -76,4 +82,4 @@ module Mongoid
 end
 
 ::Range.__send__(:include, Mongoid::Extensions::Range)
-::Range.__send__(:extend, Mongoid::Extensions::Range::ClassMethods)
+::Range.extend(Mongoid::Extensions::Range::ClassMethods)

data/lib/mongoid/extensions/regexp.rb

@@ -24,4 +24,4 @@ module Mongoid
   end
 end
 
-::Regexp.__send__(:extend, Mongoid::Extensions::Regexp::ClassMethods)
+::Regexp.extend(Mongoid::Extensions::Regexp::ClassMethods)

data/lib/mongoid/extensions/set.rb

@@ -52,4 +52,4 @@ module Mongoid
 end
 
 ::Set.__send__(:include, Mongoid::Extensions::Set)
-::Set.__send__(:extend, Mongoid::Extensions::Set::ClassMethods)
+::Set.extend(Mongoid::Extensions::Set::ClassMethods)

data/lib/mongoid/extensions/string.rb

@@ -11,7 +11,7 @@ module Mongoid
       # @example Evolve the string.
       #   "test".__evolve_object_id__
       #
-      # @return [ String, Moped::BSON::ObjectId ] The evolved string.
+      # @return [ String, BSON::ObjectId ] The evolved string.
       #
       # @since 3.0.0
       def __evolve_object_id__
@@ -23,7 +23,7 @@ module Mongoid
       # @example Evolve the string.
       #   "test".__mongoize_object_id__
       #
-      # @return [ String, Moped::BSON::ObjectId, nil ] The mongoized string.
+      # @return [ String, BSON::ObjectId, nil ] The mongoized string.
       #
       # @since 3.0.0
       def __mongoize_object_id__
@@ -96,18 +96,6 @@ module Mongoid
         delete("=").sub(/\_before\_type\_cast$/, '')
       end
 
-      # Convert the string to an array with the string in it.
-      #
-      # @example Convert the string to an array.
-      #   "Testing".to_a
-      #
-      # @return [ Array ] An array with only the string in it.
-      #
-      # @since 1.0.0
-      def to_a
-        [ self ]
-      end
-
       # Is this string a writer?
       #
       # @example Is the string a setter method?
@@ -129,7 +117,7 @@ module Mongoid
       #
       # @since 3.0.15
       def valid_method_name?
-        /[@$"]/ !~ to_sym.inspect
+        /[@$"]/ !~ self
       end
 
       # Does the string end with _before_type_cast?
@@ -165,11 +153,11 @@ module Mongoid
       # @example Convert to the object id.
       #   string.convert_to_object_id
       #
-      # @return [ String, Moped::BSON::ObjectId ] The string or the id.
+      # @return [ String, BSON::ObjectId ] The string or the id.
       #
       # @since 3.0.0
       def convert_to_object_id
-        Moped::BSON::ObjectId.legal?(self) ? Moped::BSON::ObjectId.from_string(self) : self
+        BSON::ObjectId.legal?(self) ? BSON::ObjectId.from_string(self) : self
       end
 
       module ClassMethods
@@ -208,4 +196,4 @@ module Mongoid
 end
 
 ::String.__send__(:include, Mongoid::Extensions::String)
-::String.__send__(:extend, Mongoid::Extensions::String::ClassMethods)
+::String.extend(Mongoid::Extensions::String::ClassMethods)

data/lib/mongoid/extensions/symbol.rb

@@ -51,4 +51,4 @@ module Mongoid
 end
 
 ::Symbol.__send__(:include, Mongoid::Extensions::Symbol)
-::Symbol.__send__(:extend, Mongoid::Extensions::Symbol::ClassMethods)
+::Symbol.extend(Mongoid::Extensions::Symbol::ClassMethods)

data/lib/mongoid/extensions/time.rb

@@ -68,8 +68,8 @@ module Mongoid
           return nil if object.blank?
           begin
             time = object.__mongoize_time__
-            if time.respond_to?(:sec_fraction)
-              ::Time.at(time.to_i, time.sec_fraction * 10**6).utc
+            if object.respond_to?(:sec_fraction)
+              ::Time.at(time.to_i, object.sec_fraction * 10**6).utc
             elsif time.respond_to?(:subsec)
               ::Time.at(time.to_i, time.subsec * 10**6).utc
             else
@@ -85,4 +85,4 @@ module Mongoid
 end
 
 ::Time.__send__(:include, Mongoid::Extensions::Time)
-::Time.__send__(:extend, Mongoid::Extensions::Time::ClassMethods)
+::Time.extend(Mongoid::Extensions::Time::ClassMethods)

data/lib/mongoid/extensions/time_with_zone.rb

@@ -53,4 +53,4 @@ module Mongoid
 end
 
 ::ActiveSupport::TimeWithZone.__send__(:include, Mongoid::Extensions::TimeWithZone)
-::ActiveSupport::TimeWithZone.__send__(:extend, Mongoid::Extensions::TimeWithZone::ClassMethods)
+::ActiveSupport::TimeWithZone.extend(Mongoid::Extensions::TimeWithZone::ClassMethods)

data/lib/mongoid/factory.rb

@@ -15,12 +15,12 @@ module Mongoid
     # @param [ Hash ] options The mass assignment scoping options.
     #
     # @return [ Document ] The instantiated document.
-    def build(klass, attributes = nil, options = {})
+    def build(klass, attributes = nil)
       type = (attributes || {})["_type"]
       if type && klass._types.include?(type)
-        type.constantize.new(attributes, options)
+        type.constantize.new(attributes)
       else
-        klass.new(attributes, options)
+        klass.new(attributes)
       end
     end
 
@@ -32,14 +32,16 @@ module Mongoid
     #
     # @param [ Class ] klass The class to instantiate from if _type is not present.
     # @param [ Hash ] attributes The document attributes.
+    # @param [ Array ] selected_fields If instantiated from a criteria using
+    #   #only we give the document a list of the selected fields.
     #
     # @return [ Document ] The instantiated document.
-    def from_db(klass, attributes = nil, criteria_instance_id = nil)
+    def from_db(klass, attributes = nil, selected_fields = nil)
       type = (attributes || {})["_type"]
       if type.blank?
-        klass.instantiate(attributes, criteria_instance_id)
+        klass.instantiate(attributes, selected_fields)
       else
-        type.camelize.constantize.instantiate(attributes, criteria_instance_id)
+        type.camelize.constantize.instantiate(attributes, selected_fields)
       end
     end
   end

data/lib/mongoid/fields.rb

@@ -10,6 +10,28 @@ module Mongoid
   module Fields
     extend ActiveSupport::Concern
 
+    # For fields defined with symbols use the correct class.
+    #
+    # @since 4.0.0
+    TYPE_MAPPINGS = {
+      array: Array,
+      big_decimal: BigDecimal,
+      binary: BSON::Binary,
+      boolean: Mongoid::Boolean,
+      date: Date,
+      date_time: DateTime,
+      float: Float,
+      hash: Hash,
+      integer: Integer,
+      object_id: BSON::ObjectId,
+      range: Range,
+      regexp: Regexp,
+      set: Set,
+      string: String,
+      symbol: Symbol,
+      time: Time
+    }.with_indifferent_access
+
     included do
       class_attribute :aliased_fields
       class_attribute :localized_fields
@@ -25,15 +47,13 @@ module Mongoid
 
       field(
         :_id,
-        default: ->{ Moped::BSON::ObjectId.new },
+        default: ->{ BSON::ObjectId.new },
         pre_processed: true,
-        type: Moped::BSON::ObjectId
+        type: BSON::ObjectId
       )
 
       alias :id :_id
       alias :id= :_id=
-
-      attr_protected(:id, :_id, :_type) if Mongoid.protect_sensitive_fields?
     end
 
     # Apply all default values to the document which are not procs.
@@ -264,13 +284,13 @@ module Mongoid
         add_field(name, fields[name].options.merge(type: type))
       end
 
-      # Convenience method for determining if we are using +Moped::BSON::ObjectIds+ as
+      # Convenience method for determining if we are using +BSON::ObjectIds+ as
       # our id.
       #
       # @example Does this class use object ids?
       #   person.using_object_ids?
       #
-      # @return [ true, false ] If the class uses Moped::BSON::ObjectIds for the id.
+      # @return [ true, false ] If the class uses BSON::ObjectIds for the id.
       #
       # @since 1.0.0
       def using_object_ids?
@@ -434,10 +454,11 @@ module Mongoid
       def create_field_setter(name, meth, field)
         generated_methods.module_eval do
           re_define_method("#{meth}=") do |value|
-            write_attribute(name, value)
+            val = write_attribute(name, value)
             if field.foreign_key?
               remove_ivar(field.metadata.name)
             end
+            val
           end
         end
       end
@@ -472,8 +493,9 @@ module Mongoid
       def create_translations_getter(name, meth)
         generated_methods.module_eval do
           re_define_method("#{meth}_translations") do
-            attributes[name] ||= {}
+            (attributes[name] ||= {}).with_indifferent_access
           end
+          alias_method :"#{meth}_t", :"#{meth}_translations"
         end
       end
 
@@ -498,6 +520,7 @@ module Mongoid
             end
             attributes[name] = value
           end
+          alias_method :"#{meth}_t=", :"#{meth}_translations="
         end
       end
 
@@ -532,10 +555,20 @@ module Mongoid
 
       def field_for(name, options)
         opts = options.merge(klass: self)
+        type_mapping = TYPE_MAPPINGS[options[:type]]
+        opts[:type] = type_mapping || unmapped_type(options)
         return Fields::Localized.new(name, opts) if options[:localize]
         return Fields::ForeignKey.new(name, opts) if options[:identity]
         Fields::Standard.new(name, opts)
       end
+
+      def unmapped_type(options)
+        if "Boolean" == options[:type].to_s
+          Mongoid::Boolean
+        else
+          options[:type] || Object
+        end
+      end
     end
   end
 end