MDL-86023 core_task: Task failures always log a stack trace

Author: Fragonite

lib/classes/cron.php

@@ -430,14 +430,12 @@ public static function run_inner_scheduled_task(\core\task\task_base $task) {
             }
             mtrace('... used ' . display_size(memory_get_peak_usage()) . ' peak memory');
             mtrace('Scheduled task failed: ' . $fullname . ',' . $e->getMessage());
-            if ($CFG->debugdeveloper) {
-                if (!empty($e->debuginfo)) {
-                    mtrace("Debug info:");
-                    mtrace($e->debuginfo);
-                }
-                mtrace("Backtrace:");
-                mtrace(format_backtrace($e->getTrace(), true));
+            if (!empty($e->debuginfo)) {
+                mtrace("Debug info:");
+                mtrace($e->debuginfo);
             }
+            mtrace("Backtrace:");
+            mtrace(format_backtrace($e->getTrace(), true));
             \core\task\manager::scheduled_task_failed($task);
         } finally {
             // Reset debugging if it changed.
@@ -542,14 +540,12 @@ public static function run_inner_adhoc_task(\core\task\adhoc_task $task) {
             }
             mtrace('... used ' . display_size(memory_get_peak_usage()) . ' peak memory');
             mtrace("Adhoc task failed: " . get_class($task) . "," . $e->getMessage());
-            if ($CFG->debugdeveloper) {
-                if (!empty($e->debuginfo)) {
-                    mtrace("Debug info:");
-                    mtrace($e->debuginfo);
-                }
-                mtrace("Backtrace:");
-                mtrace(format_backtrace($e->getTrace(), true));
+            if (!empty($e->debuginfo)) {
+                mtrace("Debug info:");
+                mtrace($e->debuginfo);
             }
+            mtrace("Backtrace:");
+            mtrace(format_backtrace($e->getTrace(), true));
             \core\task\manager::adhoc_task_failed($task);
         } finally {
             // Reset debug level if it changed.