Improved error handling for inventory equipments, fermentables, hops, miscs, suppliers, water and yeasts.

Sun, 04 Jul 2021 14:25:52 +0200

author
Michiel Broek <mbroek@mbse.eu>
date
Sun, 04 Jul 2021 14:25:52 +0200
changeset 767
08c0343b622b
parent 766
86289b0c4a27
child 768
ae1195153fa2

Improved error handling for inventory equipments, fermentables, hops, miscs, suppliers, water and yeasts.

www/includes/db_inventory_equipments.php file | annotate | diff | comparison | revisions
www/includes/db_inventory_fermentables.php file | annotate | diff | comparison | revisions
www/includes/db_inventory_hops.php file | annotate | diff | comparison | revisions
www/includes/db_inventory_miscs.php file | annotate | diff | comparison | revisions
www/includes/db_inventory_suppliers.php file | annotate | diff | comparison | revisions
www/includes/db_inventory_water.php file | annotate | diff | comparison | revisions
www/includes/db_inventory_yeasts.php file | annotate | diff | comparison | revisions
www/js/inv_equipments.js file | annotate | diff | comparison | revisions
www/js/inv_fermentables.js file | annotate | diff | comparison | revisions
www/js/inv_hops.js file | annotate | diff | comparison | revisions
www/js/inv_miscs.js file | annotate | diff | comparison | revisions
www/js/inv_suppliers.js file | annotate | diff | comparison | revisions
www/js/inv_waters.js file | annotate | diff | comparison | revisions
www/js/inv_yeasts.js file | annotate | diff | comparison | revisions
--- a/www/includes/db_inventory_equipments.php	Sun Jul 04 12:16:25 2021 +0200
+++ b/www/includes/db_inventory_equipments.php	Sun Jul 04 14:25:52 2021 +0200
@@ -57,7 +57,7 @@
 	if (! $result) {
 		syslog(LOG_NOTICE, "db_inventory_equipment: ".$sql." result: ".mysqli_error($connect));
 		$response['error'] = true;
-		$response['msg'] = "SQL fout: ".mysqli_error($link);
+		$response['msg'] = "SQL fout: ".mysqli_error($connect);
 	}
 	exit(json_encode($response));
 
@@ -68,7 +68,7 @@
 	if (! $result) {
                 syslog(LOG_NOTICE, "db_inventory_equipment: ".$sql." result: ".mysqli_error($connect));
                 $response['error'] = true;
-		$response['msg'] = "SQL fout: ".mysqli_error($link);
+		$response['msg'] = "SQL fout: ".mysqli_error($connect);
 		exit(json_encode($response));
 	}
 	if ($row = mysqli_fetch_array($result, MYSQLI_ASSOC)) {
@@ -87,7 +87,7 @@
 	if (! $result) {
 		syslog(LOG_NOTICE, "db_inventory_equipment: ".$sql." result: ".mysqli_error($connect));
 		$response['error'] = true;
-		$response['msg'] = "SQL fout: ".mysqli_error($link);
+		$response['msg'] = "SQL fout: ".mysqli_error($connect);
 	}
 	exit(json_encode($response));
 
--- a/www/includes/db_inventory_fermentables.php	Sun Jul 04 12:16:25 2021 +0200
+++ b/www/includes/db_inventory_fermentables.php	Sun Jul 04 14:25:52 2021 +0200
@@ -12,6 +12,11 @@
 }
 mysqli_set_charset($connect, "utf8" );
 
