846d9da4722c8aeb4a35effdd9ad0b7de123d15b
[moodle.git] / h5p / h5plib / v124 / joubel / core / readme_moodle.txt
1 H5P PHP library
2 ---------------
4 Downloaded last release from: https://github.com/h5p/h5p-php-library/releases
6 Import procedure:
8 - Copy all the files from the folder repository in this directory.
10 Removed:
11  * composer.json
12  * .gitignore
14 Added:
15  * readme_moodle.txt
17 Downloaded version: 1.24 release
20 === 3.8 ===
21 1. In order to allow the dependency path to be overridden by child H5PCore classes, a couple of minor changes have been added to the
22 h5p.classes.php file:
23     - Into the getDependenciesFiles method, the line 2435:
24         $dependency['path'] = 'libraries/' . H5PCore::libraryToString($dependency, TRUE);
26       has been changed to:
27         $dependency['path'] = $this->getDependencyPath($dependency);
29      - The method getDependencyPath has been added (line 2455). It might be rewritten by child classes.
30 A PR has been sent to the H5P library with these changes:
31 https://github.com/h5p/h5p-php-library/compare/master...andrewnicols:libraryPathSubclass
32 Hopefully, when upgrading, these patch won't be needed because it will be included in the H5P library by default.
35 2. As the mbstring extension is optional in Moodle, the following changes have been hardcoded to the library:
36 2.1. Comment the following methods in h5p.classes.php file where the extension_loaded('mbstring') is called:
37     * isValidPackage
38     * checkSetupErrorMessage
39     * validateText
40     * validateContentFiles
42 2.2. Change all the mb_uses straight to the core_text() alternatives. Version 1.24 has 3 ocurrences in h5p.classes.php
43 and 1 ocurrence in h5p-metadata.class.php.
45 3. Another PR has been sent to H5P library (https://github.com/h5p/h5p-php-library/pull/69) to fix some php74 minor problems. The same fix is being applied locally by MDL-67077. Once we import a new version, if it includes de fix, this won't be needed to reapply and can be removed.
48 The point 2 from above won't be needed once the mbstring extension becomes mandatory in Moodle. A request has been
49 sent to MDL-65809.