Merge branch 'MDL-59492-master' of git://github.com/lameze/moodle
authorJun Pataleta <jun@moodle.com>
Mon, 14 Aug 2017 04:16:08 +0000 (12:16 +0800)
committerJun Pataleta <jun@moodle.com>
Mon, 14 Aug 2017 04:16:08 +0000 (12:16 +0800)
lib/authlib.php

index 657f887..4af4bcb 100644 (file)
@@ -651,10 +651,10 @@ class auth_plugin_base {
         $data = [];
         foreach ($identityproviders as $idp) {
             if (!empty($idp['icon'])) {
-                // Pre-3.3 auth plugins provide icon as a pix_icon instance.
+                // Pre-3.3 auth plugins provide icon as a pix_icon instance. New auth plugins (since 3.3) provide iconurl.
                 $idp['iconurl'] = $output->image_url($idp['icon']->pix, $idp['icon']->component);
-            } else if ($idp['iconurl'] instanceof moodle_url) {
-                // New auth plugins (since 3.3) provide iconurl.
+            }
+            if ($idp['iconurl'] instanceof moodle_url) {
                 $idp['iconurl'] = $idp['iconurl']->out(false);
             }
             unset($idp['icon']);