Handle submission of all 0 quantities gracefully

README.md

@@ -73,6 +73,9 @@ This plugin has been tested with Craft 2.5 and Craft Commerce 1.0.1187.
 
 ## Changelog
 
+0.0.8 
+* [Fixed] Handle submissions with all zero quantities more gracefully
+
 0.0.7 
 * [Added] Add custom events similar to the standard Commerce add to cart events.
 * [Fixed] Improved the documentation a bit based on recent Slack chats

multiadd/MultiAddPlugin.php

@@ -22,7 +22,7 @@ public function getName()
 
     public function getVersion()
     {
-        return '0.0.7';
+        return '0.0.8';
     }
 
     public function getDeveloper()

multiadd/controllers/MultiAddController.php

@@ -50,7 +50,24 @@ public function actionMultiAdd()
             $errors[] = "No items?";
             craft()->urlManager->setRouteVariables(['error' => 'No items?']);
         } 
+
+        //prevent submission of all 0 qtys
+        $itemsToProcess = false;
+        foreach ($items as $key => $item) {
+            $qty = isset($item['qty']) ? (int)$item['qty'] : 0;
+            if ($qty >0){
+                $itemsToProcess = true;
+                break;
+            }
+        }
+
+        if(!$itemsToProcess){
+            $errors[] = "All items have 0 quantity?";
+            craft()->urlManager->setRouteVariables(['error' => 'No items?']);
+        } 
         else {
+
+
             // Do some cart-adding using our new, faster, rollback-able service
             if (!$errors) {
                 $error = "";

multiadd/services/MultiAdd_CartService.php

@@ -108,7 +108,7 @@ public function multiAddToCart($order, $items, &$error = '')
         //Be bold, be brave, assume success...!
         $success = true;
 
-        //raising event - passes in only the first line item which isn't ideal...
+        //raising event
         $event = new Event($this, [
             'lineItems' => $lineItems,
             'order' => $order,