active_cart 0.0.12 → 0.0.13

data/VERSION

@@ -1 +1 @@
-0.0.12
+0.0.13

data/lib/active_cart/cart.rb

@@ -178,11 +178,11 @@ module ActiveCart
   private
     def with_callbacks(type, item, quantity, options, &blk)
       return false unless item.send("before_#{type}".to_sym, quantity, options) if item.respond_to?("before_#{type}".to_sym)
-      return false unless @storage_engine.send("before_#{type}".to_sym, quantity, options) if @storage_engine.respond_to?("before_#{type}".to_sym)
+      return false unless @storage_engine.send("before_#{type}".to_sym, item, quantity, options) if @storage_engine.respond_to?("before_#{type}".to_sym)
       
       yield
       
-      @storage_engine.send("after_#{type}".to_sym, quantity, options) if @storage_engine.respond_to?("after_#{type}".to_sym)
+      @storage_engine.send("after_#{type}".to_sym, item, quantity, options) if @storage_engine.respond_to?("after_#{type}".to_sym)
       item.send("after_#{type}".to_sym, quantity, options) if item.respond_to?("after_#{type}".to_sym)
     end
   end

data/test/unit/cart_test.rb

@@ -138,27 +138,27 @@ class CartTest < ActiveSupport::TestCase
         context 'storage engines' do
           should 'fire the storage engines before_add_to_cart callback on add to cart' do
             item = TestItem.new
-            @cart_storage_engine.expects(:before_add_to_cart).with(1, { :option => 'value' }).returns(true)
+            @cart_storage_engine.expects(:before_add_to_cart).with(item, 1, { :option => 'value' }).returns(true)
             @cart.add_to_cart(item, 1, { :option => 'value' })
             assert_equal 1, @cart.quantity
           end
 
           should 'halt and return false if before_add_to_cart returns false' do
             item = TestItem.new
-            @cart_storage_engine.expects(:before_add_to_cart).with(1,  { :option => 'value' }).returns(false)
+            @cart_storage_engine.expects(:before_add_to_cart).with(item, 1,  { :option => 'value' }).returns(false)
             assert !@cart.add_to_cart(item, 1, { :option => 'value' })
             assert_equal 0, @cart.quantity
           end
 
           should 'fire the storage engines after_add_to_cart callback' do
             item = TestItem.new
-            @cart_storage_engine.expects(:after_add_to_cart).with(1, {})
+            @cart_storage_engine.expects(:after_add_to_cart).with(item, 1, {})
             @cart.add_to_cart(item, 1)
           end
 
           should 'fire the storage engines before_remove_from_cart callback on add to cart' do
             item = TestItem.new
-            @cart_storage_engine.expects(:before_remove_from_cart).with(1, {}).returns(true)
+            @cart_storage_engine.expects(:before_remove_from_cart).with(item, 1, {}).returns(true)
             @cart.remove_from_cart(item, 1)
             assert_equal 0, @cart.quantity
           end
@@ -167,14 +167,14 @@ class CartTest < ActiveSupport::TestCase
             item = TestItem.new
             @cart.add_to_cart(item, 1)
             assert_equal 1, @cart.quantity
-            @cart_storage_engine.expects(:before_remove_from_cart).with(1,  { :option => 'value' }).returns(false)
+            @cart_storage_engine.expects(:before_remove_from_cart).with(item, 1,  { :option => 'value' }).returns(false)
             assert !@cart.remove_from_cart(item, 1, { :option => 'value' })
             assert_equal 1, @cart.quantity
           end
 
           should 'fire the storage engines after_remove_from_cart callback' do
             item = TestItem.new
-            @cart_storage_engine.expects(:after_remove_from_cart).with(1,  { :option => 'value' })
+            @cart_storage_engine.expects(:after_remove_from_cart).with(item, 1,  { :option => 'value' })
             @cart.remove_from_cart(item, 1, { :option => 'value' })
           end
         end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: active_cart
 version: !ruby/object:Gem::Version 
-  version: 0.0.12
+  version: 0.0.13
 platform: ruby
 authors: 
 - Myles Eftos