Skip to content

Commit

Permalink
Add video length to initialized statement
Browse files Browse the repository at this point in the history
Was asked by Jon to add video length to the extensions sent by the intialized verb.
  • Loading branch information
KirkJohnson committed Apr 23, 2018
1 parent 12912a8 commit 4f70c16
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
2 changes: 1 addition & 1 deletion scripts/html5.js
Original file line number Diff line number Diff line change
Expand Up @@ -162,7 +162,7 @@ H5P.VideoHtml5 = (function ($) {
}
}

return videoXAPI.getArgsXAPIInitialized(video.videoWidth, video.videoHeight, video.playbackRate, video.volume, ccEnabled, ccLanguage);
return videoXAPI.getArgsXAPIInitialized(video.videoWidth, video.videoHeight, video.playbackRate, video.volume, ccEnabled, ccLanguage, video.videoHeight, video.duration);

};

Expand Down
3 changes: 2 additions & 1 deletion scripts/x-api.js
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ H5P.VideoXAPI = (function ($) {
* @returns {Object} JSON xAPI statement
*
*/
self.getArgsXAPIInitialized = function (width, height, rate, volume, ccEnabled, ccLanguage, quality) {
self.getArgsXAPIInitialized = function (width, height, rate, volume, ccEnabled, ccLanguage, quality, videoLength) {
// If quality isn't provided, set it to the height of the video.
quality = typeof quality !== 'undefined' ? quality : height;

Expand All @@ -90,6 +90,7 @@ H5P.VideoXAPI = (function ($) {
volume = formatFloat(volume);

var extensions = {};
extensions['https://w3id.org/xapi/video/extensions/length'] = videoLength;
if (typeof isFullscreen !== 'undefined') {
extensions['https://w3id.org/xapi/video/extensions/full-screen'] = isFullscreen;
}
Expand Down
2 changes: 1 addition & 1 deletion scripts/youtube.js
Original file line number Diff line number Diff line change
Expand Up @@ -240,7 +240,7 @@ H5P.VideoYouTube = (function ($) {
ccLanguage = player.getOptions('cc', 'track').languageCode;
}

return videoXAPI.getArgsXAPIInitialized(width, height, self.getPlaybackRate(), self.getVolume(), ccEnabled, ccLanguage, self.getQuality());
return videoXAPI.getArgsXAPIInitialized(width, height, self.getPlaybackRate(), self.getVolume(), ccEnabled, ccLanguage, self.getQuality(), self.duration);

};

Expand Down

0 comments on commit 4f70c16

Please sign in to comment.