From 43dda6346d71ff268d6cffb3393059ed9fe53d16 Mon Sep 17 00:00:00 2001
From: Ruslan Kabalin <ruslan.kabalin@gmail.com>
Date: Tue, 8 Oct 2024 21:54:10 +0100
Subject: [PATCH] Adjust filter plugin requirements to comply with 4.5.

Per https://moodledev.io/docs/4.5/devupdate#filter-plugins

Fixes #321
---
 .../Requirements/FilterRequirements.php       | 19 +++++++++---
 .../Requirements/FilterRequirementsTest.php   | 31 +++++++++++++------
 2 files changed, 37 insertions(+), 13 deletions(-)

diff --git a/src/PluginValidate/Requirements/FilterRequirements.php b/src/PluginValidate/Requirements/FilterRequirements.php
index cbed68db..47738373 100644
--- a/src/PluginValidate/Requirements/FilterRequirements.php
+++ b/src/PluginValidate/Requirements/FilterRequirements.php
@@ -21,15 +21,26 @@ class FilterRequirements extends GenericRequirements
 {
     public function getRequiredFiles(): array
     {
-        return array_merge(parent::getRequiredFiles(), [
-            'filter.php',
-        ]);
+        $files = [];
+        if ($this->moodleVersion >= 405) {
+            $files[] = 'classes/text_filter.php';
+        } else {
+            // This must exist in 405 if plugin supports older version, but we don't identify support range to validate.
+            $files[] = 'filter.php';
+        }
+
+        return array_merge(parent::getRequiredFiles(), $files);
     }
 
     public function getRequiredClasses(): array
     {
+        if ($this->moodleVersion <= 404) {
+            // Really we should have a class for the filter.php file, but we don't identify support range to validate.
+            return [];
+        }
+
         return [
-            FileTokens::create('filter.php')->mustHave('filter_' . $this->plugin->name),
+            FileTokens::create('classes/text_filter.php')->mustHave("filter_{$this->plugin->name}\\text_filter"),
         ];
     }
 
diff --git a/tests/PluginValidate/Requirements/FilterRequirementsTest.php b/tests/PluginValidate/Requirements/FilterRequirementsTest.php
index de2b2d4d..cf9a9f31 100644
--- a/tests/PluginValidate/Requirements/FilterRequirementsTest.php
+++ b/tests/PluginValidate/Requirements/FilterRequirementsTest.php
@@ -25,12 +25,14 @@ class FilterRequirementsTest extends \PHPUnit\Framework\TestCase
 
     protected function setUp(): void
     {
-        $this->requirements = new FilterRequirements(new Plugin('filter_activitynames', 'filter', 'activitynames', ''), 29);
+        $this->requirements404 = new FilterRequirements(new Plugin('filter_activitynames', 'filter', 'activitynames', ''), 404);
+        $this->requirements    = new FilterRequirements(new Plugin('filter_activitynames', 'filter', 'activitynames', ''), 405);
     }
 
     protected function tearDown(): void
     {
-        $this->requirements = null;
+        $this->requirements404 = null;
+        $this->requirements    = null;
     }
 
     public function testResolveRequirements()
@@ -39,16 +41,27 @@ public function testResolveRequirements()
 
         $this->assertInstanceOf(
             'MoodlePluginCI\PluginValidate\Requirements\FilterRequirements',
-            $resolver->resolveRequirements(new Plugin('', 'filter', '', ''), 29)
+            $resolver->resolveRequirements(new Plugin('', 'filter', '', ''), 404)
         );
     }
 
+    public function testGetRequiredFiles404()
+    {
+        $files = $this->requirements404->getRequiredFiles();
+
+        $this->assertTrue(in_array('filter.php', $files, true));
+        $this->assertFalse(in_array('classes/text_filter.php', $files, true));
+        foreach ($files as $file) {
+            $this->assertIsString($file);
+        }
+    }
+
     public function testGetRequiredFiles()
     {
         $files = $this->requirements->getRequiredFiles();
 
-        $this->assertNotEmpty($files);
-        $this->assertTrue(in_array('filter.php', $files, true));
+        $this->assertFalse(in_array('filter.php', $files, true));
+        $this->assertTrue(in_array('classes/text_filter.php', $files, true));
         foreach ($files as $file) {
             $this->assertIsString($file);
         }
@@ -58,10 +71,10 @@ public function testGetRequiredClasses()
     {
         $classes = $this->requirements->getRequiredClasses();
 
-        $this->assertNotEmpty($classes);
-        foreach ($classes as $class) {
-            $this->assertInstanceOf('MoodlePluginCI\PluginValidate\Finder\FileTokens', $class);
-        }
+        $this->assertCount(1, $classes);
+        $class = reset($classes);
+        $this->assertInstanceOf('MoodlePluginCI\PluginValidate\Finder\FileTokens', $class);
+        $this->assertSame('classes/text_filter.php', $class->file);
     }
 
     public function testGetRequiredStrings()