From 2dcd4f8e78c5a78161bfec5110dd81c2b98e63b2 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); } }