diff --git a/composer.json b/composer.json new file mode 100644 index 0000000..5f423f9 --- /dev/null +++ b/composer.json @@ -0,0 +1,10 @@ +{ + "name": "h5p/h5p-php-library", + "license": "GPL-3.0", + "autoload": { + "files": [ + "h5p-development.class.php", + "h5p.classes.php" + ] + } +} \ No newline at end of file diff --git a/h5p.classes.php b/h5p.classes.php index 816f116..d74e0b8 100644 --- a/h5p.classes.php +++ b/h5p.classes.php @@ -2408,6 +2408,10 @@ class H5PCore { if($platformInfo['uuid'] === '' && isset($json->uuid)) { $this->h5pF->setOption('site_uuid', $json->uuid); } + if (isset($json->latest) && !empty($json->latest)) { + $this->h5pF->setOption('update_available', $json->latest->releasedAt); + $this->h5pF->setOption('update_available_path', $json->latest->path); + } } } diff --git a/js/h5p-resizer.js b/js/h5p-resizer.js index 228cc72..772cafd 100644 --- a/js/h5p-resizer.js +++ b/js/h5p-resizer.js @@ -19,7 +19,6 @@ actionHandlers.hello = function (iframe, data, respond) { // Make iframe responsive iframe.style.width = '100%'; - iframe.contentDocument.body.style.height = 'auto'; // Tell iframe that it needs to resize when our window resizes var resize = function (event) { @@ -47,7 +46,14 @@ * @param {Function} respond Send a response to the iframe */ actionHandlers.prepareResize = function (iframe, data, respond) { - respond('resizePrepared'); + // Do not resize unless page and scrolling differs + if (iframe.clientHeight !== data.scrollHeight || + data.scrollHeight !== data.clientHeight) { + + // Reset iframe height, in case content has shrinked. + iframe.style.height = data.clientHeight + 'px'; + respond('resizePrepared'); + } }; /** @@ -59,16 +65,8 @@ * @param {Function} respond Send a response to the iframe */ actionHandlers.resize = function (iframe, data, respond) { - if (iframe.clientHeight === iframe.contentDocument.body.scrollHeight && - iframe.contentDocument.body.scrollHeight === iframe.contentWindow.document.body.clientHeight) { - return; // Do not resize unless page and scrolling differs - } - - // Reset iframe height, in case content has shrinked. - iframe.style.height = iframe.contentWindow.document.body.clientHeight + 'px'; - // Resize iframe so all content is visible. Use scrollHeight to make sure we get everything - iframe.style.height = iframe.contentDocument.body.scrollHeight + 'px'; + iframe.style.height = data.scrollHeight + 'px'; }; /** diff --git a/js/h5p-x-api.js b/js/h5p-x-api.js index 9fc6758..1cf8b74 100644 --- a/js/h5p-x-api.js +++ b/js/h5p-x-api.js @@ -95,7 +95,9 @@ H5P.EventDispatcher.prototype.triggerXAPIScored = function (score, maxScore, ver H5P.EventDispatcher.prototype.setActivityStarted = function() { if (this.activityStartTime === undefined) { // Don't trigger xAPI events in the editor - if (H5PIntegration.contents['cid-' + this.contentId] !== undefined) { + if (this.contentId !== undefined && + H5PIntegration.contents !== undefined && + H5PIntegration.contents['cid-' + this.contentId] !== undefined) { this.triggerXAPI('attempted'); } this.activityStartTime = Date.now(); @@ -114,4 +116,4 @@ H5P.xAPICompletedListener = function (event) { var contentId = event.getVerifiedStatementValue(['object', 'definition', 'extensions', 'http://h5p.org/x-api/h5p-local-content-id']); H5P.setFinished(contentId, score, maxScore); } -}; \ No newline at end of file +}; diff --git a/js/h5p.js b/js/h5p.js index 16af37f..fe45aaf 100644 --- a/js/h5p.js +++ b/js/h5p.js @@ -279,6 +279,9 @@ H5P.init = function (target) { // Initial setup/handshake is done parentIsFriendly = true; + // Make iframe responsive + document.body.style.height = 'auto'; + // Hide scrollbars for correct size document.body.style.overflow = 'hidden'; @@ -289,7 +292,7 @@ H5P.init = function (target) { // When resize has been prepared tell parent window to resize H5P.communicator.on('resizePrepared', function (data) { H5P.communicator.send('resize', { - height: document.body.scrollHeight + scrollHeight: document.body.scrollHeight }); }); @@ -307,7 +310,10 @@ H5P.init = function (target) { resizeDelay = setTimeout(function () { // Only resize if the iframe can be resized if (parentIsFriendly) { - H5P.communicator.send('prepareResize'); + H5P.communicator.send('prepareResize', { + scrollHeight: document.body.scrollHeight, + clientHeight: document.body.clientHeight + }); } else { H5P.communicator.send('hello'); @@ -1947,20 +1953,35 @@ H5P.createTitle = function (rawTitle, maxLength) { } if (H5PIntegration.saveFreq !== false) { + // When was the last state stored + var lastStoredOn = 0; // Store the current state of the H5P when leaving the page. - H5P.$window.on('beforeunload', function () { - for (var i = 0; i < H5P.instances.length; i++) { - var instance = H5P.instances[i]; - if (instance.getCurrentState instanceof Function || - typeof instance.getCurrentState === 'function') { - var state = instance.getCurrentState(); - if (state !== undefined) { - // Async is not used to prevent the request from being cancelled. - H5P.setUserData(instance.contentId, 'state', state, {deleteOnChange: true, async: false}); + var storeCurrentState = function () { + // Make sure at least 250 ms has passed since last save + var currentTime = new Date().getTime(); + if (currentTime - lastStoredOn > 250) { + lastStoredOn = currentTime; + for (var i = 0; i < H5P.instances.length; i++) { + var instance = H5P.instances[i]; + if (instance.getCurrentState instanceof Function || + typeof instance.getCurrentState === 'function') { + var state = instance.getCurrentState(); + if (state !== undefined) { + // Async is not used to prevent the request from being cancelled. + H5P.setUserData(instance.contentId, 'state', state, {deleteOnChange: true, async: false}); + } } } } + }; + // iPad does not support beforeunload, therefore using unload + H5P.$window.one('beforeunload unload', function () { + // Only want to do this once + H5P.$window.off('pagehide beforeunload unload'); + storeCurrentState(); }); + // pagehide is used on iPad when tabs are switched + H5P.$window.on('pagehide', storeCurrentState); } /**