return any errors during update_cron execution

Commifreak · Sep 27, 2023 · 87a78af · 87a78af
1 parent 4e6d310
commit 87a78af
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 4 deletions.
diff --git a/src/include/ABSettings.php b/src/include/ABSettings.php
@@ -235,7 +235,9 @@ public function checkCron() {
             unlink(ABSettings::$pluginDir . '/' . ABSettings::$cronFile);
         }
         // Let dcron know our changes via update_cron
-        exec("update_cron");
+        $out = $code = 0;
+        exec("update_cron", $out, $code);
+        return [$code, $out];
     }
 
 }

diff --git a/src/pages/content/settings.php b/src/pages/content/settings.php
@@ -122,7 +122,7 @@
 $abSettings = new ABSettings();
 
 if ($_POST) {
-    $abSettings->checkCron();
+    list($code, $out) = $abSettings->checkCron();
 }
 
 
@@ -209,6 +209,10 @@
 HTML;
 
 }
+
+if ($code != 0) {
+    echo "<h1>Cron error!</h1><p>" . implode('; ', $out) . "</p>";
+}
 ?>
 
 <form id="abSettingsForm" method="post">

diff --git a/src/scripts/checkCron.php b/src/scripts/checkCron.php
@@ -7,8 +7,14 @@
 echo "Checking cron." . PHP_EOL;
 
 $abSettings = new ABSettings();
-$abSettings->checkCron();
+list($code, $out) = $abSettings->checkCron();
+
+if ($code == 0) {
+    echo "Checking cron succeeded!" . PHP_EOL;
+} else {
+    echo "Error occurred: " . implode('; ', $out);
+}
+
 
-echo "Checking cron succeeded!" . PHP_EOL;
 
 exit(0);