11 Feb 2024

1) Edit item - Will cascade update movement, supplier items, delivery items, purchase items.
2) Delete item - Will cascade delete movement, supplier items, delivery items, purchase items.
3) Delete customer - Will cascade delete all related delivery orders, order items, and movement.
4) Delete supplier item > Will cascade delete all related purchase order items and movement.
5) Delete supplier > Will cascade delete all related purchase orders, order items, movement, and supplier items.

Author: code-boxx

README.md

@@ -6,9 +6,10 @@ Storage Boxx is an open-source PHP Inventory Management System. With a built-in
 1) Generate item QR code, built-in QR code scanner.
 2) Create item NFC tags, built-in NFC scanner.
 3) Passwordless login - QR, NFC, Web Authentication (Biometric, PIN, Face Recognition).
-4) Stock level monitoring, push notifications on low stock.
+4) Stock monitoring, push notifications on low stock.
 5) Customers and delivery order management.
-6) Installable progressive web app.
+6) Suppliers and purchase order management.
+7) Installable progressive web app.
 <br><br>
 
 ## :camera: SCREENSHOTS

TODO.txt

@@ -1,15 +1,6 @@
 STORAGE BOXX TO DO / WISH LIST
 ==============================
 
-PRIORITY
-========
-*) Edit/Delete item > deal with cascade effects
-*) Edit/Delete customer > deal with cascade effects
-*) Edit/Delete supplier > deal with cascade effects
-
-
-GOOD TO HAVE
-============
 * Check Item update, add More Info
   * Suppliers 
   * Purchases + Projected In/Out

assets/PAGE-cus.js

