MDL-67788 h5p: fix issue for non logged-in users

valmynd · Apr 16, 2020 · 0a05ca3 · 0a05ca3
1 parent 2ac1d9a
commit 0a05ca3
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/h5p/classes/helper.php b/h5p/classes/helper.php
@@ -312,6 +312,8 @@ public static function get_core_settings(): array {
         $factory = new factory();
         $core = $factory->get_core();
 
+        // When there is a logged in user, her information will be passed to the player. It will be used for tracking.
+        $usersettings = isloggedin() ? ['name' => $USER->username, 'mail' => $USER->email] : [];
         $settings = array(
             'baseUrl' => $basepath,
             'url' => "{$basepath}pluginfile.php/{$systemcontext->instanceid}/core_h5p",
@@ -321,7 +323,7 @@ public static function get_core_settings(): array {
             'saveFreq' => false,
             'siteUrl' => $CFG->wwwroot,
             'l10n' => array('H5P' => $core->getLocalization()),
-            'user' => ['name' => $USER->username, 'mail' => $USER->email],
+            'user' => $usersettings,
             'hubIsEnabled' => true,
             'reportingIsEnabled' => false,
             'crossorigin' => null,