MDL-62336 media_videojs: ensure videos have unique IDs
authorMark Nelson <markn@moodle.com>
Mon, 28 May 2018 08:43:57 +0000 (16:43 +0800)
committerMark Nelson <markn@moodle.com>
Mon, 28 May 2018 08:43:57 +0000 (16:43 +0800)
media/player/videojs/classes/plugin.php

index 5315e50..9386106 100644 (file)
@@ -149,9 +149,10 @@ class media_videojs_plugin extends core_media_player_native {
         // which is BEFORE we have a chance to load any additional libraries (youtube).
         // The data-setup-lazy is just a tag name that video.js does not recognise so we can manually initialise
         // it when we are sure the dependencies are loaded.
+        static $playercounter = 1;
         $attributes = [
             'data-setup-lazy' => '{' . join(', ', $datasetup) . '}',
-            'id' => 'id_videojs_' . uniqid(),
+            'id' => 'id_videojs_' . uniqid() . '_' . $playercounter++,
             'class' => get_config('media_videojs', $isaudio ? 'audiocssclass' : 'videocssclass')
         ];