Tokenizer/PHP: readonly vs union types bug fix

Done some more stress testing and found that the retokenization of `|` to `T_TYPE_UNION` was broken when combined with the `readonly` keyword.

Fixed now, includes test.

Author: jrfnl

src/Tokenizers/PHP.php

@@ -2710,7 +2710,8 @@ protected function processAdditional()
 
                     if ($suspectedType === 'property or parameter'
                         && (isset(Util\Tokens::$scopeModifiers[$this->tokens[$x]['code']]) === true
-                        || $this->tokens[$x]['code'] === T_VAR)
+                        || $this->tokens[$x]['code'] === T_VAR
+                        || $this->tokens[$x]['code'] === T_READONLY)
                     ) {
                         // This will also confirm constructor property promotion parameters, but that's fine.
                         $confirmed = true;

tests/Core/Tokenizer/BitwiseOrTest.inc

@@ -33,6 +33,9 @@ class TypeUnion
     /* testTypeUnionPropertyFullyQualified */
     public \Fully\Qualified\NameA|\Fully\Qualified\NameB $fullyQual;
 
+    /* testTypeUnionPropertyWithReadOnlyKeyword */
+    protected readonly string|null $array;
+
     public function paramTypes(
         /* testTypeUnionParam1 */
         int|float $paramA /* testBitwiseOrParamDefaultValue */ = CONSTANT_A | CONSTANT_B,

tests/Core/Tokenizer/BitwiseOrTest.php

@@ -105,6 +105,7 @@ public function dataTypeUnion()
             ['/* testTypeUnionPropertyNamespaceRelative */'],
             ['/* testTypeUnionPropertyPartiallyQualified */'],
             ['/* testTypeUnionPropertyFullyQualified */'],
+            ['/* testTypeUnionPropertyWithReadOnlyKeyword */'],
             ['/* testTypeUnionParam1 */'],
             ['/* testTypeUnionParam2 */'],
             ['/* testTypeUnionParam3 */'],