Fix #1176: catching all exceptions when sending mails and adding errors to the end report

Author: ghecquet

core/src/plugins/core.mailer/class.AjxpMailer.php

@@ -196,7 +196,7 @@ public function mailConsumeQueue ($action, $httpVars, $fileVars) {
             $subject = ConfService::getMessages()["core.mailer.9"];
 
             $success = 0;
-            $error = 0;
+            $errors = [];
             foreach ($results as $emailType => $recipients) {
 
                 $isHTML = $emailType == "html";
@@ -208,7 +208,6 @@ public function mailConsumeQueue ($action, $httpVars, $fileVars) {
 
                     $body = $this->_buildDigest($workspaces, $emailType);
 
-                    $success++;
                     try {
                         $mailer->sendMail(
                             [$recipient],
@@ -220,9 +219,8 @@ public function mailConsumeQueue ($action, $httpVars, $fileVars) {
                         );
 
                         $success++;
-                    } catch (AJXP_Exception $e) {
-                        $error++;
-                        $logInfo("Failed to send email to " . $recipient . ": " . $e->getMessage());
+                    } catch (Exception $e) {
+                        $errors[] = "Failed to send email to " . $recipient . ": " . $e->getMessage();
                     }
                 }
             }
@@ -236,7 +234,7 @@ public function mailConsumeQueue ($action, $httpVars, $fileVars) {
                 throw new AJXP_Exception($e->getMessage());
             }
 
-            $output = array("report" => "Sent ".$success." emails");
+            $output = array("report" => "Sent ".$success." emails", "errors" => $errors);
             echo json_encode($output);
         }
     }