archive/pillar
0
0
Fork 0
Code Issues 42 Pull Requests Projects Releases Wiki Activity

VideoJS: don't use videojs.registerPlugin() to start Google Analytics

Browse Source 
The `registerPlugin()` call should only be done once, and not for every
video shown.

This removes the warning about the 'analytics' plugin already being
registered, which you see when navigating from one video to another via
the JSTree.
This commit is contained in:
Sybren A. Stüvel
2018-08-31 15:52:18 +02:00
parent 05c488c484
commit 3f8e0396cf
1 changed files with 2 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/templates/nodes/custom/asset/video/view_embed.pug
View File

@@ -59,16 +59,14 @@ script(type="text/javascript").
} }
}; };
videojs.registerPlugin('analytics', function() { videojs(videoPlayer, options).ready(function() {
this.ga({ this.ga({
'eventLabel' : '{{ node._id }} - {{ node.name }}', 'eventLabel' : '{{ node._id }} - {{ node.name }}',
'eventCategory' : '{{ node.project }}', 'eventCategory' : '{{ node.project }}',
'eventsToTrack' : ['start', 'error', 'percentsPlayed'] 'eventsToTrack' : ['start', 'error', 'percentsPlayed']
}); });
});
videojs(videoPlayer, options).ready(function() {
this.hotkeys(); this.hotkeys();
}); });
function addVideoPlayerButton(data) { function addVideoPlayerButton(data) {