From a618c707f24c8a41128caaf41cb3d7c9cb20dd7e Mon Sep 17 00:00:00 2001 From: John Okely Date: Tue, 13 Sep 2016 16:32:43 +0800 Subject: [PATCH] MDL-53832 enrol_lti: Support user images Original subtask MDL-55718. --- enrol/lti/classes/tool_provider.php | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/enrol/lti/classes/tool_provider.php b/enrol/lti/classes/tool_provider.php index 6d8ad5f0c79..24543363815 100644 --- a/enrol/lti/classes/tool_provider.php +++ b/enrol/lti/classes/tool_provider.php @@ -258,10 +258,10 @@ class tool_provider extends ToolProvider\ToolProvider { } // Update user image. - $image = false; - if (!empty($this->resourceLink->getSetting('user_image'))) { - $image = $this->resourceLink->getSetting('user_image'); - } else if (!empty($this->resourceLink->getSetting('custom_user_image'))) { + if (isset($this->user) && isset($this->user->image) && !empty($this->user->image)) { + $image = $this->user->image; + } else { + // Use custom_user_image parameter as a fallback. $image = $this->resourceLink->getSetting('custom_user_image'); } @@ -363,7 +363,7 @@ class tool_provider extends ToolProvider\ToolProvider { protected function onRegister() { global $PAGE; - $returnurl = $_POST['launch_presentation_return_url']; + $returnurl = $this->returnUrl; if (strpos($returnurl, '?') === false) { $separator = '?'; } else {