complete deprecation of a bunch of stuff

Author: Mike Dirolf

ext/cbson/cbson.c

@@ -371,10 +371,6 @@ static int write_element_allow_id(VALUE key, VALUE value, VALUE extra, int allow
                 memcpy(buffer->buffer + length_location, &obj_length, 4);
                 break;
             }
-            if (strcmp(cls, "Mongo::Undefined") == 0) {
-                write_name_and_type(buffer, key, 0x0A); // just use nil type
-                break;
-            }
         }
     case T_DATA:
         {

lib/mongo.rb

@@ -2,7 +2,6 @@
 require 'mongo/types/dbref'
 require 'mongo/types/objectid'
 require 'mongo/types/regexp_of_holding'
-require 'mongo/types/undefined'
 
 require 'mongo/errors'
 require 'mongo/connection'
@@ -18,37 +17,3 @@ module Mongo
 
   VERSION = "0.14.1"
 end
-
-# DEPRECATED - the XGen namespace is deprecated and will be removed - use Mongo or GridFS instead
-MongoCopy = Mongo
-module XGen
-  require 'mongo/gridfs'
-  GridFSCopy = GridFS
-
-  def self.included(other_module)
-    warn "the XGen module is deprecated and will be removed - use Mongo or GridFS instead (included from: #{other_module})"
-  end
-
-  module Mongo
-    include MongoCopy
-
-    def self.included(other_module)
-      warn "the XGen::Mongo module is deprecated and will be removed - use Mongo instead (included from: #{other_module})"
-    end
-
-    module Driver
-      include MongoCopy
-
-      def self.included(other_module)
-        warn "the XGen::Mongo::Driver module is deprecated and will be removed - use Mongo instead (included from: #{other_module})"
-      end
-    end
-    module GridFS
-      include GridFSCopy
-
-      def self.included(other_module)
-        warn "the XGen::Mongo::GridFS module is deprecated and will be removed - use GridFS instead (included from: #{other_module})"
-      end
-    end
-  end
-end

lib/mongo/collection.rb

@@ -148,14 +148,6 @@ def find_one(spec_or_object_id=nil, options={})
       find(spec, options.merge(:limit => -1)).next_object
     end
 
-    # DEPRECATED - use find_one instead
-    #
-    # Find the first record that matches +selector+. See #find.
-    def find_first(selector={}, options={})
-      warn "Collection#find_first is deprecated and will be removed. Please use Collection#find_one instead."
-      find_one(selector, options)
-    end
-
     # Save a document in this collection.
     #
     # If +to_save+ already has an '_id' then an update (upsert) operation
@@ -214,32 +206,6 @@ def clear
       remove({})
     end
 
-    # DEPRECATED - use update(... :upsert => true) instead
-    #
-    # Update records that match +selector+ by applying +obj+ as an update.
-    # If no match, inserts (???).
-    def repsert(selector, obj)
-      warn "Collection#repsert is deprecated and will be removed. Please use Collection#update instead."
-      update(selector, obj, :upsert => true)
-    end
-
-    # DEPRECATED - use update(... :upsert => false) instead
-    #
-    # Update records that match +selector+ by applying +obj+ as an update.
-    def replace(selector, obj)
-      warn "Collection#replace is deprecated and will be removed. Please use Collection#update instead."
-      update(selector, obj)
-    end
-
-    # DEPRECATED - use update(... :upsert => false) instead
-    #
-    # Update records that match +selector+ by applying +obj+ as an update.
-    # Both +selector+ and +modifier_obj+ are required.
-    def modify(selector, modifier_obj)
-      warn "Collection#modify is deprecated and will be removed. Please use Collection#update instead."
-      update(selector, modifier_obj)
-    end
-
     # Update a single document in this collection.
     #
     # :spec :: a hash specifying elements which must be present for
@@ -420,14 +386,8 @@ def options
     end
 
     # Get the number of documents in this collection.
-    #
-    # Specifying a +selector+ is DEPRECATED and will be removed. Please use
-    # find(selector).count() instead.
-    def count(selector=nil)
-      if selector
-        warn "specifying a selector for Collection#count is deprecated and will be removed. Please use Collection.find(selector).count instead."
-      end
-      find(selector || {}).count()
+    def count()
+      find().count()
     end
 
     protected

lib/mongo/connection.rb

@@ -148,12 +148,4 @@ def single_db_command(db_name, cmd)
       end
     end
   end
-
-  class Mongo < Connection
-    def initialize(pair_or_host=nil, port=nil, options={})
-      super(pair_or_host, port, options)
-
-      warn "Mongo::Mongo is deprecated and will be removed - please use Mongo::Connection"
-    end
-  end
 end

lib/mongo/db.rb

@@ -388,12 +388,6 @@ def replace_in_db(collection_name, selector, obj)
       }
     end
 
