1.0.11

Task: Fix subtotal always being used when cart conditions on the total bring it to $0

composer.json

@@ -1,6 +1,6 @@
 {
   "name": "wearepixel/laravel-cart",
-  "version": "1.0.9",
+  "version": "1.0.11",
   "description": "A cart implementation for Laravel",
   "keywords": [
     "laravel",

src/Cart.php

@@ -406,9 +406,6 @@ public function removeCartCondition($conditionName)
 
         $conditions->pull($conditionName);
 
-        // reset indexes
-        $conditions = $conditions->values();
-
         $this->saveConditions($conditions);
     }
 
@@ -461,9 +458,6 @@ public function removeItemCondition($itemId, $conditionName)
             }
         }
 
-        // reset indexes
-        $item['conditions'] = array_values($item['conditions']);
-
         $this->update($itemId, [
             'conditions' => $item['conditions'],
         ]);
@@ -604,10 +598,14 @@ public function getTotal(): int|float
             return Helpers::formatValue($subTotal, $this->config['format_numbers'], $this->config);
         }
 
-        $conditionsForTotal->each(function (CartCondition $condition) use ($subTotal, &$total) {
+        $index = 0;
-            $toBeCalculated = $total > 0 ? $total : $subTotal;
+
+        $conditionsForTotal->each(function (CartCondition $condition) use ($subTotal, &$total, &$index) {
+            $toBeCalculated = ($index > 0) ? $total : $subTotal;
 
             $total = $condition->applyCondition($toBeCalculated);
+
+            $index++;
         });
 
         return Helpers::formatValue($total, $this->config['format_numbers'], $this->config);

tests/Unit/CartConditionsTest.php

@@ -10,7 +10,7 @@
     $events->shouldReceive('dispatch');
 
     $this->cart = new Cart(
-        new SessionMock,
+        new SessionMock(),
         $events,
         'cart',
         'SAMPLESESSIONKEY',
@@ -119,7 +119,7 @@
 
         // now we should have only 1 condition left on that item
         expect($this->cart->getConditions())->toHaveCount(1, 'Item should have one condition left');
-        expect($this->cart->getConditions()[0]->getName())->toEqual('Mother\'s Day Gift Pack');
+        expect($this->cart->getConditions()->first()->getName())->toEqual('Mother\'s Day Gift Pack');
     });
 });
 
@@ -1310,7 +1310,7 @@
 
         // now we should have only 1 condition left on that item
         expect($this->cart->get(1)['conditions'])->toHaveCount(1, 'Item should have one condition left');
-        expect($this->cart->get(1)['conditions'][0]->getName())->toEqual('Mother\'s Day Gift Pack');
+        expect($this->cart->get(1)['conditions'][1]->getName())->toEqual('Mother\'s Day Gift Pack');
     });
 
     test('can add item with multiple negative item conditions', function () {