From cd6b59ec8a944f88c3763301c9edfb0f9955fa4f Mon Sep 17 00:00:00 2001
From: Justus Dieckmann <justusdieckmann@wwu.de>
Date: Wed, 21 Aug 2024 18:12:39 +0200
Subject: [PATCH 1/2] Pass data properly to paella

---
 templates/player.mustache | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/templates/player.mustache b/templates/player.mustache
index 60b5499..5ff60be 100644
--- a/templates/player.mustache
+++ b/templates/player.mustache
@@ -28,7 +28,7 @@
       "playerid": "ocplayer_0",
       "configurl": "/filter/opencast/config.json",
       "modplayerpath": "/mod/opencast/player.html",
-      "data": "{&quot;metadata&quot;:{&quot;title&quot;: &quot;Test video&quot;, &quot;etc&quot;: &quot;...&quot;}"
+      "data": "{\"metadata\":{\"title\": \"Test video\", \"etc\": \"...\"}"
   }
 }}
 
@@ -51,7 +51,7 @@ function init_{{playerid}}() {
         iframeWindow.MoodlePaellaPlayer.initPaella(
             '{{{configurl}}}',
             '{{{themeurl}}}',
-            JSON.parse('{{{data}}}')
+            {{{data}}}
         );
     }
 }

From 91bb076b43c893571967e95f4b103f3140f85dad Mon Sep 17 00:00:00 2001
From: Justus Dieckmann <justusdieckmann@wwu.de>
Date: Wed, 21 Aug 2024 18:21:03 +0200
Subject: [PATCH 2/2] Make CI happy

---
 classes/privacy/provider.php |  2 +-
 lang/en/filter_opencast.php  |  4 ++--
 tests/replacement_test.php   | 18 +++++++++---------
 3 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/classes/privacy/provider.php b/classes/privacy/provider.php
index c1203e1..0172d72 100644
--- a/classes/privacy/provider.php
+++ b/classes/privacy/provider.php
@@ -38,7 +38,7 @@ class provider implements \core_privacy\local\metadata\null_provider {
      *
      * @return  string
      */
-    public static function get_reason() : string {
+    public static function get_reason(): string {
         return 'privacy:metadata';
     }
 }
diff --git a/lang/en/filter_opencast.php b/lang/en/filter_opencast.php
index 50da383..bfb6591 100644
--- a/lang/en/filter_opencast.php
+++ b/lang/en/filter_opencast.php
@@ -26,7 +26,7 @@
 $string['filtername'] = 'Opencast';
 $string['pluginname'] = 'Opencast Filter';
 $string['privacy:metadata'] = 'The Opencast filter plugin does not store any personal data.';
-$string['setting_episodeurl'] = 'URL templates for filtering';
-$string['setting_episodeurl_desc'] = 'URLs matching this template are replaced with the Opencast player. You must use the placeholder [EPISODEID] to indicate where the episode ID is contained in the URL e.g. http://stable.opencast.de/play/[EPISODEID]. If you want to filter for multiple URLs, enter each URL in a new line.';
 $string['setting_configurl'] = 'URL to Paella config.json';
 $string['setting_configurl_desc'] = 'URL of the config.json used by Paella Player. Can either be a absolute URL or a URL relative to the wwwroot.';
+$string['setting_episodeurl'] = 'URL templates for filtering';
+$string['setting_episodeurl_desc'] = 'URLs matching this template are replaced with the Opencast player. You must use the placeholder [EPISODEID] to indicate where the episode ID is contained in the URL e.g. http://stable.opencast.de/play/[EPISODEID]. If you want to filter for multiple URLs, enter each URL in a new line.';
diff --git a/tests/replacement_test.php b/tests/replacement_test.php
index 19cd111..25c092a 100644
--- a/tests/replacement_test.php
+++ b/tests/replacement_test.php
@@ -36,7 +36,7 @@
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  * @group      filter_opencast
  */
-class replacement_test extends \advanced_testcase {
+final class replacement_test extends \advanced_testcase {
 
     public function setUp(): void {
         $this->resetAfterTest();
@@ -52,7 +52,7 @@ public function setUp(): void {
      * @param string $input input for filter
      * @param string $output expected filter output.
      */
-    public function test_replacement($input, $output) {
+    public function test_replacement($input, $output): void {
         $filter = new testable_filter(\context_system::instance(), []);
         $this->assertEquals($output, $filter->filter($input));
     }
@@ -60,24 +60,24 @@ public function test_replacement($input, $output) {
     /**
      * Provides test cases.
      */
-    public function replacement_provider() {
+    public static function replacement_provider(): array {
         return [
             [
                 ' <p> hello </p> <video src="http://localhost:8080/play/f78ac136-8252-4b8e-bfea-4786c6993f03"> hello </video>',
-                ' <p> hello </p> <oc-video episode="f78ac136-8252-4b8e-bfea-4786c6993f03"/>'
+                ' <p> hello </p> <oc-video episode="f78ac136-8252-4b8e-bfea-4786c6993f03"/>',
             ],
             [
                 '<video src="https://somethingother.com"></video><video>
 <source
 src="https://stable.opencast.de/play/370e5bef-1d59-4440-858a-4df62e767dfc">
 </video>',
-                '<video src="https://somethingother.com"></video><oc-video episode="370e5bef-1d59-4440-858a-4df62e767dfc"/>'
+                '<video src="https://somethingother.com"></video><oc-video episode="370e5bef-1d59-4440-858a-4df62e767dfc"/>',
             ],
             [
                 '<video
 autoplay loopdiloop
 src="http://localhost:8080/play/f9e7b289-c8be-462f-80bf-d1f493c6ed55"></video>',
-                '<oc-video episode="f9e7b289-c8be-462f-80bf-d1f493c6ed55"/>'
+                '<oc-video episode="f9e7b289-c8be-462f-80bf-d1f493c6ed55"/>',
             ],
             [
                 'begin <video>
@@ -85,20 +85,20 @@ public function replacement_provider() {
 <source
 src="https://stable.opencast.de/play/2e0ca3bb-df8e-4913-9380-c925efaf5ac2">
 </video> end',
-                'begin <oc-video episode="2e0ca3bb-df8e-4913-9380-c925efaf5ac2"/> end'
+                'begin <oc-video episode="2e0ca3bb-df8e-4913-9380-c925efaf5ac2"/> end',
             ],
             [
                 'and a link <a href="https://www.google.com">link</a>
 <a href="http://localhost:8080/play/09b9d154-c849-429d-adea-3df4f76429b6">look, a video!</a>',
                 'and a link <a href="https://www.google.com">link</a>
-<oc-video episode="09b9d154-c849-429d-adea-3df4f76429b6"/>'
+<oc-video episode="09b9d154-c849-429d-adea-3df4f76429b6"/>',
             ],
             [
                 'and now two <a
 href="http://localhost:8080/play/64b085e9-0142-4a10-a08e-3dbce055e740">look, a video!</a>
 <video src="http://localhost:8080/play/329885fe-d18e-4c6b-a896-dbc66463a6b2"></video>.',
                 'and now two <oc-video episode="64b085e9-0142-4a10-a08e-3dbce055e740"/>
-<oc-video episode="329885fe-d18e-4c6b-a896-dbc66463a6b2"/>.'
+<oc-video episode="329885fe-d18e-4c6b-a896-dbc66463a6b2"/>.',
             ],
         ];
     }