Merge pull request Dolibarr#32555 from FHenry/20.0_fix_catching_error…

…_on_Notification fix: manage error in Notification trigger
atm-maxime · Jan 7, 2025 · 973b6b8 · 973b6b8
2 parents 5a5c9db + 1d8f9fd
commit 973b6b8
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/htdocs/core/triggers/interface_50_modNotification_Notification.class.php b/htdocs/core/triggers/interface_50_modNotification_Notification.class.php
@@ -94,7 +94,11 @@ public function runTrigger($action, $object, User $user, Translate $langs, Conf
 		dol_syslog("Trigger '".$this->name."' for action '".$action."' launched by ".__FILE__.". id=".$object->id);
 
 		$notify = new Notify($this->db);
-		$notify->send($action, $object);
+		$resultSend = $notify->send($action, $object);
+		if ($resultSend < 0) {
+			$this->errors = array_merge($this->errors, $notify->errors);
+			return $resultSend;
+		}
 
 		return 1;
 	}