Update h5p packages

sourcecode/apis/contentauthor/.gitignore

@@ -31,9 +31,11 @@ Homestead.json
 !/public/js/videos/brightcove.js
 !/public/js/videos/streamps.js
 !/public/js/h5p/ndlah5p-youtube.js
+!public/js/h5p/core-override
 !public/css
 /public/css/*
 !/public/css/ndlah5p-youtube.css
+
 mix-manifest.json
 app/Libraries/oauth-php/example/server/cache/
 ca1.sql

sourcecode/apis/contentauthor/app/H5PContent.php

@@ -325,14 +325,7 @@ public static function getContentTypeInfo(string $contentType): ?ContentTypeData
 
         if ($library->has_icon) {
             $h5pFramework = app(H5PFrameworkInterface::class);
-
-            $library_folder = H5PCore::libraryToString([
-                'machineName' => $library->machine_name,
-                'majorVersion' => $library->major_version,
-                'minorVersion' => $library->minor_version
-            ], true);
-
-
+            $library_folder = $library->getLibraryString(true);
             $icon_path = $h5pFramework->getLibraryFileUrl($library_folder, 'icon.svg');
 
             if (!empty($icon_path)) {

sourcecode/apis/contentauthor/app/H5PLibrariesHubCache.php

@@ -41,10 +41,16 @@ public function libraries()
 
     public function getLibraryString($folderName = false)
     {
-        return \H5PCore::libraryToString([
-            'machineName' => $this->name,
-            'majorVersion' => $this->major_version,
-            'minorVersion' => $this->minor_version,
-        ], $folderName);
+        return $folderName ?
+            \H5PCore::libraryToFolderName([
+                'machineName' => $this->name,
+                'majorVersion' => $this->major_version,
+                'minorVersion' => $this->minor_version,
+            ]) :
+            \H5PCore::libraryToString([
+                'machineName' => $this->name,
+                'majorVersion' => $this->major_version,
+                'minorVersion' => $this->minor_version,
+            ]);
     }
 }

sourcecode/apis/contentauthor/app/H5PLibrary.php

@@ -36,6 +36,10 @@ class H5PLibrary extends Model
 
     protected $guarded = ['id'];
 
+    protected $casts = [
+        'patch_version_in_folder_name' => 'bool',
+    ];
+
     protected static function boot()
     {
         parent::boot();
@@ -117,9 +121,65 @@ public function getVersions($asModels = false)
         return $asModels !== true ? $versions : $this->hydrate($versions->toArray());
     }
 
-    public function getLibraryString($folderName = false)
+    /**
+     * @param bool $folderName True to get name suitable for folder
+     * @param bool|null $fullVersion Null to use patchVersionInFolderName, true/false to override
+     */
+    public function getLibraryString(bool $folderName = false, ?bool $fullVersion = null): string
+    {
+        return self::getLibraryName([
+            'machineName' => $this->name,
+            'majorVersion' => $this->major_version,
+            'minorVersion' => $this->minor_version,
+            'patchVersion' => $this->patch_version,
+            'patchVersionInFolderName' => $this->patch_version_in_folder_name,
+
+        ], $folderName, $fullVersion);
+    }
+
+    /**
+     * @param array $libraryData Key names: machineName/name, majorVersion, minorVersion, patchVersion, patchVersionInFolderName
+     * @param bool|null $fullVersion Null to use patchVersionInFolderName, true/false to override
+     * @throws \InvalidArgumentException If requesting full version without patchVersion present in data
+     */
+    public static function libraryToFolderName(array $libraryData, ?bool $fullVersion = null): string
+    {
+        return self::getLibraryName($libraryData, true, $fullVersion);
+    }
+
+    /**
+     * @param array $libraryData Key names: machineName/name, majorVersion, minorVersion, patchVersion, patchVersionInFolderName
+     * @param bool|null $fullVersion Null to use patchVersionInFolderName, true/false to override
+     * @throws \InvalidArgumentException If requesting full version without patchVersion present in data
+     */
+    public static function libraryToString(array $libraryData, ?bool $fullVersion = null): string
     {
-        return \H5PCore::libraryToString($this->getLibraryH5PFriendly(), $folderName);
+        return self::getLibraryName($libraryData, false, $fullVersion);
+    }
+
+    /**
+     * @throws \InvalidArgumentException If requesting full version without patchVersion present in data
+     */
+    private static function getLibraryName(array $libraryData, bool $asFolder, ?bool $fullVersion): string
+    {
+        $usePatch = $fullVersion === true || ($fullVersion === null && array_key_exists('patchVersionInFolderName', $libraryData) && $libraryData['patchVersionInFolderName']);
+        if ($usePatch && !isset($libraryData['patchVersion'])) {
+            throw new \InvalidArgumentException('Full version name requested but patch version missing');
+        }
+
+        if ($usePatch) {
+            $format = $asFolder ? '%s-%d.%d.%d' : '%s %d.%d.%d';
+        } else {
+            $format = $asFolder ? '%s-%d.%d' : '%s %d.%d';
+        }
+
+        return sprintf(
+            $format,
+            $libraryData['machineName'] ?? $libraryData['name'],
+            $libraryData['majorVersion'],
+            $libraryData['minorVersion'],
+            $libraryData['patchVersion'] ?? ''
+        );
     }
 
     public function getLibraryH5PFriendly($machineName = 'name')
