From: Michael <heluecht@pirati.ca>
Date: Sun, 14 May 2017 07:01:38 +0000 (+0000)
Subject: Bugfix: The commit in dba::delete was never executed
X-Git-Url: https://git.mxchange.org/?a=commitdiff_plain;h=05c890f256b6dece64225565790ff2fa02ec6194;p=friendica.git

Bugfix: The commit in dba::delete was never executed
---

diff --git a/include/dba.php b/include/dba.php
index b9d7f71e53..3af8522516 100644
--- a/include/dba.php
+++ b/include/dba.php
@@ -845,7 +845,7 @@ class dba {
 	}
 
 	/**
-	 * @brief Insert a row into a table
+	 * @brief Delete a row from a table
 	 *
 	 * @param string $table Table name
 	 * @param array $param parameter array
@@ -918,7 +918,9 @@ class dba {
 
 		if (!$in_process) {
 			// Now we finalize the process
-			if (!self::$in_transaction) {
+			$do_transaction = !self::$in_transaction;
+
+			if ($do_transaction) {
 				self::transaction();
 			}
 
@@ -932,7 +934,7 @@ class dba {
 					logger(dba::replace_parameters($sql, $command['param']), LOGGER_DATA);
 
 					if (!self::e($sql, $command['param'])) {
-						if (!self::$in_transaction) {
+						if ($do_transaction) {
 							self::rollback();
 						}
 						return false;
@@ -962,7 +964,7 @@ class dba {
 						logger(dba::replace_parameters($sql, $field_values), LOGGER_DATA);
 
 						if (!self::e($sql, $field_values)) {
-							if (!self::$in_transaction) {
+							if ($do_transaction) {
 								self::rollback();
 							}
 							return false;
@@ -970,7 +972,7 @@ class dba {
 					}
 				}
 			}
-			if (!self::$in_transaction) {
+			if ($do_transaction) {
 				self::commit();
 			}
 			return true;
diff --git a/include/enotify.php b/include/enotify.php
index e351f0d115..08129da1a5 100644
--- a/include/enotify.php
+++ b/include/enotify.php
@@ -485,9 +485,7 @@ function notification($params) {
 		);
 		if ($p && (count($p) > 1)) {
 			for ($d = 1; $d < count($p); $d ++) {
-				q("DELETE FROM `notify` WHERE `id` = %d",
-					intval($p[$d]['id'])
-				);
+				dba::delete('notify', array('id' => $p[$d]['id']));
 			}
 
 			// only continue on if we stored the first one