- detect deprecations in PHPUnit,

- fixed all instances of PHP 8.4 deprecated implicitly marking parameters as nullable.

Author: thunderer

Diff for: phpunit.xml.dist

@@ -1,10 +1,10 @@
 <?xml version="1.0" encoding="UTF-8" ?>
 <phpunit
-    colors = "true"
-    backupGlobals = "false"
-    processIsolation = "false"
-    stopOnFailure = "false"
-    bootstrap = "vendor/autoload.php"
+    colors="true"
+    failOnNotice="true"
+    failOnWarning="true"
+    failOnDeprecation="true"
+    bootstrap="vendor/autoload.php"
     xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
     xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.3/phpunit.xsd">
 

Diff for: src/Event/FilterShortcodesEvent.php

@@ -18,9 +18,16 @@ class FilterShortcodesEvent
     /** @var ParsedShortcodeInterface[] */
     private $shortcodes = array();
 
-    /** @param ParsedShortcodeInterface[] $shortcodes */
-    public function __construct(array $shortcodes, ProcessedShortcode $parent = null)
+    /**
+     * @param ParsedShortcodeInterface[] $shortcodes
+     * @param ProcessedShortcode|null $parent
+     */
+    public function __construct(array $shortcodes, $parent = null)
     {
+        if(null !== $parent && false === $parent instanceof ProcessedShortcode) {
+            throw new \LogicException('Parameter $parent must be an instance of ProcessedShortcode.');
+        }
+
         $this->parent = $parent;
         $this->setShortcodes($shortcodes);
     }

Diff for: src/Event/ReplaceShortcodesEvent.php

@@ -25,9 +25,14 @@ class ReplaceShortcodesEvent
     /**
      * @param string $text
      * @param ReplacedShortcode[] $replacements
+     * @param ShortcodeInterface|null $shortcode
      */
-    public function __construct($text, array $replacements, ShortcodeInterface $shortcode = null)
+    public function __construct($text, array $replacements, $shortcode = null)
     {
+        if(null !== $shortcode && false === $shortcode instanceof ShortcodeInterface) {
+            throw new \LogicException('Parameter $shortcode must be an instance of ShortcodeInterface.');
+        }
+
         $this->shortcode = $shortcode;
         $this->text = $text;
 

Diff for: src/Parser/RegexParser.php

@@ -21,8 +21,13 @@ final class RegexParser implements ParserInterface
     /** @var non-empty-string */
     private $parametersRegex;
 
-    public function __construct(SyntaxInterface $syntax = null)
+    /** @param SyntaxInterface|null $syntax */
+    public function __construct($syntax = null)
     {
+        if(null !== $syntax && false === $syntax instanceof SyntaxInterface) {
+            throw new \LogicException('Parameter $syntax must be an instance of SyntaxInterface.');
+        }
+
         $this->syntax = $syntax ?: new Syntax();
         $this->shortcodeRegex = RegexBuilderUtility::buildShortcodeRegex($this->syntax);
         $this->singleShortcodeRegex = RegexBuilderUtility::buildSingleShortcodeRegex($this->syntax);

Diff for: src/Parser/RegularParser.php

@@ -35,8 +35,13 @@ final class RegularParser implements ParserInterface
     const TOKEN_STRING = 6;
     const TOKEN_WS = 7;
 
-    public function __construct(SyntaxInterface $syntax = null)
+    /** @param SyntaxInterface|null $syntax */
+    public function __construct($syntax = null)
     {
+        if(null !== $syntax && false === $syntax instanceof SyntaxInterface) {
+            throw new \LogicException('Parameter $syntax must be an instance of SyntaxInterface.');
+        }
+
         $this->lexerRegex = $this->prepareLexer($syntax ?: new CommonSyntax());
         $this->nameRegex = '~^'.RegexBuilderUtility::buildNameRegex().'$~us';
     }

Diff for: src/Processor/Processor.php

@@ -86,11 +86,15 @@ private function dispatchEvent($name, $event)
 
     /**
      * @param string $text
+     * @param ProcessedShortcode|null $parent
      *
      * @return string
      */
-    private function processIteration($text, ProcessorContext $context, ProcessedShortcode $parent = null)
+    private function processIteration($text, ProcessorContext $context, $parent = null)
     {
+        if(null !== $parent && false === $parent instanceof ProcessedShortcode) {
+            throw new \LogicException('Parameter $parent must be an instance of ProcessedShortcode.');
+        }
         if (null !== $this->recursionDepth && $context->recursionLevel > $this->recursionDepth) {
             return $text;
         }

Diff for: src/Serializer/TextSerializer.php

@@ -15,8 +15,13 @@ final class TextSerializer implements SerializerInterface
     /** @var SyntaxInterface */
     private $syntax;
 
-    public function __construct(SyntaxInterface $syntax = null)
+    /** @param SyntaxInterface|null $syntax */
+    public function __construct($syntax = null)
     {
+        if(null !== $syntax && false === $syntax instanceof SyntaxInterface) {
+            throw new \LogicException('Parameter $syntax must be an instance of SyntaxInterface.');
+        }
+
         $this->syntax = $syntax ?: new Syntax();
     }