patternseek / componentview

lib/AbstractViewComponent.php

Braces +5 added lines, -5 removed lines

@@ -171,7 +171,7 @@ 
         if ($execMethodName) { // Used to test for null but it could easily be an empty string
             $this->log( "Rendering with exec: {$execMethodName}, args:".var_export($execArgs, true ), LogLevel::DEBUG );
             $out = $this->execMethod( $execMethodName, $execArgs );
-        }else {
+        } else {
             $this->log( "Rendering without exec", LogLevel::DEBUG );
             $out = $this->template->render( $this->state, $this->props );
             if (!( $out instanceof ViewComponentResponse )) {
@@ -197,12 +197,12 @@ 
         if (count( $methodName ) == 1) {
             $methodName = $methodName[ 0 ] . 'Handler';
             $out = $this->$methodName( $args );
-        }else {
+        } else {
             $childName = array_shift( $methodName );
             $child = $this->childComponents[ $childName ];
             if ($child instanceof AbstractViewComponent) {
                 $out = $child->execMethod( $methodName, $args );
-            }else {
+            } else {
                 throw new \Exception( implode( ".", $methodName ) . " is not a valid method." );
             }
         }
@@ -283,7 +283,7 @@ 
         }
         if (null !== ( $pPath = $this->parent->getPath() )) {
             return $pPath . '.' . $this->handle;
-        }else {
+        } else {
             return $this->handle;
         }
     }
@@ -306,7 +306,7 @@ 
             }
             $child = new $type( $handle, $this, $this->exec, $this->logger );
             $this->childComponents[ $handle ] = $child;
-        }else {
+        } else {
             // exec, di and logger are set recursively in rehydrate()
             $child = $this->childComponents[ $handle ];
         }

Doc Comments +4 added lines, -4 removed lines

@@ -142,7 +142,7 @@ 
     }
 
     /**
-     * @return array
+     * @return string[]
      */
     public function __sleep()
     {
@@ -184,7 +184,7 @@ 
     /**
      * Execute a component method within the page or component.
      * Called first on a top level component which then passes the call down to the appropriate sub-component (or executes on itself if appropriate).
-     * @param array|string $methodName A methodname in the format subComponent.anotherSubComponent.methodName. Either dotted string as described, or parts in an array. The top level page component shouldn't be included
+     * @param string $methodName A methodname in the format subComponent.anotherSubComponent.methodName. Either dotted string as described, or parts in an array. The top level page component shouldn't be included
      * @param array $args
      * @throws \Exception
      * @return Response
@@ -214,7 +214,7 @@ 
     }
 
     /**
-     * @param $execMethod
+     * @param string $execMethod
      * @return string
      */
     public function getExecPath( $execMethod )
@@ -322,7 +322,7 @@ 
      * Render a child component.
      *
      * @param $handle
-     * @return Response
+     * @return string
      * @throws \Exception
      */
     public function renderChild( $handle )

lib/Template/TwigTemplate.php

Braces +1 added lines, -1 removed lines

@@ -100,7 +100,7 @@
 
         if (class_exists( "Puli\\TwigExtension\\PuliExtension" ) && null !== $this->repo) {
             $loader = new \Twig_Loader_Chain( [ new PuliTemplateLoader( $this->repo ), new Twig_Loader_String() ] );
-        }else {
+        } else {
             $loader = new Twig_Loader_String();
         }
         return $loader;

lib/Response.php

Braces +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
         if( null == $responseCode ){
             if( $type == "redirect" ){
                 $responseCode = 301;
-            }else{
+            } else{
                 $responseCode = 200;
             }
         }

lib/Template/PhpTemplate.php

Doc Comments +1 added lines, -1 removed lines

@@ -98,7 +98,7 @@
     }
 
     /**
-     * @param $templatePath
+     * @param string $templatePath
      * @return null|string
      * @throws \Exception
      */

Braces +2 added lines, -2 removed lines

@@ -86,11 +86,11 @@
             /** @noinspection PhpIncludeInspection */
             include( $realPath );
             $rendered = ob_get_clean();
-        }elseif ($this->templateString) {
+        } elseif ($this->templateString) {
             ob_start();
             eval( "?>" . $this->templateString );
             $rendered = ob_get_clean();
-        }else {
+        } else {
             throw new \Exception( "Neither valid template path no valid template string passed to PhpTemplate." );
         }