+$response = array(
+   'error' => false,
+   'msg' => 'Ok',
+);
+
 // get data and store in a json array
 if (isset($_POST['insert']) || isset($_POST['update'])) {
 	if (isset($_POST['insert'])) {
@@ -57,20 +62,13 @@
 	if (isset($_POST['update'])) {
 		$sql .= " WHERE record='" . $_POST['record'] . "';";
 	}
-	//syslog(LOG_NOTICE, $sql);
-
 	$result = mysqli_query($connect, $sql);
 	if (! $result) {
 		syslog(LOG_NOTICE, "db_inventory_fermentables: ".$sql." result: ".mysqli_error($connect));
-	//} else {
-	//	if (isset($_POST['update'])) {
-	//		syslog(LOG_NOTICE, "db_inventory_fermentables: updated record ".$_POST['record']);
-	//	} else {
-	//		$lastid = mysqli_insert_id($connect);
-	//		syslog(LOG_NOTICE, "db_inventory_fermentables: inserted record ".$lastid);
-	//	}
+		$response['error'] = true;
+		$response['msg'] = "SQL fout: ".mysqli_error($connect);
 	}
-	echo $result;
+        exit(json_encode($response));
 
 } else if (isset($_POST['delete'])) {
 	// DELETE COMMAND
@@ -78,10 +76,10 @@
 	$result = mysqli_query($connect, $sql);
 	if (! $result) {
 		syslog(LOG_NOTICE, "db_inventory_fermentables: ".$sql." result: ".mysqli_error($connect));
-	} else {
-		syslog(LOG_NOTICE, "db_inventory_fermentables: deleted record ".$_POST['record']);
-	}
-	echo $result;
+		$response['error'] = true;
+                $response['msg'] = "SQL fout: ".mysqli_error($connect);
+        }
+	exit(json_encode($response));
 
 } else {
 	// SELECT COMMAND
@@ -117,6 +115,12 @@
 		);
 	}
 	header("Content-type: application/json");
-	echo json_encode($fermentables);
+	exit(json_encode($fermentables));
 }
+
+syslog(LOG_NOTICE, "db_inventory_fermentables: missing arguments");
+$response['error'] = true;
+$response['msg'] = "missing arguments";
+echo json_encode($response);
+
 ?>
--- a/www/includes/db_inventory_hops.php	Sun Jul 04 12:16:25 2021 +0200
+++ b/www/includes/db_inventory_hops.php	Sun Jul 04 14:25:52 2021 +0200
@@ -12,6 +12,11 @@
 }
 mysqli_set_charset($connect, "utf8" );
 
+$response = array(
+   'error' => false,
+   'msg' => 'Ok',
+);
+
 if (isset($_POST['insert']) || isset($_POST['update'])) {
 	if (isset($_POST['insert'])) {
 		$sql  = "INSERT INTO `inventory_hops` SET ";
@@ -51,20 +56,13 @@
 	if (isset($_POST['update'])) {
 		$sql .= "' WHERE record='" . $_POST['record'] . "';";
 	}
-	//syslog(LOG_NOTICE, $sql);
-
 	$result = mysqli_query($connect, $sql);
 	if (! $result) {
 		syslog(LOG_NOTICE, "db_inventory_hops: ".$sql." result: ".mysqli_error($connect));
-	//} else {
-	//	if (isset($_POST['update'])) {
-	//		syslog(LOG_NOTICE, "db_inventory_hops: updated record ".$_POST['record']);
-	//	} else {
-	//		$lastid = mysqli_insert_id($connect);
-	//		syslog(LOG_NOTICE, "db_inventory_hops: inserted record ".$lastid);
-	//	}
-	}
-	echo $result;
+                $response['error'] = true;
+                $response['msg'] = "SQL fout: ".mysqli_error($connect);
+        }
+	exit(json_encode($response));
 
 } else if (isset($_POST['delete'])) {
 	// DELETE COMMAND
@@ -72,10 +70,10 @@
 	$result = mysqli_query($connect, $sql);
 	if (! $result) {
 		syslog(LOG_NOTICE, "db_inventory_hops: ".$sql." result: ".mysqli_error($connect));
-	} else {
-		syslog(LOG_NOTICE, "db_inventory_hops: deleted record ".$_POST['record']);
-	}
-	echo $result;
+                $response['error'] = true;
+                $response['msg'] = "SQL fout: ".mysqli_error($connect);
+        }
+	exit(json_encode($response));
 
 } else {
 	// SELECT COMMAND
@@ -106,6 +104,12 @@
 		);
 	}
 	header("Content-type: application/json");
-	echo json_encode($hops);
+	exit(json_encode($hops));
 }
+
+syslog(LOG_NOTICE, "db_inventory_hops: missing arguments");
+$response['error'] = true;
+$response['msg'] = "missing arguments";
+echo json_encode($response);
+
 ?>
--- a/www/includes/db_inventory_miscs.php	Sun Jul 04 12:16:25 2021 +0200
+++ b/www/includes/db_inventory_miscs.php	Sun Jul 04 14:25:52 2021 +0200
@@ -11,6 +11,11 @@
 }
 mysqli_set_charset($connect, "utf8" );
 