@@ -60,12 +60,16 @@ var cus = {
 
   // (F) DELETE CUSTOMER
   //  id : customer id
-  del : id => cb.modal("Please confirm", `Delete customer? This customer will be lost, but orders will remain.`, () => cb.api({
-    mod : "customers", act : "del",
-    data : { id : id },
-    passmsg : "Customer deleted",
-    onpass : cus.list
-  })),
+  del : id => cb.modal(
+    `<i class="icon icon-warning"></i> Delete Customer?`,
+    `<strong class="text-danger">This customer will be removed. All related orders and movement history will be deleted as well.</strong>`,
+    () => cb.api({
+      mod : "customers", act : "del",
+      data : { id : id },
+      passmsg : "Customer deleted",
+      onpass : cus.list
+    })
+  ),
 
   // (H) IMPORT CUSTOMERS
   import : () => im.init({

assets/PAGE-items.js

@@ -76,12 +76,16 @@ var item = {
 
   // (G) DELETE ITEM
   //  sku : item SKU
-  del : sku => cb.modal("Please confirm", `Delete ${sku}? All movement history will be lost!`, () => cb.api({
-    mod : "items", act : "del",
-    data : { sku : sku },
-    passmsg : "Item Deleted",
-    onpass : item.list
-  })),
+  del : sku => cb.modal(
+    `<i class="icon icon-warning"></i> Delete ${sku}?`, 
+    `<strong class="text-danger">All movement history for this item will be deleted, item will also be removed from all orders and suppliers.</strong>`,
+    () => cb.api({
+      mod : "items", act : "del",
+      data : { sku : sku },
+      passmsg : "Item Deleted",
+      onpass : item.list
+    })
+  ),
 
   // (H) IMPORT ITEMS
   import : () => im.init({

assets/PAGE-sup-items.js

@@ -96,7 +96,9 @@ var items = {
 
   // (G) DELETE ITEM
   //  sku : item sku
-  del : sku => cb.modal("Please confirm", `Remove this item from the supplier?`, () => cb.api({
+  del : sku => cb.modal(
+    `<i class="icon icon-warning"></i> Remove Item?`, 
+    `<strong class="text-danger">Item will also be removed from all related PO and movement history.</strong>`, () => cb.api({
     mod : "suppliers", act : "delItem",
     data : { id : items.id, sku : sku },
     passmsg : "Supplier item deleted",

assets/PAGE-sup.js

@@ -60,12 +60,16 @@ var sup = {
 
   // (F) DELETE SUPPLIER
   //  id : supplier id
-  del : id => cb.modal("Please confirm", `Delete supplier? This supplier and its items will be lost!`, () => cb.api({
-    mod : "suppliers", act : "del",
-    data : { id : id },
-    passmsg : "Supplier deleted",
-    onpass : sup.list
-  })),
+  del : id => cb.modal(
+    `<i class="icon icon-warning"></i> Delete Supplier?`, 
+    `<strong class="text-danger">This supplier will be deleted. Along with all related orders, items, and movement history.`,
+    () => cb.api({
+      mod : "suppliers", act : "del",
+      data : { id : id },
+      passmsg : "Supplier deleted",
+      onpass : sup.list
+    })
+  ),
 
   // (G) SUPPLIER ITEMS CSV DOWNLOAD
   csv : id => {

lib/LIB-Customers.php

@@ -22,9 +22,24 @@ function save ($name, $tel, $email, $addr=null, $id=null) {
   }
 
   // (B) DELETE CUSTOMER
+  //  DANGER - CASCADE DELETE!
   //  $id : customer id
   function del ($id) {
+    $this->DB->start();
+    $this->DB->query(
+      "DELETE `item_mvt`
+       FROM `item_mvt`
+       LEFT JOIN `deliveries` USING (`d_id`) 
+       WHERE `cus_id`=?", [$id]
+    );
+    $this->DB->query(
+      "DELETE `deliveries`, `deliveries_items`
+       FROM `deliveries_items`
+       LEFT JOIN `deliveries` USING (`d_id`) 
+       WHERE `cus_id`=?", [$id]
+    );
     $this->DB->delete("customers", "`cus_id`=?", [$id]);
+    $this->DB->end();
     return true;
   }
 

lib/LIB-Items.php

@@ -30,6 +30,7 @@ function save ($sku, $name, $unit, $price=0, $low=0, $desc=null, $osku=null) {
     }
 
     // (A4) UPDATE ITEM
+    // DANGER - CASCADE UPDATE!
     else {
       $this->DB->update(
         "items", ["item_sku", "item_name", "item_desc", "item_unit", "item_price", "item_low"], 
@@ -39,6 +40,7 @@ function save ($sku, $name, $unit, $price=0, $low=0, $desc=null, $osku=null) {
         $this->DB->update("item_mvt", ["item_sku"], "`item_sku`=?", [$sku, $osku]);
         $this->DB->update("suppliers_items", ["item_sku"], "`item_sku`=?", [$sku, $osku]);
         $this->DB->update("deliveries_items", ["item_sku"], "`item_sku`=?", [$sku, $osku]);
+        $this->DB->update("purchases_items", ["item_sku"], "`item_sku`=?", [$sku, $osku]);
       }
     }
 
@@ -60,12 +62,15 @@ function import ($sku, $name, $unit, $price, $low=0, $desc=null) {
   }
 
   // (C) DELETE ITEM
-  // WARNING : STOCK MOVEMENT WILL BE REMOVED AS WELL
+  //  DANGER - CASCADE DELETE!
   //  $sku : item SKU
   function del ($sku) {
     $this->DB->start();
-    $this->DB->delete("items", "`item_sku`=?", [$sku]);
     $this->DB->delete("item_mvt", "`item_sku`=?", [$sku]);
+    $this->DB->delete("suppliers_items", "`item_sku`=?", [$sku]);
+    $this->DB->delete("deliveries_items", "`item_sku`=?", [$sku]);
+    $this->DB->delete("purchases_items", "`item_sku`=?", [$sku]);
+    $this->DB->delete("items", "`item_sku`=?", [$sku]);
     $this->DB->end();
     return true;
   }

lib/LIB-Suppliers.php

@@ -22,11 +22,25 @@ function save ($name, $tel, $email, $addr=null, $id=null) {
   }
 
   // (B) DELETE SUPPLIER
+  //  DANGER - CASCADE DELETE!
   //  $id : supplier id
   function del ($id) {
     $this->DB->start();
-    $this->DB->delete("suppliers", "`sup_id`=?", [$id]);
+    $this->DB->query(
+      "DELETE `item_mvt`
+       FROM `item_mvt`
+       LEFT JOIN `purchases` USING (`p_id`) 
+       WHERE `sup_id`=?", [$id]
+    );
+    $this->DB->query(
+      "DELETE `purchases_items`
+       FROM `purchases_items`
+       LEFT JOIN `purchases` USING (`p_id`) 
+       WHERE `sup_id`=?", [$id]
+    );
+    $this->DB->delete("purchases", "`sup_id`=?", [$id]);
     $this->DB->delete("suppliers_items", "`sup_id`=?", [$id]);
+    $this->DB->delete("suppliers", "`sup_id`=?", [$id]);
     $this->DB->end();
     return true;
   }
@@ -121,10 +135,25 @@ function saveItem ($id, $sku, $ssku, $price, $osku=null) {
   }
 
   // (G) DELETE ITEM FROM SUPPLIER
+  //  DANGER - CASCADE DELETE!
   //  $id : supplier id
   //  $sku : item sku
   function delItem ($id, $sku) {
+    $this->DB->start();
+    $this->DB->query(
+      "DELETE `item_mvt`
+       FROM `item_mvt`
+       LEFT JOIN `purchases` USING (`p_id`) 
+       WHERE `sup_id`=? AND `item_sku`=?", [$id, $sku]
+    );
+    $this->DB->query(
+      "DELETE `purchases_items`
+       FROM `purchases_items`
+       LEFT JOIN `purchases` USING (`p_id`) 
+       WHERE `sup_id`=? AND `item_sku`=?", [$id, $sku]
+    );
     $this->DB->delete("suppliers_items", "`sup_id`=? AND `item_sku`=?", [$id, $sku]);
+    $this->DB->end();
     return true;
   }