TYPO3 / Fluid

src/Core/Compiler/NodeConverter.php

Doc Comments -2 removed lines

@@ -159,7 +159,6 @@ 
      * Convert a single ViewHelperNode into its cached representation. If the ViewHelper implements the "Compilable" facet,
      * the ViewHelper itself is asked for its cached PHP code representation. If not, a ViewHelper is built and then invoked.
      *
-     * @param ViewHelperNode $node
      * @return array
      * @see convert()
      */
@@ -315,7 +314,6 @@ 
     }
 
     /**
-     * @param ArrayNode $node
      * @return array
      * @see convert()
      */

Unused Use Statements -1 removed lines

@@ -15,7 +15,6 @@
 use TYPO3Fluid\Fluid\Core\Parser\SyntaxTree\NodeInterface;
 use TYPO3Fluid\Fluid\Core\Parser\SyntaxTree\NumericNode;
 use TYPO3Fluid\Fluid\Core\Parser\SyntaxTree\ObjectAccessorNode;
-use TYPO3Fluid\Fluid\Core\Parser\SyntaxTree\PostponedViewHelperNode;
 use TYPO3Fluid\Fluid\Core\Parser\SyntaxTree\RootNode;
 use TYPO3Fluid\Fluid\Core\Parser\SyntaxTree\TextNode;
 use TYPO3Fluid\Fluid\Core\Parser\SyntaxTree\ViewHelperNode;

src/Core/Parser/Sequencer.php

Doc Comments +2 added lines, -2 removed lines

@@ -685,7 +685,7 @@ 
     }
 
     /**
-     * @param \Iterator|Position[] $sequence
+     * @param \Iterator $sequence
      * @param ArgumentDefinition[] $definitions
      * @param bool $numeric
      * @return ArrayNode
@@ -838,7 +838,7 @@ 
      * be a node as such - which is only necessary if the quoted expression
      * contains other (dynamic) values like an inline syntax.
      *
-     * @param \Iterator|Position[] $sequence
+     * @param \Iterator $sequence
      * @param int $leadingEscapes A backwards compatibility measure: when passed, this number of escapes must precede a closing quote for it to trigger node closing.
      * @param bool $allowArray
      * @return RootNode

Spacing +8 added lines, -8 removed lines

@@ -129,7 +129,7 @@ 
     {
         $position = new Position($this->splitter->context, $this->splitter->index);
         $ascii = (string) $this->source->bytes[$this->splitter->index];
-        $message .=  ' ASCII: ' . $ascii . ': ' . $this->extractSourceDumpOfLineAtPosition($position);
+        $message .= ' ASCII: ' . $ascii . ': ' . $this->extractSourceDumpOfLineAtPosition($position);
         $error = new SequencingException($message, $code);
         return $error;
     }
@@ -542,7 +542,7 @@ 
                     $isArray = $allowArray;
                     $callDetected = false;
                     $potentialAccessor = $potentialAccessor ?? $captured;
-                    $text .=  $this->source->source[$this->splitter->index - 1];
+                    $text .= $this->source->source[$this->splitter->index - 1];
                     if (isset($potentialAccessor)) {
                         $childNodeToAdd = new ObjectAccessorNode($potentialAccessor);
                         $node = isset($node) ? $node->addChildNode($childNodeToAdd) : $childNodeToAdd; //$node ?? (is_numeric($potentialAccessor) ? $potentialAccessor + 0 : new ObjectAccessorNode($potentialAccessor));
@@ -577,7 +577,7 @@ 
                     $this->splitter->switch($this->contexts->inline);
                     if ($childNodeToAdd) {
                         $escapingEnabledBackup = $this->escapingEnabled;
-                        $this->escapingEnabled = (bool)$node->isChildrenEscapingEnabled();
+                        $this->escapingEnabled = (bool) $node->isChildrenEscapingEnabled();
                         if ($childNodeToAdd instanceof ObjectAccessorNode) {
                             $this->callInterceptor($childNodeToAdd, InterceptorInterface::INTERCEPT_OBJECTACCESSOR);
                         }
@@ -650,7 +650,7 @@ 
                             $node = new TextNode($text);
                             $interceptionPoint = InterceptorInterface::INTERCEPT_TEXT;
                         }
-                    } elseif ($hasPass && $this->resolver->isAliasRegistered((string)$potentialAccessor)) {
+                    } elseif ($hasPass && $this->resolver->isAliasRegistered((string) $potentialAccessor)) {
                         // Fourth priority check is for a pass to a ViewHelper alias, e.g. "{value | raw}" in which case
                         // we look for the alias used and create a ViewHelperNode with no arguments.
                         $childNodeToAdd = $node;
@@ -671,7 +671,7 @@ 
                     }
 
                     $escapingEnabledBackup = $this->escapingEnabled;
-                    $this->escapingEnabled = (bool)((isset($viewHelper) && $node->isOutputEscapingEnabled()) || $escapingEnabledBackup);
+                    $this->escapingEnabled = (bool) ((isset($viewHelper) && $node->isOutputEscapingEnabled()) || $escapingEnabledBackup);
                     $this->callInterceptor($node, $interceptionPoint, $this->state);
                     $this->escapingEnabled = $escapingEnabledBackup;
                     return $node;
@@ -714,7 +714,7 @@ 
                         throw $this->createErrorAtPosition('Unexpected colon or equals sign, no preceding key', 1559250839);
                     }
                     if ($definitions !== null && !$numeric && !isset($definitions[$key])) {
-                        throw $this->createUnsupportedArgumentError((string)$key, $definitions);
+                        throw $this->createUnsupportedArgumentError((string) $key, $definitions);
                     }
                     break;
 
@@ -771,7 +771,7 @@ 
                     } elseif (isset($captured)) {
                         $key = $key ?? ($numeric ? ++$itemCount : $captured);
                         if (!$numeric && isset($definitions) && !isset($definitions[$key])) {
-                            throw $this->createUnsupportedArgumentError((string)$key, $definitions);
+                            throw $this->createUnsupportedArgumentError((string) $key, $definitions);
                         }
                         $array[$key] = is_numeric($captured) ? $captured + 0 : new ObjectAccessorNode($captured);
                     }
@@ -811,7 +811,7 @@ 
                         }
                     }
                     if (!$numeric && isset($key, $definitions) && !isset($definitions[$key])) {
-                        throw $this->createUnsupportedArgumentError((string)$key, $definitions);
+                        throw $this->createUnsupportedArgumentError((string) $key, $definitions);
                     }
                     $this->escapingEnabled = $escapingEnabledBackup;
                     return new ArrayNode($array);

src/ViewHelpers/CViewHelper.php

Unused Use Statements -2 removed lines

@@ -6,9 +6,7 @@
  * See LICENSE.txt that was shipped with this package.
  */
 
