MDL-69174 mod_h5pactivity: component passed always
authorSara Arjona <sara@moodle.com>
Wed, 2 Sep 2020 10:09:27 +0000 (12:09 +0200)
committerSara Arjona <sara@moodle.com>
Wed, 2 Sep 2020 10:09:30 +0000 (12:09 +0200)
The $component = 'mod_h5pactivity' will be passed always to the H5P
player because it's required for being able to call these activities
from outside using embed code. The xAPI implementation checks whether
the information should be tracked or not.

mod/h5pactivity/view.php

index 9feac00..d79308b 100644 (file)
@@ -88,14 +88,11 @@ if (isset($reviewurl)) {
     echo $OUTPUT->render($widget);
 }
 
-if ($manager->is_tracking_enabled()) {
-    $trackcomponent = 'mod_h5pactivity';
-} else {
-    $trackcomponent = '';
+if (!$manager->is_tracking_enabled()) {
     $message = get_string('previewmode', 'mod_h5pactivity');
     echo $OUTPUT->notification($message, \core\output\notification::NOTIFY_WARNING);
 }
 
-echo player::display($fileurl, $config, true, $trackcomponent);
+echo player::display($fileurl, $config, true, 'mod_h5pactivity');
 
 echo $OUTPUT->footer();