MDL-77933 enrol_lti: return site name and logo during registration

Instead of the Moodle icon and the 'moodle' lang string, return the site fullname and compact logo during dynamic registration.
moodle · Jun 8, 2023 · 5a423b4 · 5a423b4
1 parent 52a4f2d
commit 5a423b4
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/enrol/lti/register.php b/enrol/lti/register.php
@@ -42,7 +42,7 @@
 use enrol_lti\local\ltiadvantage\service\application_registration_service;
 
 require_once(__DIR__."/../../config.php");
-global $OUTPUT, $PAGE, $CFG;
+global $OUTPUT, $PAGE, $CFG, $SITE;
 require_once($CFG->libdir . '/filelib.php');
 
 $PAGE->set_context(context_system::instance());
@@ -113,9 +113,9 @@
         $CFG->wwwroot . '/enrol/lti/launch_deeplink.php',
     ],
      // TODO: Consider whether to support client_name#ja syntax for multi language support - see MDL-73109.
-    'client_name' => get_string('moodle', 'enrol_lti'),
+    'client_name' => format_string($SITE->fullname, true, ['context' => context_system::instance()]),
     'jwks_uri' => $CFG->wwwroot . '/enrol/lti/jwks.php',
-    'logo_uri' => $OUTPUT->image_url('moodlelogo')->out(false),
+    'logo_uri' => $OUTPUT->get_compact_logo_url() ? $OUTPUT->get_compact_logo_url()->out(false) : '',
     'token_endpoint_auth_method' => 'private_key_jwt',
     'scope' => implode(" ", $scopes),
     'https://purl.imsglobal.org/spec/lti-tool-configuration' => [