From 6f54ece65581c55ee886e9277078ddee197c5393 Mon Sep 17 00:00:00 2001
From: Jake Dallimore <jake@moodle.com>
Date: Fri, 5 May 2023 11:34:15 +0800
Subject: [PATCH] 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.
---
enrol/lti/register.php | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/enrol/lti/register.php b/enrol/lti/register.php
index 78eeb049ed5..b5e3bf34830 100644
--- a/enrol/lti/register.php
+++ b/enrol/lti/register.php
@@ -34,6 +34,7 @@
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
*/
+use core\context\system;
use enrol_lti\local\ltiadvantage\repository\application_registration_repository;
use enrol_lti\local\ltiadvantage\repository\context_repository;
use enrol_lti\local\ltiadvantage\repository\deployment_repository;
@@ -42,7 +43,7 @@ use enrol_lti\local\ltiadvantage\repository\user_repository;
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 +114,9 @@ $regrequest = (object) [
$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' => 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' => [