Fix Dolibarr#35086 do not show TicketMarkedAsClosed when close() failed

Ticket::close() returns >0 on success, 0 on no-op, <0 on error (including when a
custom TICKET_CLOSE trigger blocks the close). Two action handlers in ticket/card.php
relied on PHP's truthiness, so a returned -1 was treated as success, showing the
green TicketMarkedAsClosed message together with the trigger's error. Check >0
explicitly in both confirm_close/confirm_abandon and confirm_public_close.

Signed-off-by: Dolicraft <contact@dolicraft.com>

Author: Dolicraft

htdocs/ticket/card.php

@@ -461,7 +461,7 @@
 			$object->context['contact_id'] = GETPOSTINT('contactid');
 		}
 
-		if ($object->close($user, ($action == "confirm_abandon" ? 1 : 0))) {	// Test on pemrission already done
+		if ($object->close($user, ($action == "confirm_abandon" ? 1 : 0)) > 0) {	// Test on pemrission already done
 			setEventMessages($langs->trans('TicketMarkedAsClosed'), null, 'mesgs');
 
 			$url = 'card.php?track_id=' . GETPOST('track_id', 'alpha');
@@ -478,13 +478,15 @@
 		if ($_SESSION['email_customer'] == $object->origin_email || $_SESSION['email_customer'] == $object->thirdparty->email) {
 			$object->context['contact_id'] = GETPOSTINT('contactid');
 
-			$object->close($user);
+			if ($object->close($user) > 0) {
+				setEventMessages('<div class="confirm">' . $langs->trans('TicketMarkedAsClosed') . '</div>', null, 'mesgs');
 
-			setEventMessages('<div class="confirm">' . $langs->trans('TicketMarkedAsClosed') . '</div>', null, 'mesgs');
-
-			$url = 'card.php?track_id=' . GETPOST('track_id', 'alpha');
-			header("Location: " . $url);
-			exit;
+				$url = 'card.php?track_id=' . GETPOST('track_id', 'alpha');
+				header("Location: " . $url);
+				exit;
+			}
+			setEventMessages($object->error, $object->errors, 'errors');
+			$action = '';
 		} else {
 			setEventMessages($object->error, $object->errors, 'errors');
 			$action = '';