@@ -128,12 +188,19 @@ public function getLibraryH5PFriendly($machineName = 'name')
             'machineName' => $this->$machineName,
             'majorVersion' => $this->major_version,
             'minorVersion' => $this->minor_version,
+            'patchVersion' => $this->patch_version,
+            'patchVersionInFolderName' => $this->patch_version_in_folder_name,
         ];
     }
 
     public function getTitleAndVersionString()
     {
-        return \H5PCore::libraryToString($this->getLibraryH5PFriendly('title'));
+        return self::getLibraryName([
+            'machineName' => $this->title,
+            'majorVersion' => $this->major_version,
+            'minorVersion' => $this->minor_version,
+            'patchVersion' => $this->patch_version,
+        ], false, true);
     }
 
     public function scopeFromLibrary($query, $value)

sourcecode/apis/contentauthor/app/Http/Controllers/H5PController.php

@@ -175,7 +175,7 @@ public function create(Request $request, H5PCore $core, $contenttype = null): Vi
                 ->latestVersion()
                 ->first();
             if (!empty($library)) {
-                $contenttype = $library->getLibraryString(false);
+                $contenttype = $library->getLibraryString(false, false);
             } else {
                 $contenttype = false;
             }
@@ -332,7 +332,7 @@ public function edit(Request $request, int $id): View
 
         $state = H5PStateDataObject::create($displayOptions + [
             'id' => $h5pContent->id,
-            'library' => $library->getLibraryString(),
+            'library' => $library->getLibraryString(false, false),
             'libraryid' => $h5pContent->library_id,
             'parameters' => $params,
             'language_iso_639_3' => $contentLanguage,

sourcecode/apis/contentauthor/app/Libraries/H5P/AjaxRequest.php

@@ -259,8 +259,8 @@ private function libraryRebuild(Request $request): array
 
         $libraries = collect();
         $this->getLibraryDetails($H5PLibrary, $libraries);
-        if ($libraries->has($H5PLibrary->getLibraryString())) {
-            $libraryData = $libraries->get($H5PLibrary->getLibraryString());
+        if ($libraries->has($H5PLibrary->getLibraryString(false, false))) {
+            $libraryData = $libraries->get($H5PLibrary->getLibraryString(false, false));
             if (array_key_exists('semantics', $libraryData)) {
                 $H5PLibrary->semantics = $libraryData['semantics'];
                 $H5PLibrary->save();
@@ -307,15 +307,15 @@ private function getLibraryDetails(H5PLibrary $H5PLibrary, Collection $affectedL
         $libraryData = $validator->getLibraryData($H5PLibrary->getLibraryString(true), $tmpLibraryFolder, $tmpLibraries);
         $libraryData['libraryId'] = $H5PLibrary->id;
 
-        if (!$affectedLibraries->has($H5PLibrary->getLibraryString())) {
-            $affectedLibraries->put($H5PLibrary->getLibraryString(), $libraryData);
+        if (!$affectedLibraries->has($H5PLibrary->getLibraryString(false, false))) {
+            $affectedLibraries->put($H5PLibrary->getLibraryString(false, false), $libraryData);
         }
         foreach (['preloadedDependencies', 'dynamicDependencies', 'editorDependencies'] as $value) {
             if (!empty($libraryData[$value])) {
                 foreach ($libraryData[$value] as $library) {
                     /** @var H5PLibrary $dependentLibrary */
                     $dependentLibrary = H5PLibrary::fromLibrary($library)->first();
-                    if (!$affectedLibraries->has($dependentLibrary->getLibraryString())) {
+                    if (!$affectedLibraries->has($dependentLibrary->getLibraryString(false, false))) {
                         $affectedLibraries = $this->getLibraryDetails($dependentLibrary, $affectedLibraries);
                     }
                 }

sourcecode/apis/contentauthor/app/Libraries/H5P/EditorStorage.php

@@ -65,7 +65,7 @@ public function getLibraries($libraries = null)
                 ->map(function ($h5pLibrary) {
                     /** @var H5PLibrary $h5pLibrary */
                     $library = [
-                        'uberName' => $h5pLibrary->getLibraryString(),
+                        'uberName' => $h5pLibrary->getLibraryString(false, false),
                         'name' => $h5pLibrary->name,
                         'majorVersion' => $h5pLibrary->major_version,
                         'minorVersion' => $h5pLibrary->minor_version,
@@ -101,7 +101,7 @@ public function getLibraries($libraries = null)
                         $library->minorVersion = $library->minor_version;
 
                         // Add new library
-                        $library->uberName = $library->getLibraryString();
+                        $library->uberName = $library->getLibraryString(false, false);
                         if ($index > 0) {
                             $library->isOld = true;
                         }

sourcecode/apis/contentauthor/app/Libraries/H5P/Framework.php

@@ -21,6 +21,7 @@
 use Illuminate\Contracts\Filesystem\Filesystem;
 use Illuminate\Support\Facades\DB;
 use Illuminate\Support\Facades\Request;
+use Illuminate\Support\Stringable;
 use InvalidArgumentException;
 use PDO;
 use Psr\Http\Message\ResponseInterface;
@@ -132,19 +133,35 @@ public function getPlatformInfo()
         ];
     }
 
-    public function fetchExternalData($url, $data = null, $blocking = true, $stream = null): string|null
-    {
+    public function fetchExternalData(
+        $url,
+        $data = null,
+        $blocking = true,
+        $stream = null,
+        $fullData = false,
+        $headers = [],
+        $files = [],
+        $method = 'POST'
+    ): string|array|null {
         $method = $data ? 'POST' : 'GET';
         $options = [RequestOptions::FORM_PARAMS => $data];
         if ($stream !== null) {
             $options[RequestOptions::SINK] = $stream;
         }
+        $options[RequestOptions::HEADERS] = $headers;
 
         return $this->httpClient->requestAsync($method, $url, $options)
-            ->then(static function (ResponseInterface $response) use ($blocking) {
+            ->then(static function (ResponseInterface $response) use ($blocking, $fullData) {
                 if (!$blocking) {
                     return null;
                 }
+                if ($fullData) {
+                    return [
+                        'status' => $response->getStatusCode(),
+                        'headers' => $response->getHeaders(),
+                        'data' => $response->getBody()->getContents(),
+                    ];
+                }
 
                 return $response->getBody()->getContents();
             })
@@ -500,7 +517,8 @@ public function saveLibraryData(&$libraryData, $new = true)
             'metadata_settings' => $libraryData['metadataSettings'],
             'add_to' => $libraryData['addTo'],
             'has_icon' => $libraryData['hasIcon'] ?? 0,
-            'tutorial_url' => ''
+            'tutorial_url' => '',
+            'patch_version_in_folder_name' => true,
         ]);
         $libraryData['libraryId'] = $h5pLibrary->id;
 
@@ -819,13 +837,16 @@ public function loadLibrary($machineName, $majorVersion, $minorVersion): array|f
             'preloadedCss' => $h5pLibrary->preloaded_css,
             'dropLibraryCss' => $h5pLibrary->drop_library_css,
             'semantics' => $h5pLibrary->semantics,
+            'patchVersionInFolderName' => $h5pLibrary->patch_version_in_folder_name,
         ];
 
         foreach ($h5pLibrary->libraries as $dependency) {
             $library[$dependency->dependency_type . 'Dependencies'][] = [
                 'machineName' => $dependency->requiredLibrary->name,
                 'majorVersion' => $dependency->requiredLibrary->major_version,
                 'minorVersion' => $dependency->requiredLibrary->minor_version,
+                'patchVersion' => $dependency->requiredLibrary->patch_version,
+                'patchVersionInFolderName' => $dependency->requiredLibrary->patch_version_in_folder_name,
             ];
         }
 
@@ -907,10 +928,11 @@ public function deleteLibrary($library): void
             throw new TypeError(sprintf('Expected object, %s given', get_debug_type($library)));
         }
 
+        /** @var H5PLibrary $libraryModel */
         $libraryModel = H5PLibrary::findOrFail($library->id);
         $libraryModel->deleteOrFail();
 
-        app(CerpusStorageInterface::class)->deleteLibrary($libraryModel);
+        app(\H5PFileStorage::class)->deleteLibrary($libraryModel->getLibraryH5PFriendly());
     }
 
     /**
@@ -953,6 +975,8 @@ public function loadContent($id)
             'libraryName' => $h5pcontent->library->name,
             'libraryMajorVersion' => $h5pcontent->library->major_version,
             'libraryMinorVersion' => $h5pcontent->library->minor_version,
+            'libraryPatchVersion' => $h5pcontent->library->patch_version,
+            'libraryFullVersionName' => $h5pcontent->library->getLibraryString(),
             'libraryEmbedTypes' => $h5pcontent->library->embed_types,
             'libraryFullscreen' => $h5pcontent->library->fullscreen,
             'language' => $h5pcontent->metadata->default_language ?? null,
@@ -986,6 +1010,8 @@ public function loadContent($id)
      *   - preloadedJs(optional): comma separated string with js file paths
      *   - preloadedCss(optional): comma separated sting with css file paths
      *   - dropCss(optional): csv of machine names
+     *   - dependencyType: editor or preloaded
+     *   - patchVersionInFolderName: Is patch version a part of the folder name
      */
     public function loadContentDependencies($id, $type = null)
     {
@@ -1003,6 +1029,7 @@ public function loadContentDependencies($id, $type = null)
               , hl.preloaded_js AS preloadedJs
               , hcl.drop_css AS dropCss
               , hcl.dependency_type AS dependencyType
+              , hl.patch_version_in_folder_name AS patchVersionInFolderName
         FROM h5p_contents_libraries hcl
         JOIN h5p_libraries hl ON hcl.library_id = hl.id
         WHERE hcl.content_id = ?";
@@ -1334,4 +1361,27 @@ public function libraryHasUpgrade($library)
         $h5pLibrary = H5PLibrary::fromLibrary($library)->first();
         return !is_null($h5pLibrary) && $h5pLibrary->isUpgradable();
     }
+
+    public function replaceContentHubMetadataCache($metadata, $lang)
+    {
+        // H5P Content Hub is not in use
+    }
+
+    public function getContentHubMetadataCache($lang = 'en')
+    {
+        // H5P Content Hub is not in use
+        return new Stringable();
+    }
+
+    public function getContentHubMetadataChecked($lang = 'en')
+    {
+        // H5P Content Hub is not in use
+        return now()->toRfc7231String();
+    }
+
+    public function setContentHubMetadataChecked($time, $lang = 'en')
+    {
+        // H5P Content Hub is not in use
+        return true;
+    }
 }

sourcecode/apis/contentauthor/app/Libraries/H5P/H5PExport.php

@@ -6,7 +6,6 @@
 use App\H5PLibrary;
 use App\Libraries\H5P\Interfaces\H5PAdapterInterface;
 use App\Libraries\H5P\Interfaces\H5PExternalProviderInterface;
-use H5PCore;
 use H5PExport as H5PDefaultExport;
 use Illuminate\Support\Collection;
 
@@ -33,9 +32,13 @@ public function generateExport($convertMediaToLocal = false)
         }
         /** @var H5PLibrary $h5PLibrary */
         $h5PLibrary = $this->content->library;
-        $library = H5PCore::libraryFromString($h5PLibrary->getLibraryString());
-        $library['libraryId'] = $h5PLibrary->id;
-        $library['name'] = $h5PLibrary->name;
+        $library = [
+            'machineName' => $h5PLibrary->name,
+            'majorVersion' => $h5PLibrary->major_version,
+            'minorVersion' => $h5PLibrary->minor_version,
+            'libraryId' => $h5PLibrary->id,
+            'name' => $h5PLibrary->name,
+        ];
 
         $contents = $this->content->toArray();
         $contents['params'] = $this->content->parameters;
@@ -46,7 +49,7 @@ public function generateExport($convertMediaToLocal = false)
         /** @var \H5PContentValidator $validator */
         $validator = resolve(\H5PContentValidator::class);
         $params = (object)[
-            'library' => $h5PLibrary->getLibraryString(),
+            'library' => $h5PLibrary->getLibraryString(false, false),
             'params' => json_decode($this->content->parameters)
         ];
         if (!$params->params) {