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 +6 added lines, -3 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
@@ -213,6 +213,9 @@ 
         return $out;
     }
 
+    /**
+     * @param string $execMethod
+     */
     public function getExecPath( $execMethod )
     {
         $path = $this->getPath();
@@ -318,7 +321,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;

Doc Comments +2 added lines, -2 removed lines

@@ -69,9 +69,9 @@
     }
 
     /**
-     * @param array|ViewState $state
+     * @param ViewState $state
      * @param array $props
-     * @return string
+     * @return Response
      */
     protected function doRender( ViewState $state, array $props = [] )
     {

lib/Response.php

Doc Comments +2 added lines, -2 removed lines

@@ -27,8 +27,8 @@
     public $content;
 
     /**
-     * @param $type
-     * @param $content
+     * @param string $type
+     * @param string $content
      * @param null $responseCode
      */
     function __construct( $type, $content, $responseCode = null )

Braces +1 added lines, -1 removed lines

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