mongoid_acts_as_tree 0.1.0 → 0.1.1

data/CHANGELOG

@@ -0,0 +1,17 @@
+*0.1.1 (April 5th, 2010)*
+
+*Imroved #children.delete()
+
+*Add #children.clear, #children=, #children.replace
+
+*Ruby1.8.7p174 compatible
+
+*Ruby1.9.1p243 compatible
+
+
+*0.1.0 (April 4th, 2010)*
+
+*Support for Mongoid instead of mongo-mapper
+
+*Proxy object #children added with methods: <<, delete
+

data/README.rdoc

@@ -16,7 +16,7 @@ Enable the tree functionality by declaring acts_as_tree on your model
     include Mongoid::Document
     include Mongoid::Acts::Tree
 
-    field :name, String
+    field :name, :type => String
 
     acts_as_tree
   end

data/VERSION

@@ -1 +1 @@
-0.1.0
+0.1.1

data/lib/mongoid_acts_as_tree.rb

@@ -108,6 +108,15 @@ module Mongoid
         	Children.new self
         end
 
+        def children=(new_children_list)
+        	self.children.clear
+        	new_children_list.each do | child |
+        		self.children << child
+        	end
+        end
+
+				alias replace children=
+
         def descendants
           return [] if new_record?
           self.class.where(path_field => self._id).order_by tree_order
@@ -145,7 +154,7 @@ module Mongoid
 
           if @_will_move
             @_will_move = false
-            for child in self.children
+            self.children.each do | child |
               child.fix_position
               child.save
             end
@@ -160,17 +169,14 @@ module Mongoid
 
 			#proxy class
 			class Children < Array
+				#TODO: improve accessors to options to eliminate object[object.parent_id_field]
 
 				def initialize(owner)
 					@parent = owner
 					self.concat find_children_for_owner.to_a
 				end
 
-				def find_children_for_owner
-					@parent.class.where(@parent.parent_id_field => @parent.id).
-						order_by @parent.tree_order
-				end
-
+				#Add new child to list of object children
 				def <<(object)
 					if object.descendants.include? @parent
 						object.instance_variable_set :@_cyclic, true
@@ -185,20 +191,38 @@ module Mongoid
 					super(object)
 				end
 
+				#Deletes object only from children list.
+				#To delete object use <tt>object.destroy</tt>.
 				def delete(object_or_id)
 					object = case object_or_id
 						when String
-							@parent.class.where(:id => object_or_id)
+							@parent.class.find object_or_id
 						else
 							object_or_id
 					end
 
-					object._parent_id = nil
-					object._parent_ids = (object._parent_ids || []) - [@parent.id]
+					object[object.parent_id_field] = nil
+					object[object.path_field]      = []
+					object[object.depth_field]     = 0
 					object.save
+
 					super(object)
 				end
 
+				#Clear children list
+				def clear
+					self.each do | child |
+						@parent.children.delete child
+					end
+				end
+
+				private
+
+				def find_children_for_owner
+					@parent.class.where(@parent.parent_id_field => @parent.id).
+						order_by @parent.tree_order
+				end
+
 			end
 
       module Fields

data/mongoid_acts_as_tree.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{mongoid_acts_as_tree}
-  s.version = "0.1.0"
+  s.version = "0.1.1"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Jakob Vidmar, Aliaksandr Rahalevich"]
-  s.date = %q{2010-04-04}
+  s.date = %q{2010-04-05}
   s.description = %q{Port of the old, venerable ActsAsTree with a bit of a twist}
   s.email = %q{saksmlz@gmail.com}
   s.extra_rdoc_files = [
@@ -19,6 +19,7 @@ Gem::Specification.new do |s|
   s.files = [
     ".document",
      ".gitignore",
+     "CHANGELOG",
      "LICENSE",
      "README.rdoc",
      "Rakefile",

data/test/test_tree.rb

@@ -27,6 +27,28 @@ class TestMongoidActsAsTree < Test::Unit::TestCase
 			assert child.parent == @root_1
 		end
 
+		should "delete child" do
+			@root_1.children.delete @child_1
+			assert_equal(2, @root_1.children.size)
+			@root_1.children.delete @child_2.id
+			assert_equal(@child_3, @root_1.children.first)
+		end
+
+		should "clear children list" do
+			@root_1.children.clear
+			assert_equal([], @root_1.children)
+		end
+
+		should "replace children list" do
+			new_children_list = [Category.create(:name => "test 1"), Category.create(:name => "test 2")]
+
+			@root_1.children = new_children_list
+			assert_equal(new_children_list, @root_1.children)
+
+			@root_1.children = []
+			assert_equal([], @root_1.children)
+		end
+
     should "have roots" do
       assert eql_arrays?(Category.roots, [@root_1, @root_2])
     end

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 0
   - 1
-  - 0
-  version: 0.1.0
+  - 1
+  version: 0.1.1
 platform: ruby
 authors: 
 - Jakob Vidmar, Aliaksandr Rahalevich
@@ -14,7 +14,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-04-04 00:00:00 +03:00
+date: 2010-04-05 00:00:00 +03:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -57,6 +57,7 @@ extra_rdoc_files:
 files: 
 - .document
 - .gitignore
+- CHANGELOG
 - LICENSE
 - README.rdoc
 - Rakefile