From 52d523fa9cea75dd138cf9a055a5b7e01d4f4023 Mon Sep 17 00:00:00 2001
From: Yannick Forget <yannickforget@live.nl>
Date: Mon, 3 Oct 2022 08:40:35 +0200
Subject: [PATCH] MDL-61880 core: Retain OAuth 2.0 user field mappings after
 issuer update

---
 lib/classes/oauth2/api.php | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/lib/classes/oauth2/api.php b/lib/classes/oauth2/api.php
index 4eb1eb8513b..c6aece1b0e8 100644
--- a/lib/classes/oauth2/api.php
+++ b/lib/classes/oauth2/api.php
@@ -332,15 +332,15 @@ class api {
         // Will throw exceptions on validation failures.
         if ($create) {
             $issuer->create();
+
+            // Perform service discovery.
+            $classname = self::get_service_classname($issuer->get('servicetype'));
+            $classname::discover_endpoints($issuer);
+            self::guess_image($issuer);
         } else {
             $issuer->update();
         }
 
-        // Perform service discovery.
-        $classname = self::get_service_classname($issuer->get('servicetype'));
-        $classname::discover_endpoints($issuer);
-        self::guess_image($issuer);
-
         return $issuer;
     }