sunrise-php / http-router

src/ServerRequest.php

Spacing +2 added lines, -2 removed lines

@@ -55,7 +55,7 @@ 
     {
         $route = $this->request->getAttribute(RouteInterface::class);
 
-        if (! $route instanceof RouteInterface) {
+        if (!$route instanceof RouteInterface) {
             // phpcs:ignore Generic.Files.LineLength.TooLong
             throw new LogicException('At this level of the application, the request does not contain information about the requested route.');
         }
@@ -152,7 +152,7 @@ 
         // @codeCoverageIgnoreStart
         if (!extension_loaded('intl')) {
             throw new LogicException(
-                'To get the locales consumed by the client, ' .
+                'To get the locales consumed by the client, '.
                 'the Intl (https://www.php.net/intl) extension must be installed.'
             );
         }

src/Loader/DescriptorLoader.php

Spacing +6 added lines, -6 removed lines

@@ -166,7 +166,7 @@ 
         }
 
         throw new InvalidArgumentException(sprintf(
-            'The loader "%s" only accepts directory, file or class names; ' .
+            'The loader "%s" only accepts directory, file or class names; '.
             'however, the resource "%s" is not one of them.',
             self::class,
             $resource,
@@ -210,7 +210,7 @@ 
      *
      * @throws InvalidArgumentException
      */
-    private static function getClassOrMethodDescriptor(ReflectionClass|ReflectionMethod $classOrMethod): ?Descriptor
+    private static function getClassOrMethodDescriptor(ReflectionClass | ReflectionMethod $classOrMethod): ?Descriptor
     {
         /** @var list<ReflectionAttribute<Descriptor>> $annotations */
         $annotations = $classOrMethod->getAttributes(Descriptor::class, ReflectionAttribute::IS_INSTANCEOF);
@@ -236,7 +236,7 @@ 
      */
     private static function enrichDescriptorFromClassOrMethod(
         Descriptor $descriptor,
-        ReflectionClass|ReflectionMethod $classOrMethod,
+        ReflectionClass | ReflectionMethod $classOrMethod,
     ): void {
         /** @var list<ReflectionAttribute<NamePrefix>> $annotations */
         $annotations = $classOrMethod->getAttributes(NamePrefix::class);
@@ -351,11 +351,11 @@ 
      *
      * @throws InvalidArgumentException
      */
-    private static function completeDescriptor(Descriptor $descriptor, ReflectionClass|ReflectionMethod $holder): void
+    private static function completeDescriptor(Descriptor $descriptor, ReflectionClass | ReflectionMethod $holder): void
     {
         $descriptor->holder = $holder instanceof ReflectionClass ? $holder->name : [$holder->class, $holder->name];
-        $descriptor->name = implode($descriptor->namePrefixes) . $descriptor->name;
-        $descriptor->path = implode($descriptor->pathPrefixes) . $descriptor->path;
+        $descriptor->name = implode($descriptor->namePrefixes).$descriptor->name;
+        $descriptor->path = implode($descriptor->pathPrefixes).$descriptor->path;
         $descriptor->methods = array_map(strtoupper(...), $descriptor->methods);
         $descriptor->pattern = RouteCompiler::compileRoute($descriptor->path, $descriptor->patterns);
     }

src/Helper/RouteBuilder.php

Spacing +1 added lines, -1 removed lines

@@ -99,7 +99,7 @@
         }
 
         throw new InvalidArgumentException(sprintf(
-            'The "%s" value could not be converted to a string; ' .
+            'The "%s" value could not be converted to a string; '.
             'supported types are: string, integer, backed enum and stringable object.',
             get_debug_type($value),
         ));

src/Helper/RouteMatcher.php

Spacing +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
             }
 
             throw new InvalidArgumentException(sprintf(
-                'The route "%s" could not be matched due to: %s. ' .
+                'The route "%s" could not be matched due to: %s. '.
                 'This problem is most likely related to one of the route patterns.',
                 $pattern,
                 preg_last_error_msg(),

src/Helper/RouteSimplifier.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
         $replace = [];
         foreach ($variables as $variable) {
             $search[] = $variable['statement'];
-            $replace[] = '{' . $variable['name'] . '}';
+            $replace[] = '{'.$variable['name'].'}';
         }
 
         // will be replaced by an empty string:

src/Helper/RouteParser.php

Spacing +36 added lines, -36 removed lines

@@ -66,8 +66,8 @@ 
             if ($route[$offset] === '(' && !($cursor & self::IN_VARIABLE)) {
                 if (($cursor & self::IN_OPTIONAL_PART)) {
                     throw new InvalidArgumentException(sprintf(
-                        'The route "%s" could not be parsed due to a syntax error. ' .
-                        'The attempt to open an optional part at position %d failed ' .
+                        'The route "%s" could not be parsed due to a syntax error. '.
+                        'The attempt to open an optional part at position %d failed '.
                         'because nested optional parts are not supported.',
                         $route,
                         $offset,
@@ -80,8 +80,8 @@ 
             if ($route[$offset] === ')' && !($cursor & self::IN_VARIABLE)) {
                 if (!($cursor & self::IN_OPTIONAL_PART)) {
                     throw new InvalidArgumentException(sprintf(
-                        'The route "%s" could not be parsed due to a syntax error. ' .
-                        'The attempt to close an optional part at position %d failed ' .
+                        'The route "%s" could not be parsed due to a syntax error. '.
+                        'The attempt to close an optional part at position %d failed '.
                         'because an open optional part was not found.',
                         $route,
                         $offset,
@@ -91,7 +91,7 @@ 
                 if (($cursor & self::IN_OCCUPIED_PART)) {
                     $cursor &= ~self::IN_OCCUPIED_PART;
                     // phpcs:ignore Generic.Files.LineLength.TooLong
-                    $variables[$variable]['optional_part'] = '(' . $left . ($variables[$variable]['statement'] ?? '') . $right . ')';
+                    $variables[$variable]['optional_part'] = '('.$left.($variables[$variable]['statement'] ?? '').$right.')';
                 }
 
                 $cursor &= ~self::IN_OPTIONAL_PART;
@@ -102,8 +102,8 @@ 
             if ($route[$offset] === '{' && !($cursor & self::IN_VARIABLE_PATTERN)) {
                 if (($cursor & self::IN_VARIABLE)) {
                     throw new InvalidArgumentException(sprintf(
-                        'The route "%s" could not be parsed due to a syntax error. ' .
-                        'The attempt to open a variable at position %d failed ' .
+                        'The route "%s" could not be parsed due to a syntax error. '.
+                        'The attempt to open a variable at position %d failed '.
                         'because nested variables are not supported.',
                         $route,
                         $offset,
@@ -111,8 +111,8 @@ 
                 }
                 if (($cursor & self::IN_OCCUPIED_PART)) {
                     throw new InvalidArgumentException(sprintf(
-                        'The route "%s" could not be parsed due to a syntax error. ' .
-                        'The attempt to open a variable at position %d failed ' .
+                        'The route "%s" could not be parsed due to a syntax error. '.
+                        'The attempt to open a variable at position %d failed '.
                         'because more than one variable inside an optional part is not supported.',
                         $route,
                         $offset,
@@ -130,8 +130,8 @@ 
             if ($route[$offset] === '}' && !($cursor & self::IN_VARIABLE_PATTERN)) {
                 if (!($cursor & self::IN_VARIABLE)) {
                     throw new InvalidArgumentException(sprintf(
-                        'The route "%s" could not be parsed due to a syntax error. ' .
-                        'The attempt to close a variable at position %d failed ' .
+                        'The route "%s" could not be parsed due to a syntax error. '.
+                        'The attempt to close a variable at position %d failed '.
                         'because an open variable was not found.',
                         $route,
                         $offset,
@@ -139,8 +139,8 @@ 
                 }
                 if (!isset($variables[$variable]['name'])) {
                     throw new InvalidArgumentException(sprintf(
-                        'The route "%s" could not be parsed due to a syntax error. ' .
-                        'The attempt to close a variable at position %d failed ' .
+                        'The route "%s" could not be parsed due to a syntax error. '.
+                        'The attempt to close a variable at position %d failed '.
                         'because its name is required for its declaration.',
                         $route,
                         $offset,
@@ -148,7 +148,7 @@ 
                 }
                 if (isset($names[$variables[$variable]['name']])) {
                     throw new InvalidArgumentException(sprintf(
-                        'The route "%s" at position %d could not be parsed ' .
+                        'The route "%s" at position %d could not be parsed '.
                         'because the variable name "%s" is already in use.',
                         $route,
                         $offset,
@@ -157,7 +157,7 @@ 
                 }
 
                 $cursor &= ~(self::IN_VARIABLE | self::IN_VARIABLE_NAME);
-                $variables[$variable]['statement'] = '{' . ($variables[$variable]['statement'] ?? '') . '}';
+                $variables[$variable]['statement'] = '{'.($variables[$variable]['statement'] ?? '').'}';
                 $names[$variables[$variable]['name']] = true; // @phpstan-ignore-line
                 continue;
             }
@@ -170,8 +170,8 @@ 
             if ($route[$offset] === '<' && ($cursor & self::IN_VARIABLE)) {
                 if (($cursor & self::IN_VARIABLE_PATTERN)) {
                     throw new InvalidArgumentException(sprintf(
-                        'The route "%s" could not be parsed due to a syntax error. ' .
-                        'The attempt to open a variable pattern at position %d failed ' .
+                        'The route "%s" could not be parsed due to a syntax error. '.
+                        'The attempt to open a variable pattern at position %d failed '.
                         'because nested patterns are not supported.',
                         $route,
                         $offset,
@@ -179,8 +179,8 @@ 
                 }
                 if (!($cursor & self::IN_VARIABLE_NAME)) {
                     throw new InvalidArgumentException(sprintf(
-                        'The route "%s" could not be parsed due to a syntax error. ' .
-                        'The attempt to open a variable pattern at position %d failed ' .
+                        'The route "%s" could not be parsed due to a syntax error. '.
+                        'The attempt to open a variable pattern at position %d failed '.
                         'because the pattern must be preceded by the variable name.',
                         $route,
                         $offset,
@@ -193,8 +193,8 @@ 
             if ($route[$offset] === '>' && ($cursor & self::IN_VARIABLE)) {
                 if (!($cursor & self::IN_VARIABLE_PATTERN)) {
                     throw new InvalidArgumentException(sprintf(
-                        'The route "%s" could not be parsed due to a syntax error. ' .
-                        'The attempt to close a variable pattern at position %d failed ' .
+                        'The route "%s" could not be parsed due to a syntax error. '.
+                        'The attempt to close a variable pattern at position %d failed '.
                         'because an open pattern was not found.',
                         $route,
                         $offset,
@@ -202,8 +202,8 @@ 
                 }
                 if (!isset($variables[$variable]['pattern'])) {
                     throw new InvalidArgumentException(sprintf(
-                        'The route "%s" could not be parsed due to a syntax error. ' .
-                        'The attempt to close a variable pattern at position %d failed ' .
+                        'The route "%s" could not be parsed due to a syntax error. '.
+                        'The attempt to close a variable pattern at position %d failed '.
                         'because its content is required for its declaration.',
                         $route,
                         $offset,
@@ -230,8 +230,8 @@ 
             if (($cursor & self::IN_VARIABLE_NAME)) {
                 if (!isset($variables[$variable]['name']) && $route[$offset] >= '0' && $route[$offset] <= '9') {
                     throw new InvalidArgumentException(sprintf(
-                        'The route "%s" could not be parsed due to a syntax error. ' .
-                        'An invalid character was found at position %d. ' .
+                        'The route "%s" could not be parsed due to a syntax error. '.
+                        'An invalid character was found at position %d. '.
                         'Please note that variable names cannot start with digits.',
                         $route,
                         $offset,
@@ -239,8 +239,8 @@ 
                 }
                 if (!isset(self::PCRE_SUBPATTERN_NAME_CHARSET[$route[$offset]])) {
                     throw new InvalidArgumentException(sprintf(
-                        'The route "%s" could not be parsed due to a syntax error. ' .
-                        'An invalid character was found at position %d. ' .
+                        'The route "%s" could not be parsed due to a syntax error. '.
+                        'An invalid character was found at position %d. '.
                         'Please note that variable names must consist only of digits, letters and underscores.',
                         $route,
                         $offset,
@@ -248,8 +248,8 @@ 
                 }
                 if (isset($variables[$variable]['name'][31])) {
                     throw new InvalidArgumentException(sprintf(
-                        'The route "%s" could not be parsed due to a syntax error. ' .
-                        'An extra character was found at position %d. ' .
+                        'The route "%s" could not be parsed due to a syntax error. '.
+                        'An extra character was found at position %d. '.
                         'Please note that variable names must not exceed 32 characters.',
                         $route,
                         $offset,
@@ -264,9 +264,9 @@ 
             if (($cursor & self::IN_VARIABLE_PATTERN)) {
                 if ($route[$offset] === RouteCompiler::EXPRESSION_DELIMITER) {
                     throw new InvalidArgumentException(sprintf(
-                        'The route "%s" could not be parsed due to a syntax error. ' .
-                        'An invalid character was found at position %d. ' .
-                        'Please note that variable patterns cannot contain the character "%s"; ' .
+                        'The route "%s" could not be parsed due to a syntax error. '.
+                        'An invalid character was found at position %d. '.
+                        'Please note that variable patterns cannot contain the character "%s"; '.
                         'use an octal or hexadecimal sequence instead.',
                         $route,
                         $offset,
@@ -283,8 +283,8 @@ 
             // ~~~~~~~~~^^^~
             if (($cursor & self::IN_VARIABLE)) {
                 throw new InvalidArgumentException(sprintf(
-                    'The route "%s" could not be parsed due to a syntax error. ' .
-                    'An unexpected character was found at position %d; ' .
+                    'The route "%s" could not be parsed due to a syntax error. '.
+                    'An unexpected character was found at position %d; '.
                     'a variable at this position must be closed.',
                     $route,
                     $offset,
@@ -294,8 +294,8 @@ 
 
         if (($cursor & self::IN_VARIABLE) || ($cursor & self::IN_OPTIONAL_PART)) {
             throw new InvalidArgumentException(sprintf(
-                'The route "%s" could not be parsed due to a syntax error. ' .
-                'The attempt to parse the route failed ' .
+                'The route "%s" could not be parsed due to a syntax error. '.
+                'The attempt to parse the route failed '.
                 'because it contains an unclosed variable or optional part.',
                 $route,
             ));

src/Helper/RouteCompiler.php

Spacing +4 added lines, -4 removed lines

@@ -41,7 +41,7 @@ 
         $replace = [];
         foreach ($variables as $variable) {
             $search[] = $variable['statement'];
-            $replace[] = '{' . $variable['name'] . '}';
+            $replace[] = '{'.$variable['name'].'}';
         }
 
         $route = str_replace($search, $replace, $route);
@@ -53,12 +53,12 @@ 
         foreach ($variables as $variable) {
             $pattern = $patterns[$variable['name']] ?? $variable['pattern'] ?? self::DEFAULT_VARIABLE_PATTERN;
 
-            $search[] = '{' . $variable['name'] . '}';
-            $replace[] = '(?<' . $variable['name'] . '>' . $pattern . ')';
+            $search[] = '{'.$variable['name'].'}';
+            $replace[] = '(?<'.$variable['name'].'>'.$pattern.')';
         }
 
         $route = str_replace($search, $replace, $route);
 
-        return '#^' . $route . '$#uD';
+        return '#^'.$route.'$#uD';
     }
 }