Trigger completed statement at 95% or more watched

When video ends, if the watched segments cover at least 95% of the total video time, then fire the xAPI completed statement. This covers edge cases where small imprecisions in play/pause timecodes cause getProgress() to report less-than-100% completion, even if the entire video was watched.
h5p · Mar 23, 2018 · 166c8d3 · 166c8d3
1 parent 91936f3
commit 166c8d3
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/scripts/html5.js b/scripts/html5.js
@@ -222,7 +222,7 @@ H5P.VideoHtml5 = (function ($) {
               if (length > 0) {
                 // Length passed in as current time, because at end of video when this is fired currentTime reset to 0 if on loop
                 var progress = videoXAPI.getProgress(length, length);
-                if (progress >= 1) {
+                if (progress >= 0.95) {
                   extraTrigger = 'finished';
                   extraArg = videoXAPI.getArgsXAPICompleted(video.currentTime, video.duration, progress);
                   lastSend = 'finished';

diff --git a/scripts/youtube.js b/scripts/youtube.js
@@ -146,7 +146,7 @@ H5P.VideoYouTube = (function ($) {
                 if (length > 0) {
                   // Length passed in as current time, because at end of video when this is fired currentTime reset to 0 if on loop
                   var progress = videoXAPI.getProgress(length, length);
-                  if (progress >= 1) {
+                  if (progress >= 0.95) {
                     var arg = videoXAPI.getArgsXAPICompleted(player.getCurrentTime(), self.duration, progress);
                     self.trigger('finished', arg);
                   }