MDL-53832 enrol_lti: Support user images

Original subtask MDL-55718.
mvaraujo · Oct 19, 2016 · a618c70 · a618c70
1 parent 8e0038f
commit a618c70
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/enrol/lti/classes/tool_provider.php b/enrol/lti/classes/tool_provider.php
@@ -258,10 +258,10 @@ protected function onLaunch() {
         }
 
         // 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 @@ protected function onLaunch() {
     protected function onRegister() {
         global $PAGE;
 
-        $returnurl = $_POST['launch_presentation_return_url'];
+        $returnurl = $this->returnUrl;
         if (strpos($returnurl, '?') === false) {
             $separator = '?';
         } else {