+$response = array(
+   'error' => false,
+   'msg' => 'Ok',
+);
+
 if (isset($_POST['insert']) || isset($_POST['update'])) {
 	if (isset($_POST['insert'])) {
 		$sql  = "INSERT INTO `inventory_miscs` SET ";
@@ -44,20 +49,13 @@
 	if (isset($_POST['update'])) {
 		$sql .= " WHERE record='" . $_POST['record'] . "';";
 	}
-	//syslog(LOG_NOTICE, $sql);
-
 	$result = mysqli_query($connect, $sql);
 	if (! $result) {
 		syslog(LOG_NOTICE, "db_inventory_miscs: ".$sql." result: ".mysqli_error($connect));
-	//} else {
-	//	if (isset($_POST['update'])) {
-	//		syslog(LOG_NOTICE, "db_inventory_miscs: updated record ".$_POST['record']);
-	//	} else {
-	//		$lastid = mysqli_insert_id($connect);
-	//		syslog(LOG_NOTICE, "db_inventory_miscs: inserted record ".$lastid);
-	//	}
-	}
-	echo $result;
+                $response['error'] = true;
+                $response['msg'] = "SQL fout: ".mysqli_error($connect);
+        }
+        exit(json_encode($response));
 
 } else if (isset($_POST['delete'])) {
 	// DELETE COMMAND
@@ -65,10 +63,10 @@
 	$result = mysqli_query($connect, $sql);
 	if (! $result) {
 		syslog(LOG_NOTICE, "db_inventory_miscs: ".$sql." result: ".mysqli_error($connect));
-	} else {
-		syslog(LOG_NOTICE, "db_inventory_miscs: deleted record ".$_POST['record']);
-	}
-	echo $result;
+                $response['error'] = true;
+                $response['msg'] = "SQL fout: ".mysqli_error($connect);
+        }
+	exit(json_encode($response));
 
 } else {
 	// SELECT COMMAND
@@ -92,6 +90,12 @@
 		);
 	}
 	header("Content-type: application/json");
-	echo json_encode($miscs);
+	exit(json_encode($miscs));
 }
+
+syslog(LOG_NOTICE, "db_inventory_miscs: missing arguments");
+$response['error'] = true;
+$response['msg'] = "missing arguments";
+echo json_encode($response);
+
 ?>
--- a/www/includes/db_inventory_suppliers.php	Sun Jul 04 12:16:25 2021 +0200
+++ b/www/includes/db_inventory_suppliers.php	Sun Jul 04 14:25:52 2021 +0200
@@ -10,6 +10,11 @@
 }
 mysqli_set_charset($connect, "utf8" );
 
+$response = array(
+   'error' => false,
+   'msg' => 'Ok',
+);
+
 // get data and store in a json array
 $query = "SELECT * FROM inventory_suppliers ORDER BY name";
 if (isset($_POST['insert'])) {
@@ -27,10 +32,10 @@
 	$result = mysqli_query($connect, $sql);
 	if (! $result) {
 		syslog(LOG_NOTICE, "db_inventory_suppliers: ".$sql." result: ".mysqli_error($connect));
-	//} else {
-	//	syslog(LOG_NOTICE, "db_inventory_suppliers: inserted ".$_POST['name']);
-	}
-	echo $result;
+                $response['error'] = true;
+                $response['msg'] = "SQL fout: ".mysqli_error($connect);
+        }
+	exit(json_encode($response));
 
 } else if (isset($_POST['update'])) {
 	// UPDATE COMMAND
@@ -47,22 +52,21 @@
 	$result = mysqli_query($connect, $sql);
 	if (! $result) {
 		syslog(LOG_NOTICE, "db_inventory_suppliers: ".$sql." result: ".mysqli_error($connect));
-	//} else {
-	//	syslog(LOG_NOTICE, "db_inventory_suppliers: updated record ".$_POST['record']);
-	}
-	echo $result;
+                $response['error'] = true;
+                $response['msg'] = "SQL fout: ".mysqli_error($connect);
+        }
+	exit(json_encode($response));
 
 } else if (isset($_POST['delete'])) {
 	// DELETE COMMAND
-	// FIXME: need to check if the record is in use
 	$sql = "DELETE FROM `inventory_suppliers` WHERE record='".$_POST['record']."';";
 	$result = mysqli_query($connect, $sql);
 	if (! $result) {
 		syslog(LOG_NOTICE, "db_inventory_suppliers: ".$sql." result: ".mysqli_error($connect));
-	} else {
-		syslog(LOG_NOTICE, "db_inventory_suppliers: deleted record ".$_POST['record']);
-	}
-	echo $result;
+                $response['error'] = true;
+                $response['msg'] = "SQL fout: ".mysqli_error($connect);
+        }
+        exit(json_encode($response));
 
 } else {
 	// SELECT COMMAND
@@ -82,6 +86,12 @@
 		);
 	}
 	header("Content-type: application/json");
