spiral / framework

src/Stempler/src/Traverser.php

Spacing +28 added lines, -28 removed lines

@@ -31,7 +31,7 @@ 
      */
     public function __construct(array $visitors = [])
     {
-        foreach ($visitors as $visitor) {
+        foreach ($visitors as $visitor){
             $this->addVisitor($visitor);
         }
     }
@@ -51,8 +51,8 @@ 
      */
     public function removeVisitor(VisitorInterface $visitor): void
     {
-        foreach ($this->visitors as $index => $added) {
-            if ($added === $visitor) {
+        foreach ($this->visitors as $index => $added){
+            if ($added === $visitor){
                 unset($this->visitors[$index]);
                 break;
             }
@@ -73,22 +73,22 @@ 
         $context = $context ?? new VisitorContext();
 
         $ctx = clone $context;
-        foreach ($nodes as $index => $node) {
-            if ($this->stopTraversal) {
+        foreach ($nodes as $index => $node){
+            if ($this->stopTraversal){
                 break;
             }
 
             $traverseChildren = true;
             $breakVisitorID = null;
 
-            if ($node instanceof NodeInterface) {
+            if ($node instanceof NodeInterface){
                 $ctx = $context->withNode($node);
             }
 
-            foreach ($this->visitors as $visitorID => $visitor) {
+            foreach ($this->visitors as $visitorID => $visitor){
                 $result = $visitor->enterNode($node, $ctx);
 
-                switch (true) {
+                switch (true){
                     case $result === null:
                         break;
 
@@ -112,23 +112,23 @@ 
 
                     default:
                         throw new \LogicException(
-                            'enterNode() returned invalid value of type ' . gettype($result)
+                            'enterNode() returned invalid value of type '.gettype($result)
                         );
                 }
             }
 
             // sub nodes
-            if ($traverseChildren && $node instanceof NodeInterface) {
+            if ($traverseChildren && $node instanceof NodeInterface){
                 $nodes[$index] = $this->traverseNode($node, $ctx);
-                if ($this->stopTraversal) {
+                if ($this->stopTraversal){
                     break;
                 }
             }
 
-            foreach ($this->visitors as $visitorID => $visitor) {
+            foreach ($this->visitors as $visitorID => $visitor){
                 $result = $visitor->leaveNode($node, $ctx);
 
-                switch (true) {
+                switch (true){
                     case $result === null:
                         break;
 
@@ -146,11 +146,11 @@ 
 
                     default:
                         throw new \LogicException(
-                            'leaveNode() returned invalid value of type ' . gettype($result)
+                            'leaveNode() returned invalid value of type '.gettype($result)
                         );
                 }
 
-                if ($breakVisitorID === $visitorID) {
+                if ($breakVisitorID === $visitorID){
                     break;
                 }
             }
@@ -169,18 +169,18 @@ 
     private function traverseNode(NodeInterface $node, VisitorContext $context): NodeInterface
     {
         $ctx = clone $context;
-        foreach ($node as $name => $_) {
+        foreach ($node as $name => $_){
             $child = &$node->$name;
-            if (is_array($child)) {
+            if (is_array($child)){
                 $child = $this->traverse($child, $ctx);
-                if ($this->stopTraversal) {
+                if ($this->stopTraversal){
                     break;
                 }
 
                 continue;
             }
 
-            if (!$child instanceof NodeInterface) {
+            if (!$child instanceof NodeInterface){
                 continue;
             }
 
@@ -189,9 +189,9 @@ 
             $traverseChildren = true;
             $breakVisitorID = null;
 
-            foreach ($this->visitors as $visitorID => $visitor) {
+            foreach ($this->visitors as $visitorID => $visitor){
                 $result = $visitor->enterNode($child, $ctx);
-                switch (true) {
+                switch (true){
                     case $result === null:
                         break;
 
@@ -214,22 +214,22 @@ 
 
                     default:
                         throw new \LogicException(
-                            'enterNode() returned invalid value of type ' . gettype($result)
+                            'enterNode() returned invalid value of type '.gettype($result)
                         );
                 }
             }
 
-            if ($traverseChildren) {
+            if ($traverseChildren){
                 $child = $this->traverseNode($child, $ctx);
-                if ($this->stopTraversal) {
+                if ($this->stopTraversal){
                     break;
                 }
             }
 
-            foreach ($this->visitors as $visitorID => $visitor) {
+            foreach ($this->visitors as $visitorID => $visitor){
                 $result = $visitor->leaveNode($child, $ctx);
 
-                switch (true) {
+                switch (true){
                     case $result === null:
                         break;
 
@@ -243,11 +243,11 @@ 
 
                     default:
                         throw new \LogicException(
-                            'leaveNode() returned invalid value of type ' . gettype($result)
+                            'leaveNode() returned invalid value of type '.gettype($result)
                         );
                 }
 
-                if ($breakVisitorID === $visitorID) {
+                if ($breakVisitorID === $visitorID){
                     break;
                 }
             }

Braces +48 added lines, -24 removed lines

@@ -31,7 +31,8 @@ 
      */
     public function __construct(array $visitors = [])
     {
-        foreach ($visitors as $visitor) {
+        foreach ($visitors as $visitor)
+        {
             $this->addVisitor($visitor);
         }
     }
@@ -51,8 +52,10 @@ 
      */
     public function removeVisitor(VisitorInterface $visitor): void
     {
-        foreach ($this->visitors as $index => $added) {
-            if ($added === $visitor) {
+        foreach ($this->visitors as $index => $added)
+        {
+            if ($added === $visitor)
+            {
                 unset($this->visitors[$index]);
                 break;
             }
@@ -73,22 +76,27 @@ 
         $context = $context ?? new VisitorContext();
 
         $ctx = clone $context;
-        foreach ($nodes as $index => $node) {
-            if ($this->stopTraversal) {
+        foreach ($nodes as $index => $node)
+        {
+            if ($this->stopTraversal)
+            {
                 break;
             }
 
             $traverseChildren = true;
             $breakVisitorID = null;
 
-            if ($node instanceof NodeInterface) {
+            if ($node instanceof NodeInterface)
+            {
                 $ctx = $context->withNode($node);
             }
 
-            foreach ($this->visitors as $visitorID => $visitor) {
+            foreach ($this->visitors as $visitorID => $visitor)
+            {
                 $result = $visitor->enterNode($node, $ctx);
 
-                switch (true) {
+                switch (true)
+                {
                     case $result === null:
                         break;
 
@@ -118,17 +126,21 @@ 
             }
 
             // sub nodes
-            if ($traverseChildren && $node instanceof NodeInterface) {
+            if ($traverseChildren && $node instanceof NodeInterface)
+            {
                 $nodes[$index] = $this->traverseNode($node, $ctx);
-                if ($this->stopTraversal) {
+                if ($this->stopTraversal)
+                {
                     break;
                 }
             }
 
-            foreach ($this->visitors as $visitorID => $visitor) {
+            foreach ($this->visitors as $visitorID => $visitor)
+            {
                 $result = $visitor->leaveNode($node, $ctx);
 
-                switch (true) {
+                switch (true)
+                {
                     case $result === null:
                         break;
 
@@ -150,7 +162,8 @@ 
                         );
                 }
 
-                if ($breakVisitorID === $visitorID) {
+                if ($breakVisitorID === $visitorID)
+                {
                     break;
                 }
             }
@@ -169,18 +182,22 @@ 
     private function traverseNode(NodeInterface $node, VisitorContext $context): NodeInterface
     {
         $ctx = clone $context;
-        foreach ($node as $name => $_) {
+        foreach ($node as $name => $_)
+        {
             $child = &$node->$name;
-            if (is_array($child)) {
+            if (is_array($child))
+            {
                 $child = $this->traverse($child, $ctx);
-                if ($this->stopTraversal) {
+                if ($this->stopTraversal)
+                {
                     break;
                 }
 
                 continue;
             }
 
-            if (!$child instanceof NodeInterface) {
+            if (!$child instanceof NodeInterface)
+            {
                 continue;
             }
 
@@ -189,9 +206,11 @@ 
             $traverseChildren = true;
             $breakVisitorID = null;
 
-            foreach ($this->visitors as $visitorID => $visitor) {
+            foreach ($this->visitors as $visitorID => $visitor)
+            {
                 $result = $visitor->enterNode($child, $ctx);
-                switch (true) {
+                switch (true)
+                {
                     case $result === null:
                         break;
 
@@ -219,17 +238,21 @@ 
                 }
             }
 
-            if ($traverseChildren) {
+            if ($traverseChildren)
+            {
                 $child = $this->traverseNode($child, $ctx);
-                if ($this->stopTraversal) {
+                if ($this->stopTraversal)
+                {
                     break;
                 }
             }
 
-            foreach ($this->visitors as $visitorID => $visitor) {
+            foreach ($this->visitors as $visitorID => $visitor)
+            {
                 $result = $visitor->leaveNode($child, $ctx);
 
-                switch (true) {
+                switch (true)
+                {
                     case $result === null:
                         break;
 
@@ -247,7 +270,8 @@ 
                         );
                 }
 
-                if ($breakVisitorID === $visitorID) {
+                if ($breakVisitorID === $visitorID)
+                {
                     break;
                 }
             }

src/Stempler/src/Builder.php

Spacing +15 added lines, -15 removed lines

@@ -118,14 +118,14 @@ 
      */
     public function compileTemplate(Template $tpl): Result
     {
-        try {
-            if (isset($this->visitors[self::STAGE_COMPILE])) {
+        try{
+            if (isset($this->visitors[self::STAGE_COMPILE])){
                 $traverser = new Traverser($this->visitors[self::STAGE_COMPILE]);
                 $tpl = $traverser->traverse([$tpl])[0];
             }
 
             return $this->compiler->compile($tpl);
-        } catch (CompilerException $e) {
+        }catch (CompilerException $e){
             throw $this->mapException($e);
         }
     }
@@ -141,21 +141,21 @@ 
         $source = $this->loader->load($path);
         $stream = new StringStream($source->getContent());
 
-        try {
+        try{
             $tpl = $this->parser->withPath($path)->parse($stream);
             $tpl->setContext(new Context(
                 new Token(Token::TYPE_RAW, 0, ''),
                 $path
             ));
-        } catch (ParserException $e) {
+        }catch (ParserException $e){
             throw $this->mapException($e);
         }
 
-        try {
+        try{
             return $this->process($tpl);
-        } catch (ContextExceptionInterface $e) {
+        }catch (ContextExceptionInterface $e){
             throw $this->mapException($e);
-        } catch (\Throwable $e) {
+        }catch (\Throwable $e){
             throw $e;
         }
     }
@@ -168,17 +168,17 @@ 
      */
     private function process(Template $template): Template
     {
-        if (isset($this->visitors[self::STAGE_PREPARE])) {
+        if (isset($this->visitors[self::STAGE_PREPARE])){
             $traverser = new Traverser($this->visitors[self::STAGE_PREPARE]);
             $template = $traverser->traverse([$template])[0];
         }
 
-        if (isset($this->visitors[self::STAGE_TRANSFORM])) {
+        if (isset($this->visitors[self::STAGE_TRANSFORM])){
             $traverser = new Traverser($this->visitors[self::STAGE_TRANSFORM]);
             $template = $traverser->traverse([$template])[0];
         }
 
-        if (isset($this->visitors[self::STAGE_FINALIZE])) {
+        if (isset($this->visitors[self::STAGE_FINALIZE])){
             $traverser = new Traverser($this->visitors[self::STAGE_FINALIZE]);
             $template = $traverser->traverse([$template])[0];
         }
@@ -194,17 +194,17 @@ 
      */
     private function mapException(ContextExceptionInterface $e): ContextExceptionInterface
     {
-        if ($e->getContext()->getPath() === null) {
+        if ($e->getContext()->getPath() === null){
             return $e;
         }
 
-        try {
+        try{
             $source = $this->loader->load($e->getContext()->getPath());
-        } catch (LoaderException $te) {
+        }catch (LoaderException $te){
             return $e;
         }
 
-        if ($source->getFilename() === null) {
+        if ($source->getFilename() === null){
             return $e;
         }
 

Braces +35 added lines, -15 removed lines

@@ -118,14 +118,18 @@ 
      */
     public function compileTemplate(Template $tpl): Result
     {
-        try {
-            if (isset($this->visitors[self::STAGE_COMPILE])) {
+        try
+        {
+            if (isset($this->visitors[self::STAGE_COMPILE]))
+            {
                 $traverser = new Traverser($this->visitors[self::STAGE_COMPILE]);
                 $tpl = $traverser->traverse([$tpl])[0];
             }
 
             return $this->compiler->compile($tpl);
-        } catch (CompilerException $e) {
+        }
+        catch (CompilerException $e)
+        {
             throw $this->mapException($e);
         }
     }
@@ -141,21 +145,29 @@ 
         $source = $this->loader->load($path);
         $stream = new StringStream($source->getContent());
 
-        try {
+        try
+        {
             $tpl = $this->parser->withPath($path)->parse($stream);
             $tpl->setContext(new Context(
                 new Token(Token::TYPE_RAW, 0, ''),
                 $path
             ));
-        } catch (ParserException $e) {
+        }
+        catch (ParserException $e)
+        {
             throw $this->mapException($e);
         }
 
-        try {
+        try
+        {
             return $this->process($tpl);
-        } catch (ContextExceptionInterface $e) {
+        }
+        catch (ContextExceptionInterface $e)
+        {
             throw $this->mapException($e);
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
             throw $e;
         }
     }
@@ -168,17 +180,20 @@ 
      */
     private function process(Template $template): Template
     {
-        if (isset($this->visitors[self::STAGE_PREPARE])) {
+        if (isset($this->visitors[self::STAGE_PREPARE]))
+        {
             $traverser = new Traverser($this->visitors[self::STAGE_PREPARE]);
             $template = $traverser->traverse([$template])[0];
         }
 
-        if (isset($this->visitors[self::STAGE_TRANSFORM])) {
+        if (isset($this->visitors[self::STAGE_TRANSFORM]))
+        {
             $traverser = new Traverser($this->visitors[self::STAGE_TRANSFORM]);
             $template = $traverser->traverse([$template])[0];
         }
 
-        if (isset($this->visitors[self::STAGE_FINALIZE])) {
+        if (isset($this->visitors[self::STAGE_FINALIZE]))
+        {
             $traverser = new Traverser($this->visitors[self::STAGE_FINALIZE]);
             $template = $traverser->traverse([$template])[0];
         }
@@ -194,17 +209,22 @@ 
      */
     private function mapException(ContextExceptionInterface $e): ContextExceptionInterface
     {
-        if ($e->getContext()->getPath() === null) {
+        if ($e->getContext()->getPath() === null)
+        {
             return $e;
         }
 
-        try {
+        try
+        {
             $source = $this->loader->load($e->getContext()->getPath());
-        } catch (LoaderException $te) {
+        }
+        catch (LoaderException $te)
+        {
             return $e;
         }
 
-        if ($source->getFilename() === null) {
+        if ($source->getFilename() === null)
+        {
             return $e;
         }
 

src/Stempler/src/Compiler.php

Spacing +4 added lines, -4 removed lines

@@ -41,16 +41,16 @@
     {
         $result = $result ?? new Result();
 
-        if (is_array($node)) {
-            foreach ($node as $child) {
+        if (is_array($node)){
+            foreach ($node as $child){
                 $this->compile($child, $result);
             }
 
             return $result;
         }
 
-        foreach ($this->renders as $renderer) {
-            if ($renderer->render($this, $result, $node)) {
+        foreach ($this->renders as $renderer){
+            if ($renderer->render($this, $result, $node)){
                 return $result;
             }
         }

Braces +8 added lines, -4 removed lines

@@ -41,16 +41,20 @@
     {
         $result = $result ?? new Result();
 
-        if (is_array($node)) {
-            foreach ($node as $child) {
+        if (is_array($node))
+        {
+            foreach ($node as $child)
+            {
                 $this->compile($child, $result);
             }
 
             return $result;
         }
 
-        foreach ($this->renders as $renderer) {
-            if ($renderer->render($this, $result, $node)) {
+        foreach ($this->renders as $renderer)
+        {
+            if ($renderer->render($this, $result, $node))
+            {
                 return $result;
             }
         }

src/Stempler/src/Directive/LoopDirective.php

Spacing +2 added lines, -2 removed lines

@@ -60,7 +60,7 @@ 
      */
     public function renderBreak(Directive $directive): string
     {
-        if (isset($directive->values[0])) {
+        if (isset($directive->values[0])){
             return sprintf('<?php break %s; ?>', $directive->values[0]);
         }
 
@@ -73,7 +73,7 @@ 
      */
     public function renderContinue(Directive $directive): string
     {
-        if (isset($directive->values[0])) {
+        if (isset($directive->values[0])){
             return sprintf('<?php continue %s; ?>', $directive->values[0]);
         }
 

Braces +4 added lines, -2 removed lines

@@ -60,7 +60,8 @@ 
      */
     public function renderBreak(Directive $directive): string
     {
-        if (isset($directive->values[0])) {
+        if (isset($directive->values[0]))
+        {
             return sprintf('<?php break %s; ?>', $directive->values[0]);
         }
 
@@ -73,7 +74,8 @@ 
      */
     public function renderContinue(Directive $directive): string
     {
-        if (isset($directive->values[0])) {
+        if (isset($directive->values[0]))
+        {
             return sprintf('<?php continue %s; ?>', $directive->values[0]);
         }
 

src/Stempler/src/Directive/ConditionalDirective.php

Spacing +3 added lines, -3 removed lines

@@ -128,7 +128,7 @@ 
      */
     public function renderCase(Directive $directive): string
     {
-        if ($this->firstSwitchCase) {
+        if ($this->firstSwitchCase){
             $this->firstSwitchCase = false;
 
             return sprintf('case (%s): ?>', $directive->body);
@@ -143,7 +143,7 @@ 
      */
     public function renderDefault(Directive $directive): string
     {
-        if ($this->firstSwitchCase) {
+        if ($this->firstSwitchCase){
             $this->firstSwitchCase = false;
 
             return 'default: ?>';
@@ -167,7 +167,7 @@ 
      */
     public function renderBreak(Directive $directive): string
     {
-        if (isset($directive->values[0])) {
+        if (isset($directive->values[0])){
             return sprintf('<?php break %s; ?>', $directive->values[0]);
         }
 

Braces +6 added lines, -3 removed lines

@@ -128,7 +128,8 @@ 
      */
     public function renderCase(Directive $directive): string
     {
-        if ($this->firstSwitchCase) {
+        if ($this->firstSwitchCase)
+        {
             $this->firstSwitchCase = false;
 
             return sprintf('case (%s): ?>', $directive->body);
@@ -143,7 +144,8 @@ 
      */
     public function renderDefault(Directive $directive): string
     {
-        if ($this->firstSwitchCase) {
+        if ($this->firstSwitchCase)
+        {
             $this->firstSwitchCase = false;
 
             return 'default: ?>';
@@ -167,7 +169,8 @@ 
      */
     public function renderBreak(Directive $directive): string
     {
-        if (isset($directive->values[0])) {
+        if (isset($directive->values[0]))
+        {
             return sprintf('<?php break %s; ?>', $directive->values[0]);
         }
 

src/Stempler/src/Directive/AbstractDirective.php

Spacing +3 added lines, -3 removed lines

@@ -36,7 +36,7 @@ 
      */
     public function hasDirective(string $name): bool
     {
-        return $this->r->hasMethod('render' . ucfirst($name));
+        return $this->r->hasMethod('render'.ucfirst($name));
     }
 
     /**
@@ -45,10 +45,10 @@ 
      */
     public function render(Directive $directive): ?string
     {
-        if (!$this->hasDirective($directive->name)) {
+        if (!$this->hasDirective($directive->name)){
             return null;
         }
 
-        return call_user_func([$this, 'render' . ucfirst($directive->name)], $directive);
+        return call_user_func([$this, 'render'.ucfirst($directive->name)], $directive);
     }
 }

Braces +2 added lines, -1 removed lines

@@ -45,7 +45,8 @@
      */
     public function render(Directive $directive): ?string
     {
-        if (!$this->hasDirective($directive->name)) {
+        if (!$this->hasDirective($directive->name))
+        {
             return null;
         }
 

src/Stempler/src/Directive/DirectiveGroup.php

Spacing +4 added lines, -4 removed lines

@@ -42,8 +42,8 @@ 
      */
     public function hasDirective(string $name): bool
     {
-        foreach ($this->directives as $directiveRenderer) {
-            if ($directiveRenderer->hasDirective($name)) {
+        foreach ($this->directives as $directiveRenderer){
+            if ($directiveRenderer->hasDirective($name)){
                 return true;
             }
         }
@@ -57,8 +57,8 @@ 
      */
     public function render(Directive $directive): ?string
     {
-        foreach ($this->directives as $directiveRenderer) {
-            if ($directiveRenderer->hasDirective($directive->name)) {
+        foreach ($this->directives as $directiveRenderer){
+            if ($directiveRenderer->hasDirective($directive->name)){
                 return $directiveRenderer->render($directive);
             }
         }

Braces +8 added lines, -4 removed lines

@@ -42,8 +42,10 @@ 
      */
     public function hasDirective(string $name): bool
     {
-        foreach ($this->directives as $directiveRenderer) {
-            if ($directiveRenderer->hasDirective($name)) {
+        foreach ($this->directives as $directiveRenderer)
+        {
+            if ($directiveRenderer->hasDirective($name))
+            {
                 return true;
             }
         }
@@ -57,8 +59,10 @@ 
      */
     public function render(Directive $directive): ?string
     {
-        foreach ($this->directives as $directiveRenderer) {
-            if ($directiveRenderer->hasDirective($directive->name)) {
+        foreach ($this->directives as $directiveRenderer)
+        {
+            if ($directiveRenderer->hasDirective($directive->name))
+            {
                 return $directiveRenderer->render($directive);
             }
         }

src/Stempler/tests/Transform/BaseTest.php

Spacing +4 added lines, -4 removed lines

@@ -34,7 +34,7 @@ 
 {
     protected function compile(string $source, array $visitors = [], LoaderInterface $loader = null)
     {
-        if ($loader === null) {
+        if ($loader === null){
             $loader = new StringLoader();
             $loader->set('root', $source);
         }
@@ -69,11 +69,11 @@ 
         $builder->getCompiler()->addRenderer(new DynamicRenderer(new DirectiveGroup()));
         $builder->getCompiler()->addRenderer(new HTMLRenderer());
 
-        foreach ($this->getVisitors() as $visitor) {
+        foreach ($this->getVisitors() as $visitor){
             $builder->addVisitor($visitor);
         }
 
-        foreach ($visitors as $visitor) {
+        foreach ($visitors as $visitor){
             $builder->addVisitor($visitor);
         }
 
@@ -90,6 +90,6 @@ 
      */
     protected function getFixtureLoader(): LoaderInterface
     {
-        return new DirectoryLoader(__DIR__ . '/../fixtures');
+        return new DirectoryLoader(__DIR__.'/../fixtures');
     }
 }

Braces +6 added lines, -3 removed lines

@@ -34,7 +34,8 @@ 
 {
     protected function compile(string $source, array $visitors = [], LoaderInterface $loader = null)
     {
-        if ($loader === null) {
+        if ($loader === null)
+        {
             $loader = new StringLoader();
             $loader->set('root', $source);
         }
@@ -69,11 +70,13 @@ 
         $builder->getCompiler()->addRenderer(new DynamicRenderer(new DirectiveGroup()));
         $builder->getCompiler()->addRenderer(new HTMLRenderer());
 
-        foreach ($this->getVisitors() as $visitor) {
+        foreach ($this->getVisitors() as $visitor)
+        {
             $builder->addVisitor($visitor);
         }
 
-        foreach ($visitors as $visitor) {
+        foreach ($visitors as $visitor)
+        {
             $builder->addVisitor($visitor);
         }
 

src/Stempler/tests/Transform/DynamicToPHPTest.php

Spacing +2 added lines, -2 removed lines

@@ -104,7 +104,7 @@ 
     public function testVerbatim4(): void
     {
         $this->assertSame(
-            '<script>alert(<?php echo json_encode' .
+            '<script>alert(<?php echo json_encode'.
             '("hello\' \'world", JSON_HEX_TAG | JSON_HEX_APOS | JSON_HEX_AMP | JSON_HEX_QUOT, 512); ?>)</script>',
             $res = $this->compile('<script>alert({{ "hello\' \'world" }})</script>')->getContent()
         );
@@ -127,7 +127,7 @@ 
     {
         ob_start();
 
-        eval('?>' . $body);
+        eval('?>'.$body);
 
         return ob_get_clean();
     }