results process fixes #120

openrightsgroup · Oct 31, 2021 · 47ec19e · 47ec19e
1 parent 8c9b5ad
commit 47ec19e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/api/1.2/libs/email.php b/api/1.2/libs/email.php
@@ -3,7 +3,7 @@
 function sendISPReport($mailname, $name, $email, $network, $url, $message, $report_type, $category, $renderer) {
     if (defined('SEND_EMAIL') && SEND_EMAIL == false) {
         // disable email flag for debug/testing
-        return;
+        return true;
     }
     $msg = new PHPMailer();
     if (FEATURE_EMAIL_TRACKING) {
@@ -43,7 +43,7 @@ function sendISPReport($mailname, $name, $email, $network, $url, $message, $repo
 function sendISPReminder($row, $network, $url, $renderer) {
     if (defined('SEND_EMAIL') && SEND_EMAIL == false) {
         // disable email flag for debug/testing
-        return;
+        return true;
     }
     $msg = new PHPMailer();
 
@@ -107,7 +107,7 @@ function sendBBFCReport($mailname, $name, $email, $network, $original_network, $
 
     if (defined('SEND_EMAIL') && SEND_EMAIL == false) {
         // disable email flag for debug/testing
-        return;
+        return true;
     }
 
    $msg = new PHPMailer();

diff --git a/backend/process_results.php b/backend/process_results.php
@@ -42,7 +42,7 @@
     if ($row['status'] != 'pending') {
         # we might have already seen this result, if there have been multiple results for a url/network since the
         # last reporting run
-        error_log("Report {$row['report_id']} already processed: ${row['status']}.");
+        error_log("Report {$result['report_id']} already processed: ${row['status']}.");
         continue;
     }