-	echo json_encode($suppliers);
+	exit(json_encode($suppliers));
 }
+
+syslog(LOG_NOTICE, "db_inventory_suppliers: missing arguments");
+$response['error'] = true;
+$response['msg'] = "missing arguments";
+echo json_encode($response);
+
 ?>
--- a/www/includes/db_inventory_water.php	Sun Jul 04 12:16:25 2021 +0200
+++ b/www/includes/db_inventory_water.php	Sun Jul 04 14:25:52 2021 +0200
@@ -10,6 +10,10 @@
 }
 mysqli_set_charset($connect, "utf8" );
 
+$response = array(
+   'error' => false,
+   'msg' => 'Ok',
+);
 
 if (isset($_POST['insert']) || isset($_POST['update'])) {
 	if (isset($_POST['insert'])) {
@@ -38,20 +42,13 @@
 	if (isset($_POST['update'])) {
 		$sql .= "' WHERE record='" . $_POST['record'] . "';";
 	}
-	//syslog(LOG_NOTICE, $sql);
-
 	$result = mysqli_query($connect, $sql);
 	if (! $result) {
 		syslog(LOG_NOTICE, "db_inventory_waters: ".$sql." result: ".mysqli_error($connect));
-	//} else {
-	//	if (isset($_POST['update'])) {
-	//		syslog(LOG_NOTICE, "db_inventory_waters: updated record ".$_POST['record']);
-	//	} else {
-	//		$lastid = mysqli_insert_id($connect);
-	//		syslog(LOG_NOTICE, "db_inventory_waters: inserted record ".$lastid);
-	//	}
-	}
-	echo $result;
+                $response['error'] = true;
+                $response['msg'] = "SQL fout: ".mysqli_error($connect);
+        }
+        exit(json_encode($response));
 
 } else if (isset($_POST['delete'])) {
 	// DELETE COMMAND
@@ -59,10 +56,10 @@
 	$result = mysqli_query($connect, $sql);
 	if (! $result) {
 		syslog(LOG_NOTICE, "db_inventory_waters: ".$sql." result: ".mysqli_error($connect));
-	} else {
-		syslog(LOG_NOTICE, "db_inventory_waters: deleted record ".$_POST['record']);
-	}
-	echo $result;
+                $response['error'] = true;
+                $response['msg'] = "SQL fout: ".mysqli_error($connect);
+        }
+        exit(json_encode($response));
 
 } else {
 	// SELECT COMMAND
@@ -87,6 +84,12 @@
 		);
 	}
 	header("Content-type: application/json");
-	echo json_encode($waters);
+	exit(json_encode($waters));
 }
+
+syslog(LOG_NOTICE, "db_inventory_water: missing arguments");
+$response['error'] = true;
+$response['msg'] = "missing arguments";
+echo json_encode($response);
+
 ?>
--- a/www/includes/db_inventory_yeasts.php	Sun Jul 04 12:16:25 2021 +0200
+++ b/www/includes/db_inventory_yeasts.php	Sun Jul 04 14:25:52 2021 +0200
@@ -11,6 +11,10 @@
 }
 mysqli_set_charset($connect, "utf8" );
 
