[BUGFIX] Scheduler task fails only on errors, not warnings

Let Scheduler task succeed on warnings
cobwebch · Feb 25, 2023 · 2c13284 · 2c13284
2 parents b8b905e + 178fcaa
commit 2c13284
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/Classes/Task/AutomatedSyncTask.php b/Classes/Task/AutomatedSyncTask.php
@@ -126,19 +126,17 @@ public function execute(): bool
                 );
                 if (count($messages[AbstractMessage::ERROR]) > 0) {
                     $globalStatus = 'ERROR';
-                } elseif (count($messages[AbstractMessage::WARNING]) > 0) {
-                    $globalStatus = 'WARNING';
                 }
                 // Assemble the subject and send the mail
                 $subject = empty($extensionConfiguration['reportSubject']) ? '' : $extensionConfiguration['reportSubject'];
                 $subject .= ' [' . $globalStatus . '] ' . 'Synchronization of table ' . $this->table . ', index ' . $this->index;
                 $importer->getReportingUtility()->sendMail($subject, $reportContent);
             }
         }
-        // If any warning or error happened, throw an exception
+        // If any error happened, throw an exception
         if ($globalStatus !== 'OK') {
             throw new \Exception(
-                'One or more errors or warnings happened. Please consult the log.',
+                'One or more errors happened. Please consult the log.',
                 1258116760
             );
         }