-    # DEPRECATED - use Collection#update instead
-    def modify_in_db(collection_name, selector, obj)
-      warn "DB#modify_in_db is deprecated and will be removed. Please use Collection#update instead."
-      replace_in_db(collection_name, selector, obj)
-    end
-
     # Update records in +collection_name+ that match +selector+ by
     # applying +obj+ as an update. If no match, inserts (???). Normally
     # called by Collection#repsert.
@@ -405,12 +399,6 @@ def repsert_in_db(collection_name, selector, obj)
       }
     end
 
-    # DEPRECATED - use Collection.find(selector).count() instead
-    def count(collection_name, selector={})
-      warn "DB#count is deprecated and will be removed. Please use Collection.find(selector).count instead."
-      collection(collection_name).find(selector).count()
-    end
-
     # Dereference a DBRef, getting the document it points to.
     def dereference(dbref)
       collection(dbref.namespace).find_one("_id" => dbref.object_id)

lib/mongo/types/undefined.rb

@@ -21,7 +21,6 @@
 require 'mongo/types/dbref'
 require 'mongo/types/objectid'
 require 'mongo/types/regexp_of_holding'
-require 'mongo/types/undefined'
 
 # A BSON seralizer/deserializer.
 class BSON
@@ -539,8 +538,6 @@ def bson_type(o)
       OBJECT
     when Symbol
       SYMBOL
-    when Undefined
-      NULL
     else
       raise "Unknown type of object: #{o.class.name}"
     end

lib/mongo/util/bson.rb

@@ -194,13 +194,6 @@ def test_binary_byte_buffer
     assert_equal Binary::SUBTYPE_BYTES, bin2.subtype
   end
 
-  def test_undefined
-    doc = {'undef' => Undefined.new}
-    @b.serialize(doc)
-    doc2 = @b.deserialize
-    assert_equal nil, doc2['undef']
-  end
-
   def test_put_id_first
     val = OrderedHash.new
     val['not_id'] = 1

test/test_bson.rb

@@ -110,9 +110,6 @@ def test_count
     @@test.save("x" => 1)
     @@test.save("x" => 2)
     assert_equal 2, @@test.count
-
-    # TODO remove this test - it's deprecated
-    assert_equal 1, @@test.count("x" => 1)
   end
 
   def test_find_one

test/test_collection.rb

@@ -99,28 +99,3 @@ def test_pair
     assert_equal ['foo', 123], pair[1]
   end
 end
-
-# Test for deprecated Mongo class
-class TestMongo < Test::Unit::TestCase
-
-  include Mongo
-
-  def setup
-    @host = ENV['MONGO_RUBY_DRIVER_HOST'] || 'localhost'
-    @port = ENV['MONGO_RUBY_DRIVER_PORT'] || Mongo::DEFAULT_PORT
-    @mongo = Mongo.new(@host, @port)
-  end
-
-  def test_database_info
-    @mongo.drop_database('ruby-mongo-info-test')
-    @mongo.db('ruby-mongo-info-test').collection('info-test').insert('a' => 1)
-
-    info = @mongo.database_info
-    assert_not_nil info
-    assert_kind_of Hash, info
-    assert_not_nil info['ruby-mongo-info-test']
-    assert info['ruby-mongo-info-test'] > 0
-
-    @mongo.drop_database('ruby-mongo-info-test')
-  end
-end