moodle/moodle
3
0
Fork 0
mirror of https://github.com/moodle/moodle.git synced 2025-08-09 10:56:56 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

MDL-76953 mod: deprecate libxml library method wrappers.

Browse source 
The `libxml_disable_entity_loader` method is deprecated since PHP8.0,
which is our minimum required version so we no longer need to support
calls to it using the current wrappers.
This commit is contained in:
Paul Holden 2023-06-16 15:11:39 +01:00
parent 1b1a15a308
commit 436a279dec
No known key found for this signature in database
GPG key ID: A81A96D6045F6164
5 changed files with 14 additions and 15 deletions
Download patch file Download diff file Expand all files Collapse all files

8
mod/lti/locallib.php
View file

@ -4477,7 +4477,6 @@ function lti_load_cartridge($url, $map, $propertiesmap = array()) {
// TODO MDL-46023 Replace this code with a call to the new library.
$origerrors = libxml_use_internal_errors(true);
$origentity = lti_libxml_disable_entity_loader(true);
libxml_clear_errors();
$document = new DOMDocument();
@ -4489,7 +4488,6 @@ function lti_load_cartridge($url, $map, $propertiesmap = array()) {
libxml_clear_errors();
libxml_use_internal_errors($origerrors);
lti_libxml_disable_entity_loader($origentity);
if (count($errors) > 0) {
$message = 'Failed to load cartridge.';
@ -4584,10 +4582,10 @@ function lti_new_access_token($typeid, $scopes) {
*
* @param bool $value
* @return bool
*
* @deprecated since Moodle 4.3
*/
function lti_libxml_disable_entity_loader(bool $value): bool {
if (PHP_VERSION_ID < 80000) {
return (bool)libxml_disable_entity_loader($value);
}
debugging(__FUNCTION__ . '() is deprecated, please do not use it any more', DEBUG_DEVELOPER);
return true;
}

3
mod/lti/service.php
View file

@ -70,13 +70,10 @@ if ($consumerkey === false) {
}
// TODO MDL-46023 Replace this code with a call to the new library.
$origentity = lti_libxml_disable_entity_loader(true);
$xml = simplexml_load_string($rawbody);
if (!$xml) {
lti_libxml_disable_entity_loader($origentity);
throw new Exception('Invalid XML content');
}
lti_libxml_disable_entity_loader($origentity);
$body = $xml->imsx_POXBody;
foreach ($body->children() as $child) {

4
mod/lti/upgrade.txt
View file

@ -1,5 +1,9 @@
This files describes API changes in the lti code.
=== 4.3 ===
* The `lti_libxml_disable_entity_loader` method is deprecated, as it is no longer required from PHP 8.0
=== 4.2 ===
* get_jwt_claim_mappings() function has been added to service_base class. It allows a service subplugin to be