MDL-67814 h5p: recover missed xAPI settings
authorSara Arjona <sara@moodle.com>
Thu, 16 Apr 2020 08:17:34 +0000 (10:17 +0200)
committerVíctor Déniz Falcón <victor@moodle.com>
Thu, 16 Apr 2020 08:36:05 +0000 (09:36 +0100)
h5p/classes/helper.php
h5p/classes/player.php

index f040d52..dd12ca6 100644 (file)
@@ -299,7 +299,7 @@ class helper {
      * @return array The settings.
      */
     public static function get_core_settings(): array {
      * @return array The settings.
      */
     public static function get_core_settings(): array {
-        global $CFG;
+        global $CFG, $USER;
 
         $basepath = $CFG->wwwroot . '/';
         $systemcontext = context_system::instance();
 
         $basepath = $CFG->wwwroot . '/';
         $systemcontext = context_system::instance();
@@ -321,7 +321,7 @@ class helper {
             'saveFreq' => false,
             'siteUrl' => $CFG->wwwroot,
             'l10n' => array('H5P' => $core->getLocalization()),
             'saveFreq' => false,
             'siteUrl' => $CFG->wwwroot,
             'l10n' => array('H5P' => $core->getLocalization()),
-            'user' => [],
+            'user' => ['name' => $USER->username, 'mail' => $USER->email],
             'hubIsEnabled' => true,
             'reportingIsEnabled' => false,
             'crossorigin' => null,
             'hubIsEnabled' => true,
             'reportingIsEnabled' => false,
             'crossorigin' => null,
index 30f04e6..9633388 100644 (file)
@@ -614,6 +614,8 @@ class player {
         $settings = helper::get_core_assets();
         // Added here because in the helper we don't have the h5p content id.
         $settings['moodleLibraryPaths'] = $this->core->get_dependency_roots($this->h5pid);
         $settings = helper::get_core_assets();
         // Added here because in the helper we don't have the h5p content id.
         $settings['moodleLibraryPaths'] = $this->core->get_dependency_roots($this->h5pid);
+        // Add also the Moodle component where the results will be tracked.
+        $settings['moodleComponent'] = $this->component;
 
         $cid = $this->get_cid();
         // The filterParameters function should be called before getting the dependencyfiles because it rebuild content
 
         $cid = $this->get_cid();
         // The filterParameters function should be called before getting the dependencyfiles because it rebuild content