-use TYPO3Fluid\Fluid\Core\Parser\SyntaxTree\ViewHelperNode;
 use TYPO3Fluid\Fluid\Core\Rendering\RenderingContextInterface;
-use TYPO3Fluid\Fluid\Core\Variables\VariableProviderInterface;
 use TYPO3Fluid\Fluid\Core\ViewHelper\AbstractViewHelper;
 
 class CViewHelper extends AbstractViewHelper

src/Core/Parser/SyntaxTree/ViewHelperNode.php

Spacing +1 added lines, -1 removed lines

@@ -200,7 +200,7 @@
                     && $argumentsObjectTree[$argumentName] !== null) {
                     $argumentsObjectTree[$argumentName] = new BooleanNode($argumentsObjectTree[$argumentName]);
                 } else {
-                    $argumentsObjectTree[$argumentName] = (bool)$argumentsObjectTree[$argumentName];
+                    $argumentsObjectTree[$argumentName] = (bool) $argumentsObjectTree[$argumentName];
                 }
             }
         }

src/Core/Parser/Context.php

Spacing +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
     public const CONTEXT_ROOT = 0;
     public const CONTEXT_INLINE = 1;
     public const CONTEXT_TAG = 2;
-    public const CONTEXT_ARRAY= 3;
+    public const CONTEXT_ARRAY = 3;
     public const CONTEXT_QUOTED = 4;
     public const CONTEXT_ATTRIBUTES = 5;
     public const CONTEXT_DEAD = 6;

src/Core/Parser/Configuration.php

Spacing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
     {
         $previous = $this->features[$feature];
         if (is_bool($state) || is_numeric($state) || is_null($state)) {
-            $this->features[$feature] = (bool)$state;
+            $this->features[$feature] = (bool) $state;
         } elseif (is_string($state)) {
             $this->features[$feature] = in_array(strtolower($state), ['on', 'true', 'enabled']);
         }

src/Core/Parser/Splitter.php

Spacing +1 added lines, -1 removed lines

@@ -123,7 +123,7 @@
         }
     }
 
-    public function switch(Context $context): Context
+    public function switch (Context $context): Context
     {
         $previous = $this->context;
         $this->context = $context;

src/Core/NewParser/Debugger.php

Spacing +3 added lines, -3 removed lines

@@ -77,7 +77,7 @@ 
         $symbolLine = '';
         foreach ($symbols as $index => $symbol) {
             $char = $symbol === Splitter::BYTE_BACKSLASH ? '\\' : chr($symbol);
-            $capturedLength = strlen((string)$captures[$index]) + 1;
+            $capturedLength = strlen((string) $captures[$index]) + 1;
             $symbolLine .= $colors[$contexts[$index]] ?? "\033[01;0m";
             $symbolLine .= str_repeat(' ', max($capturedLength - 1, 0)) . $char . $spacing;
             #echo str_repeat((string)$contexts[$index], $capturedLength > 0 ? $capturedLength : 1);
@@ -100,8 +100,8 @@ 
 
         $nestingLine = '';
         foreach ($nesting as $index => $depth) {
-            $capturedLength = strlen(str_replace(PHP_EOL, '', (string)$captures[$index])) + 1;
-            $nestingLine .= str_repeat((string)$depth, ($capturedLength > 0 ? $capturedLength : 0)) . $spacing;
+            $capturedLength = strlen(str_replace(PHP_EOL, '', (string) $captures[$index])) + 1;
+            $nestingLine .= str_repeat((string) $depth, ($capturedLength > 0 ? $capturedLength : 0)) . $spacing;
         }
 
         $this->writeLogLine($nestingLine);

src/Core/ViewHelper/AbstractConditionViewHelper.php

Spacing +2 added lines, -2 removed lines

@@ -123,7 +123,7 @@ 
      */
     protected static function evaluateCondition($arguments = null)
     {
-        return isset($arguments['condition']) && (bool)($arguments['condition']);
+        return isset($arguments['condition']) && (bool) ($arguments['condition']);
     }
 
     /**
@@ -209,7 +209,7 @@ 
                     if ($condition instanceof NodeInterface) {
                         $condition = $condition->evaluate($this->renderingContext);
                     }
-                    if ((bool)$condition === true) {
+                    if ((bool) $condition === true) {
                         return $childNode->evaluate($this->renderingContext);
                     }
                 } else {