From 3208e7f182809b1240cd042295287c683917bfca Mon Sep 17 00:00:00 2001 From: Jake Dallimore Date: Tue, 9 May 2023 16:06:14 +0800 Subject: [PATCH] MDL-76842 auth_lti: don't update the user after membership creation This isn't required and generates excess events. The only thing update does, other than update the user fields, is to update the picture, but in this case, that's unset in membership-based auths anyway, so it's entirely safe to remove this. --- auth/lti/auth.php | 1 - 1 file changed, 1 deletion(-) diff --git a/auth/lti/auth.php b/auth/lti/auth.php index 0757570b5ba..dbca92ab8a2 100644 --- a/auth/lti/auth.php +++ b/auth/lti/auth.php @@ -199,7 +199,6 @@ class auth_plugin_lti extends \auth_plugin_base { } } $user = $this->create_new_account($member, $iss); - $this->update_user_account($user, $member, $iss); return \core_user::get_user($user->id); } }