+$response = array(
+   'error' => false,
+   'msg' => 'Ok',
+);
 
 // get data and store in a json array
 if (isset($_POST['insert']) || isset($_POST['update'])) {
@@ -63,19 +67,13 @@
 	if (isset($_POST['update'])) {
 		$sql .= " WHERE record='" . $_POST['record'] . "';";
 	}
-
 	$result = mysqli_query($connect, $sql);
 	if (! $result) {
 		syslog(LOG_NOTICE, "db_inventory_yeasts: ".$sql." result: ".mysqli_error($connect));
-	//} else {
-	//	if (isset($_POST['update'])) {
-	//		syslog(LOG_NOTICE, "db_inventory_yeasts: updated record ".$_POST['record']);
-	//	} else {
-	//		$lastid = mysqli_insert_id($connect);
-	//		syslog(LOG_NOTICE, "db_inventory_yeasts: inserted record ".$lastid);
-	//	}
-	}
-	echo $result;
+                $response['error'] = true;
+                $response['msg'] = "SQL fout: ".mysqli_error($connect);
+        }
+        exit(json_encode($response));
 
 } else if (isset($_POST['delete'])) {
 	// DELETE COMMAND
@@ -83,10 +81,10 @@
 	$result = mysqli_query($connect, $sql);
 	if (! $result) {
 		syslog(LOG_NOTICE, "db_inventory_yeasts: ".$sql." result: ".mysqli_error($connect));
-	} else {
-		syslog(LOG_NOTICE, "db_inventory_yeasts: deleted record ".$_POST['record']);
-	}
-	echo $result;
+                $response['error'] = true;
+                $response['msg'] = "SQL fout: ".mysqli_error($connect);
+        }
+        exit(json_encode($response));
 
 } else {
 	// SELECT COMMAND
@@ -128,6 +126,12 @@
 		);
 	}
 	header("Content-type: application/json");
-	echo json_encode($yeasts);
+	exit(json_encode($yeasts));
 }
+
+syslog(LOG_NOTICE, "db_inventory_yeasts: missing arguments");
+$response['error'] = true;
+$response['msg'] = "missing arguments";
+echo json_encode($response);
+
 ?>
--- a/www/js/inv_equipments.js	Sun Jul 04 12:16:25 2021 +0200
+++ b/www/js/inv_equipments.js	Sun Jul 04 14:25:52 2021 +0200
@@ -128,7 +128,7 @@
     cache: false,
     data: data,
     type: 'POST',
