document_hash 0.0.5 → 0.0.6

data/lib/document_hash/callbacks.rb

@@ -3,5 +3,9 @@ module DocumentHash
     def after_change &block
       @after_change = block
     end
+
+    def before_change &block
+      @before_change = block
+    end
   end
 end

data/lib/document_hash/core.rb

@@ -31,8 +31,9 @@ module DocumentHash
         val.__send__ :parent_key=, key;
       end
 
+      val = execute_before_change_callback key,val
       super key, val
-      changed_key key
+      changed_key key, val
     end
 
     def reset!
@@ -55,12 +56,21 @@ module DocumentHash
 
     attr_accessor :parent, :parent_key
 
-    def changed_key key
+    def changed_key key, value
       path = Array(key)
 
-      @after_change.call path if @after_change
+      @after_change.call path, value if @after_change
       changed_attributes << path.first
-      parent.__send__ :changed_key, path.unshift(parent_key) if parent
+      parent.__send__ :changed_key, path.unshift(parent_key), value if parent
+    end
+
+    def execute_before_change_callback key, value
+      path = Array(key)
+
+      value = @before_change.call path, value if @before_change
+      value = parent.__send__ :execute_before_change_callback, path.unshift(parent_key), value if parent
+
+      value
     end
 
     def changed_attributes 

data/lib/document_hash/version.rb

@@ -1,3 +1,3 @@
 module DocumentHash
-  VERSION = "0.0.5"
+  VERSION = "0.0.6"
 end

data/spec/lib/document_spec.rb

@@ -91,10 +91,10 @@ describe DocumentHash::Core do
   it "triggers the after change block" do
     subject = DocumentHash::Core.new
     test_mock = mock("test")
-    test_mock.should_receive(:callback_mock).with([:test])
+    test_mock.should_receive(:callback_mock).with([:test], "value")
 
-    subject.after_change do |path|
-      test_mock.callback_mock path
+    subject.after_change do |path, value|
+      test_mock.callback_mock path, value
     end
 
     subject["test"] = "value"
@@ -103,12 +103,36 @@ describe DocumentHash::Core do
   it "receives the right path when multilevel" do
     subject = DocumentHash::Core[ { inner: { attribute: "value" } } ]
     test_mock = mock("test")
-    test_mock.should_receive(:callback_mock).with([:inner, :attribute])
+    test_mock.should_receive(:callback_mock).with([:inner, :attribute], "hello")
 
-    subject.after_change do |path|
-      test_mock.callback_mock path
+    subject.after_change do |path, value|
+      test_mock.callback_mock path, value
     end
 
     subject[:inner][:attribute] = "hello"
   end
+
+  it "triggers a callback before change" do
+    subject = DocumentHash::Core[ { inner: { attribute: "value" } } ]
+    test_mock = mock("test")
+    test_mock.should_receive(:callback_mock).with([:inner, :attribute], "hello")
+
+    subject.before_change do |path, value|
+      test_mock.callback_mock path, value
+    end
+
+    subject[:inner][:attribute] = "hello"
+  end
+
+  it "overrides the value being written by the before_change callback" do
+    subject = DocumentHash::Core[ { inner: { attribute: "value" } } ]
+
+    subject.before_change do |path, value|
+      "hola" 
+    end
+
+    subject[:inner][:attribute] = "hello"
+    subject[:inner][:attribute].should == "hola"
+  end
+
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: document_hash
 version: !ruby/object:Gem::Version
-  version: 0.0.5
+  version: 0.0.6
   prerelease: 
 platform: ruby
 authors: