From 79aa88fddd6c80c280fbb65ccb5e8945e9fa1dfa Mon Sep 17 00:00:00 2001
From: Sara Arjona <sara@moodle.com>
Date: Tue, 7 Sep 2021 17:09:16 +0200
Subject: [PATCH] MDL-71970 h5plib_v124: Add namespace to H5P editor library

The joubel/editor is a third-party library. A namespace has been added to
avoid collision with other plugins using it (such as mod_hvp).
That way, they will be able to have a different version without
side effects.
---
 h5p/h5plib/v124/joubel/editor/h5peditor-ajax.class.php |  6 ++++--
 .../v124/joubel/editor/h5peditor-ajax.interface.php    |  1 +
 h5p/h5plib/v124/joubel/editor/h5peditor-file.class.php |  3 +++
 .../v124/joubel/editor/h5peditor-storage.interface.php |  2 ++
 h5p/h5plib/v124/joubel/editor/h5peditor.class.php      | 10 +++++++---
 h5p/h5plib/v124/joubel/editor/readme_moodle.txt        |  9 ++++++++-
 6 files changed, 25 insertions(+), 6 deletions(-)

diff --git a/h5p/h5plib/v124/joubel/editor/h5peditor-ajax.class.php b/h5p/h5plib/v124/joubel/editor/h5peditor-ajax.class.php
index 51ba355dc97..536399287b0 100644
--- a/h5p/h5plib/v124/joubel/editor/h5peditor-ajax.class.php
+++ b/h5p/h5plib/v124/joubel/editor/h5peditor-ajax.class.php
@@ -1,5 +1,7 @@
 <?php
 
+namespace Moodle;
+
 abstract class H5PEditorEndpoints {
 
   /**
@@ -54,7 +56,7 @@ abstract class H5PEditorEndpoints {
 class H5PEditorAjax {
 
   /**
-   * @var \H5PCore
+   * @var H5PCore
    */
   public $core;
 
@@ -225,7 +227,7 @@ class H5PEditorAjax {
   private function isValidEditorToken($token) {
     $isValidToken = $this->editor->ajaxInterface->validateEditorToken($token);
     if (!$isValidToken) {
-      \H5PCore::ajaxError(
+      H5PCore::ajaxError(
         $this->core->h5pF->t('Invalid security token.'),
         'INVALID_TOKEN'
       );
diff --git a/h5p/h5plib/v124/joubel/editor/h5peditor-ajax.interface.php b/h5p/h5plib/v124/joubel/editor/h5peditor-ajax.interface.php
index d45b2b1bea1..89d42634a92 100644
--- a/h5p/h5plib/v124/joubel/editor/h5peditor-ajax.interface.php
+++ b/h5p/h5plib/v124/joubel/editor/h5peditor-ajax.interface.php
@@ -1,5 +1,6 @@
 <?php
 
+namespace Moodle;
 
 /**
  * Handles Ajax functionality that must be implemented separately for each of the
diff --git a/h5p/h5plib/v124/joubel/editor/h5peditor-file.class.php b/h5p/h5plib/v124/joubel/editor/h5peditor-file.class.php
index 89329e1174b..9f183289aee 100644
--- a/h5p/h5plib/v124/joubel/editor/h5peditor-file.class.php
+++ b/h5p/h5plib/v124/joubel/editor/h5peditor-file.class.php
@@ -1,5 +1,8 @@
 <?php
 
+namespace Moodle;
+
+use stdClass;
 /**
  * Class
  */
diff --git a/h5p/h5plib/v124/joubel/editor/h5peditor-storage.interface.php b/h5p/h5plib/v124/joubel/editor/h5peditor-storage.interface.php
index 92939c8c791..763ea5ec568 100644
--- a/h5p/h5plib/v124/joubel/editor/h5peditor-storage.interface.php
+++ b/h5p/h5plib/v124/joubel/editor/h5peditor-storage.interface.php
@@ -1,5 +1,7 @@
 <?php
 
+namespace Moodle;
+
 /**
  * A defined interface for the editor to communicate with the database of the
  * web system.
diff --git a/h5p/h5plib/v124/joubel/editor/h5peditor.class.php b/h5p/h5plib/v124/joubel/editor/h5peditor.class.php
index 37a60a58208..0f02896466d 100644
--- a/h5p/h5plib/v124/joubel/editor/h5peditor.class.php
+++ b/h5p/h5plib/v124/joubel/editor/h5peditor.class.php
@@ -1,5 +1,9 @@
 <?php
 
+namespace Moodle;
+
+use stdClass;
+
 class H5peditor {
 
   private static $hasWYSIWYGEditor = array(
@@ -56,9 +60,9 @@ class H5peditor {
   /**
    * Constructor for the core editor library.
    *
-   * @param \H5PCore $h5p Instance of core
-   * @param \H5peditorStorage $storage Instance of h5peditor storage interface
-   * @param \H5PEditorAjaxInterface $ajaxInterface Instance of h5peditor ajax
+   * @param H5PCore $h5p Instance of core
+   * @param H5peditorStorage $storage Instance of h5peditor storage interface
+   * @param H5PEditorAjaxInterface $ajaxInterface Instance of h5peditor ajax
    * interface
    */
   function __construct($h5p, $storage, $ajaxInterface) {
diff --git a/h5p/h5plib/v124/joubel/editor/readme_moodle.txt b/h5p/h5plib/v124/joubel/editor/readme_moodle.txt
index 977d96d8c5f..0ae73b67ea5 100644
--- a/h5p/h5plib/v124/joubel/editor/readme_moodle.txt
+++ b/h5p/h5plib/v124/joubel/editor/readme_moodle.txt
@@ -15,6 +15,13 @@ all the existing strings from the previous version. As you'll see, all the
 strings in en.js have been converted following these rules:
   * Prefix  "editor:" has been added.
   * Keys have been lowercased.
+- Add namespace to this library to avoid collision. It means:
+  * Add the "namespace Moodle;" line at the top of all the h5peditor*.php files in the root folder.
+  * Replace \H5Pxxx uses to H5Pxxx (for instance, in h5peditor-ajax.class.php there are several references to \H5PCore that
+must be replaced with H5PCore).
+  * Add "use stdClass;" in h5peditor.class.php and h5peditor-file.class.php (check that it's still used before replacing it when upgrading the library).
+
+
 
 Removed:
  * composer.json
@@ -26,4 +33,4 @@ Added:
 Changed:
  * Updated H5peditor::getLibraryData parameters to fix PHP8.0 warnings. See MDL-70903 for details.
 
-Downloaded version: moodle-1.20.2 release
\ No newline at end of file
+Downloaded version: moodle-1.20.2 release