-    success: function(data, status, xhr) {
+    success: function(data) {
      if (data.error) {
       console.log('delete: ' + data.msg);
       alert('Fout: ' + data.msg);
@@ -149,7 +149,7 @@
     cache: false,
     data: data,
     type: 'POST',
-    success: function(data, status, xhr) {
+    success: function(data) {
      if (data.error) {
       console.log('insert: ' + data.msg);
       alert('Fout: ' + data.msg);
--- a/www/js/inv_fermentables.js	Sun Jul 04 12:16:25 2021 +0200
+++ b/www/js/inv_fermentables.js	Sun Jul 04 14:25:52 2021 +0200
@@ -1,5 +1,5 @@
 /*****************************************************************************
- * Copyright (C) 2014-2020
+ * Copyright (C) 2014-2021
  *
  * Michiel Broek <mbroek at mbse dot eu>
  *
@@ -90,8 +90,13 @@
     cache: false,
     data: data,
     type: 'POST',
-    success: function(data, status, xhr) {
-     commit(true);
+    success: function(data) {
+     if (data.error) {
+      console.log('delete: ' + data.msg);
+      alert('Fout: ' + data.msg);
+     } else {
+      console.log('delete: success');
+     }
      location.reload(true);
     },
     error: function(jqXHR, textStatus, errorThrown) { commit(false); }
@@ -105,8 +110,13 @@
     cache: false,
     data: data,
     type: 'POST',
-    success: function(data, status, xhr) {
-     commit(true);
+    success: function(data) {
+     if (data.error) {
+      console.log('insert: ' + data.msg);
+      alert('Fout: ' + data.msg);
+     } else {
+      console.log('insert: success');
+     }
      location.reload(true);
     },
     error: function(jqXHR, textStatus, errorThrown) { commit(false); }
@@ -120,8 +130,13 @@
     cache: false,
     data: data,
     type: 'POST',
-    success: function(data, status, xhr) {
-     commit(true);
+    success: function(data) {
+     if (data.error) {
+      console.log('updaterow: ' + data.msg);
+      alert('Fout: ' + data.msg);
+     } else {
+      console.log('updaterow: success');
+     }
      location.reload(true);
     },
     error: function(jqXHR, textStatus, errorThrown) { commit(false); }
--- a/www/js/inv_hops.js	Sun Jul 04 12:16:25 2021 +0200
+++ b/www/js/inv_hops.js	Sun Jul 04 14:25:52 2021 +0200
@@ -1,5 +1,5 @@
 /*****************************************************************************
- * Copyright (C) 2014-2020
+ * Copyright (C) 2014-2021
  *
  * Michiel Broek <mbroek at mbse dot eu>
  *
@@ -85,8 +85,13 @@
     cache: false,
     data: data,
     type: 'POST',
-    success: function(data, status, xhr) {
-     commit(true);
+    success: function(data) {
+     if (data.error) {
+      console.log('delete: ' + data.msg);
+      alert('Fout: ' + data.msg);
+     } else {
+      console.log('delete: success');
+     }
      location.reload(true);
     },
     error: function(jqXHR, textStatus, errorThrown) { commit(false); }
@@ -100,8 +105,13 @@
     cache: false,
     data: data,
     type: 'POST',
-    success: function(data, status, xhr) {
-     commit(true);
+    success: function(data) {
+     if (data.error) {
+      console.log('insert: ' + data.msg);
+      alert('Fout: ' + data.msg);
+     } else {
+      console.log('insert: success');
+     }
      location.reload(true);
     },
     error: function(jqXHR, textStatus, errorThrown) { commit(false); }
@@ -115,8 +125,13 @@
     cache: false,
     data: data,
     type: 'POST',
-    success: function(data, status, xhr) {
-     commit(true);
+    success: function(data) {
+     if (data.error) {
+      console.log('updaterow: ' + data.msg);
+      alert('Fout: ' + data.msg);
+     } else {
+      console.log('updaterow: success');
+     }
      location.reload(true);
     },
     error: function(jqXHR, textStatus, errorThrown) { commit(false); }
--- a/www/js/inv_miscs.js	Sun Jul 04 12:16:25 2021 +0200
+++ b/www/js/inv_miscs.js	Sun Jul 04 14:25:52 2021 +0200
@@ -1,5 +1,5 @@
 /*****************************************************************************
- * Copyright (C) 2014-2020
+ * Copyright (C) 2014-2021
  *
  * Michiel Broek <mbroek at mbse dot eu>
  *
@@ -93,8 +93,13 @@
     cache: false,
     data: data,
     type: 'POST',
-    success: function(data, status, xhr) {
-     commit(true);
+    success: function(data) {
+     if (data.error) {
+      console.log('delete: ' + data.msg);
+      alert('Fout: ' + data.msg);
+     } else {
+      console.log('delete: success');
+     }
      location.reload(true);
     },
     error: function(jqXHR, textStatus, errorThrown) { commit(false); }
@@ -108,8 +113,13 @@
     cache: false,
     data: data,
     type: 'POST',
-    success: function(data, status, xhr) {
-     commit(true);
+    success: function(data) {
+     if (data.error) {
+      console.log('insert: ' + data.msg);
+      alert('Fout: ' + data.msg);
+     } else {
+      console.log('insert: success');
+     }
      location.reload(true);
     },
     error: function(jqXHR, textStatus, errorThrown) { commit(false); }
@@ -123,8 +133,13 @@
     cache: false,
     data: data,
     type: 'POST',
-    success: function(data, status, xhr) {
-     commit(true);
+    success: function(data) {
+     if (data.error) {
+      console.log('updaterow: ' + data.msg);
+      alert('Fout: ' + data.msg);
+     } else {
+      console.log('updaterow: success');
+     }
      location.reload(true);
     },
     error: function(jqXHR, textStatus, errorThrown) { commit(false); }
--- a/www/js/inv_suppliers.js	Sun Jul 04 12:16:25 2021 +0200
+++ b/www/js/inv_suppliers.js	Sun Jul 04 14:25:52 2021 +0200
@@ -1,5 +1,5 @@
 /*****************************************************************************
- * Copyright (C) 2014-2019
+ * Copyright (C) 2014-2021
  *
  * Michiel Broek <mbroek at mbse dot eu>
  *
@@ -84,8 +84,13 @@
     cache: false,
     data: data,
     type: 'POST',
-    success: function(data, status, xhr) {
-     commit(true);
+    success: function(data) {
+     if (data.error) {
+      console.log('delete: ' + data.msg);
+      alert('Fout: ' + data.msg);
+     } else {
+      console.log('delete: success');
+     }
      location.reload(true);
     },
     error: function(jqXHR, textStatus, errorThrown) {
@@ -101,8 +106,13 @@
     cache: false,
     data: data,
     type: 'POST',
-    success: function(data, status, xhr) {
-     commit(true);
+    success: function(data) {
+     if (data.error) {
+      console.log('insert: ' + data.msg);
+      alert('Fout: ' + data.msg);
+     } else {
+      console.log('insert: success');
+     }
      location.reload(true);
     },
     error: function(jqXHR, textStatus, errorThrown) {
@@ -118,8 +128,13 @@
     cache: false,
     data: data,
     type: 'POST',
-    success: function(data, status, xhr) {
-     commit(true);
+    success: function(data) {
+     if (data.error) {
+      console.log('updaterow: ' + data.msg);
+      alert('Fout: ' + data.msg);
+     } else {
+      console.log('updaterow: success');
+     }
      location.reload(true);
     },
     error: function(jqXHR, textStatus, errorThrown) {
--- a/www/js/inv_waters.js	Sun Jul 04 12:16:25 2021 +0200
+++ b/www/js/inv_waters.js	Sun Jul 04 14:25:52 2021 +0200
@@ -1,5 +1,5 @@
 /*****************************************************************************
- * Copyright (C) 2014-2020
+ * Copyright (C) 2014-2021
  *
  * Michiel Broek <mbroek at mbse dot eu>
  *
@@ -93,8 +93,13 @@
     cache: false,
     data: data,
     type: 'POST',
-    success: function(data, status, xhr) {
-     commit(true);
+    success: function(data) {
+     if (data.error) {
+      console.log('delete: ' + data.msg);
+      alert('Fout: ' + data.msg);
+     } else {
+      console.log('delete: success');
+     }
      location.reload(true);
     },
     error: function(jqXHR, textStatus, errorThrown) { commit(false); }
@@ -108,8 +113,13 @@
     cache: false,
     data: data,
     type: 'POST',
-    success: function(data, status, xhr) {
-     commit(true);
+    success: function(data) {
+     if (data.error) {
+      console.log('insert: ' + data.msg);
+      alert('Fout: ' + data.msg);
+     } else {
+      console.log('insert: success');
+     }
      location.reload(true);
     },
     error: function(jqXHR, textStatus, errorThrown) { commit(false); }
@@ -123,8 +133,13 @@
     cache: false,
     data: data,
     type: 'POST',
-    success: function(data, status, xhr) {
-     commit(true);
+    success: function(data) {
+     if (data.error) {
+      console.log('updaterow: ' + data.msg);
+      alert('Fout: ' + data.msg);
+     } else {
+      console.log('updaterow: success');
+     }
      location.reload(true);
     },
     error: function(jqXHR, textStatus, errorThrown) { commit(false); }
--- a/www/js/inv_yeasts.js	Sun Jul 04 12:16:25 2021 +0200
+++ b/www/js/inv_yeasts.js	Sun Jul 04 14:25:52 2021 +0200
@@ -114,8 +114,13 @@
     cache: false,
     data: data,
     type: 'POST',
-    success: function(data, status, xhr) {
-     commit(true);
+    success: function(data) {
+     if (data.error) {
+      console.log('delete: ' + data.msg);
+      alert('Fout: ' + data.msg);
+     } else {
+      console.log('delete: success');
+     }
      location.reload(true);
     },
     error: function(jqXHR, textStatus, errorThrown) { commit(false); }
@@ -129,8 +134,13 @@
     cache: false,
     data: data,
     type: 'POST',
-    success: function(data, status, xhr) {
-     commit(true);
+    success: function(data) {
+     if (data.error) {
+      console.log('insert: ' + data.msg);
+      alert('Fout: ' + data.msg);
+     } else {
+      console.log('insert: success');
+     }
      location.reload(true);
     },
     error: function(jqXHR, textStatus, errorThrown) { commit(false); }
@@ -144,8 +154,13 @@
     cache: false,
     data: data,
     type: 'POST',
-    success: function(data, status, xhr) {
-     commit(true);
+    success: function(data) {
+     if (data.error) {
+      console.log('updaterow: ' + data.msg);
+      alert('Fout: ' + data.msg);
+     } else {
+      console.log('updaterow: success');
+     }
      location.reload(true);
     },
     error: function(jqXHR, textStatus, errorThrown) { commit(false); }

mercurial