From f6e7eb2ebd24e03a5de1617167b695eb36091a1e Mon Sep 17 00:00:00 2001 From: Tim Hunt Date: Tue, 17 Oct 2023 16:38:40 +0100 Subject: [PATCH] MDL-79747 quiz stats: task should run as admin the right way The correct way to get an ad-hoc task to run as the main admin account is to leave userid as null. Setting it causes fatal errors if the main admin account is set to auth: nologin. --- mod/quiz/report/statistics/classes/task/recalculate.php | 1 - 1 file changed, 1 deletion(-) diff --git a/mod/quiz/report/statistics/classes/task/recalculate.php b/mod/quiz/report/statistics/classes/task/recalculate.php index e750973a7a1..dd2cb144677 100644 --- a/mod/quiz/report/statistics/classes/task/recalculate.php +++ b/mod/quiz/report/statistics/classes/task/recalculate.php @@ -47,7 +47,6 @@ class recalculate extends \core\task\adhoc_task { public static function instance(int $quizid): recalculate { $task = new self(); $task->set_component('quiz_statistics'); - $task->set_userid(get_admin()->id); $task->set_custom_data((object)[ 'quizid' => $quizid, ]);