MDL-67814 core_h5p: modifications to interact with the Hub selector
authorVíctor Déniz Falcón <victor@moodle.com>
Sun, 12 Apr 2020 09:59:48 +0000 (10:59 +0100)
committerAdrian Greeve <abgreeve@gmail.com>
Thu, 16 Apr 2020 03:29:35 +0000 (11:29 +0800)
The HUB selector allows you to install/update H5P content types from
the H5P editor. We have moved these features to the Content Bank, so we
will not show the HUB selector in the H5P editor.

h5p/classes/framework.php
h5p/h5plib/v124/joubel/editor/readme_moodle.txt
h5p/h5plib/v124/joubel/editor/scripts/h5peditor-library-selector.js

index 286f16f..4524f0a 100644 (file)
@@ -1265,6 +1265,10 @@ class framework implements \H5PFrameworkInterface {
      * Get stored setting.
      * Implements getOption.
      *
+     * To avoid updating the cache libraries when using the Hub selector,
+     * {@link \H5PEditorAjax::isContentTypeCacheUpdated}, the setting content_type_cache_updated_at
+     * always return the current time.
+     *
      * @param string $name Identifier for the setting
      * @param string $default Optional default value if settings is not set
      * @return mixed Return  Whatever has been stored as the setting
@@ -1277,6 +1281,11 @@ class framework implements \H5PFrameworkInterface {
             return \H5PDisplayOptionBehaviour::CONTROLLED_BY_AUTHOR_DEFAULT_OFF;
         }
 
+        // To avoid update the libraries cache using the Hub selector.
+        if ($name == 'content_type_cache_updated_at') {
+            return time();
+        }
+
         $value = get_config('core_h5p', $name);
         if ($value === false) {
             return $default;
index 43f07d3..34f3c9f 100644 (file)
@@ -6,6 +6,8 @@ Downloaded last release from: https://github.com/h5p/h5p-editor-php-library/rele
 Import procedure:
 
 - Copy all the files from the folder repository in this directory.
+- In the method ns.LibrarySelector.prototype.appendTo (scripts/h5peditor-library-selector.js),
+  comment the line "this.$selector.appendTo($element);" to avoid the display of the Hub Selector.
 
 Removed:
  * composer.json
index 2d4a02c..877fb46 100644 (file)
@@ -129,7 +129,13 @@ ns.LibrarySelector.prototype.appendTo = function ($element) {
   var self = this;
   this.$parent = $element;
 
-  this.$selector.appendTo($element);
+  /*
+    Start Moodle change
+    This line is commented out to prevent display of the hub selector in Moodle.
+    For more information see MDL-67814
+  */
+  //this.$selector.appendTo($element);
+  /* End Moodle change */
   this.$tutorialUrl.appendTo($element);
   this.$exampleUrl.appendTo($element);