mongoid-list 0.2.0 → 0.3.0

data/lib/mongoid/list/collection.rb

@@ -9,6 +9,7 @@ module Mongoid
         def update_positions!(klass, elements)
           elements.each_with_index do |element, idx|
             id = element.kind_of?(Hash) ? element['id'] : element
+            id = Moped::BSON::ObjectId.from_string(id) if id.is_a?(String)
             klass.collection.find({ _id: id }).update({ '$set' => { position: (idx + 1) } })
           end
         end

data/lib/mongoid/list/version.rb

@@ -1,5 +1,5 @@
 module Mongoid
   module List
-    VERSION = "0.2.0"
+    VERSION = "0.3.0"
   end
 end

data/mongoid-list.gemspec

@@ -7,7 +7,6 @@ require "mongoid/list/version"
 Gem::Specification.new do |s|
   s.name          = 'mongoid-list'
   s.version       = Mongoid::List::VERSION
-  s.date          = '2013-02-06'
   s.authors       = [ 'Dave Krupinski' ]
   s.email         = 'dave@davekrupinski.com'
   s.summary       = 'Simple list behavior for Mongoid'
@@ -18,15 +17,13 @@ Gem::Specification.new do |s|
   s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
   s.require_paths = ["lib"]
 
-  s.add_dependency('mongoid', [ '>= 3.0.0' ])
+  s.add_dependency('mongoid', [ '>= 3.1.0' ])
 
   s.add_development_dependency('rspec', [ '>= 2.13.0' ])
   s.add_development_dependency('guard', [ '>= 1.8.0' ])
   s.add_development_dependency('guard-rspec', [ '>= 2.5.0' ])
   s.add_development_dependency('guard-spork', [ '>= 1.5.0' ])
-  s.add_development_dependency('listen', [ '>= 1.0.0' ])
-  s.add_development_dependency('database_cleaner', [ '~> 0.9.0 ' ])
-
-  s.add_development_dependency('rspecify')
+  s.add_development_dependency('listen', [ '>= 1.1.0' ])
+  s.add_development_dependency('database_cleaner', [ '~> 1.0.0 ' ])
 
 end

data/spec/mongoid/list/collection_spec.rb

@@ -82,6 +82,33 @@ describe Mongoid::List::Collection do
 
   describe "#update_positions_in_list!" do
 
+    context "string Ids" do
+
+      let!(:obj1) { Simple.create }
+      let!(:obj2) { Simple.create }
+      let!(:obj3) { Simple.create }
+
+      before do
+        Simple.update_positions_in_list!([obj2.id.to_s, obj1.id.to_s, obj3.id.to_s])
+      end
+
+      it "should change obj1 from :position of 1 to 2" do
+        obj1.position.should eq 1
+        obj1.reload.position.should eq 2
+      end
+
+      it "should change obj2 from :position of 2 to 1" do
+        obj2.position.should eq 2
+        obj2.reload.position.should eq 1
+      end
+
+      it "should not change obj3 from :position of 3" do
+        obj3.position.should eq 3
+        obj3.reload.position.should eq 3
+      end
+
+    end
+
     context "unscoped" do
 
       let!(:obj1) { Simple.create }

data/spec/mongoid/list/embedded_spec.rb

@@ -116,6 +116,32 @@ describe Mongoid::List::Embedded do
       Container.create!
     end
 
+    context "with String Ids" do
+
+      let!(:obj1) { container.items.create! }
+      let!(:obj2) { container.items.create! }
+      let!(:obj3) { container.items.create! }
+
+      before do
+        container.items.update_positions_in_list!([ obj2.id.to_s, obj1.id.to_s, obj3.id.to_s ])
+      end
+
+      it "should change obj1 from :position of 1 to 2" do
+        obj1.position.should eq 1
+        obj1.reload.position.should eq 2
+      end
+
+      it "should change obj2 from :position of 2 to 1" do
+        obj2.position.should eq 2
+        obj2.reload.position.should eq 1
+      end
+
+      it "should not change obj3 from :position of 3" do
+        obj3.position.should eq 3
+        obj3.reload.position.should eq 3
+      end
+
+    end
     context "unscoped" do
 
       let!(:obj1) { container.items.create! }

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mongoid-list
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-02-06 00:00:00.000000000 Z
+date: 2013-05-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mongoid
@@ -18,7 +18,7 @@ dependencies:
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
-        version: 3.0.0
+        version: 3.1.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -26,7 +26,7 @@ dependencies:
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
-        version: 3.0.0
+        version: 3.1.0
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -98,7 +98,7 @@ dependencies:
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: 1.1.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -106,7 +106,7 @@ dependencies:
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: 1.1.0
 - !ruby/object:Gem::Dependency
   name: database_cleaner
   requirement: !ruby/object:Gem::Requirement
@@ -114,7 +114,7 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 0.9.0
+        version: 1.0.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -122,23 +122,7 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 0.9.0
-- !ruby/object:Gem::Dependency
-  name: rspecify
-  requirement: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
+        version: 1.0.0
 description: 
 email: dave@davekrupinski.com
 executables: []