spiral / framework

src/Hmvc/src/AbstractCore.php

Spacing +6 added lines, -6 removed lines

@@ -45,9 +45,9 @@ 
      */
     public function callAction(string $controller, string $action, array $parameters = [])
     {
-        try {
+        try{
             $method = new \ReflectionMethod($controller, $action);
-        } catch (\ReflectionException $e) {
+        }catch (\ReflectionException $e){
             throw new ControllerException(
                 "Invalid action `{$controller}`->`{$action}`",
                 ControllerException::BAD_ACTION,
@@ -55,22 +55,22 @@ 
             );
         }
 
-        if ($method->isStatic() || !$method->isPublic()) {
+        if ($method->isStatic() || !$method->isPublic()){
             throw new ControllerException(
                 "Invalid action `{$controller}`->`{$action}`",
                 ControllerException::BAD_ACTION
             );
         }
 
-        try {
+        try{
             // getting the set of arguments should be sent to requested method
             $args = $this->resolver->resolveArguments($method, $parameters);
-        } catch (ArgumentException $e) {
+        }catch (ArgumentException $e){
             throw new ControllerException(
                 "Missing/invalid parameter '{$e->getParameter()->name}' of  `{$controller}`->`{$action}`",
                 ControllerException::BAD_ARGUMENT
             );
-        } catch (ContainerExceptionInterface $e) {
+        }catch (ContainerExceptionInterface $e){
             throw new ControllerException(
                 $e->getMessage(),
                 ControllerException::ERROR,

Braces +15 added lines, -6 removed lines

@@ -42,9 +42,12 @@ 
      */
     public function callAction(string $controller, string $action, array $parameters = [])
     {
-        try {
+        try
+        {
             $method = new \ReflectionMethod($controller, $action);
-        } catch (\ReflectionException $e) {
+        }
+        catch (\ReflectionException $e)
+        {
             throw new ControllerException(
                 "Invalid action `{$controller}`->`{$action}`",
                 ControllerException::BAD_ACTION,
@@ -52,22 +55,28 @@ 
             );
         }
 
-        if ($method->isStatic() || !$method->isPublic()) {
+        if ($method->isStatic() || !$method->isPublic())
+        {
             throw new ControllerException(
                 "Invalid action `{$controller}`->`{$action}`",
                 ControllerException::BAD_ACTION
             );
         }
 
-        try {
+        try
+        {
             // getting the set of arguments should be sent to requested method
             $args = $this->resolver->resolveArguments($method, $parameters);
-        } catch (ArgumentException $e) {
+        }
+        catch (ArgumentException $e)
+        {
             throw new ControllerException(
                 "Missing/invalid parameter '{$e->getParameter()->name}' of  `{$controller}`->`{$action}`",
                 ControllerException::BAD_ARGUMENT
             );
-        } catch (ContainerExceptionInterface $e) {
+        }
+        catch (ContainerExceptionInterface $e)
+        {
             throw new ControllerException(
                 $e->getMessage(),
                 ControllerException::ERROR,

src/Router/tests/SingleActionTest.php

Spacing +2 added lines, -2 removed lines

@@ -103,9 +103,9 @@
         $this->assertSame('echoed', (string)$response->getBody());
 
         $e = null;
-        try {
+        try{
             $router->handle(new ServerRequest([], [], new Uri('/test')));
-        } catch (UndefinedRouteException $e) {
+        }catch (UndefinedRouteException $e){
         }
 
         $this->assertNotNull($e, 'Autofill not fired');

Braces +5 added lines, -2 removed lines

@@ -103,9 +103,12 @@
         $this->assertSame('echoed', (string)$response->getBody());
 
         $e = null;
-        try {
+        try
+        {
             $router->handle(new ServerRequest([], [], new Uri('/test')));
-        } catch (UndefinedRouteException $e) {
+        }
+        catch (UndefinedRouteException $e)
+        {
         }
 
         $this->assertNotNull($e, 'Autofill not fired');

src/Router/tests/Diactoros/ResponseFactory.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
         $response = new Response('php://memory', $code, []);
         $response = $response->withStatus($code, $reasonPhrase);
 
-        foreach ($this->config->getBaseHeaders() as $header => $value) {
+        foreach ($this->config->getBaseHeaders() as $header => $value){
             $response = $response->withAddedHeader($header, $value);
         }
 

Braces +2 added lines, -1 removed lines

@@ -37,7 +37,8 @@
         $response = new Response('php://memory', $code, []);
         $response = $response->withStatus($code, $reasonPhrase);
 
-        foreach ($this->config->getBaseHeaders() as $header => $value) {
+        foreach ($this->config->getBaseHeaders() as $header => $value)
+        {
             $response = $response->withAddedHeader($header, $value);
         }
 

src/Router/tests/Diactoros/UploadedFileFactory.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
         string $clientFilename = null,
         string $clientMediaType = null
     ): UploadedFileInterface {
-        if ($size === null) {
+        if ($size === null){
             $size = $stream->getSize();
         }
 

Braces +2 added lines, -1 removed lines

@@ -26,7 +26,8 @@
         string $clientFilename = null,
         string $clientMediaType = null
     ): UploadedFileInterface {
-        if ($size === null) {
+        if ($size === null)
+        {
             $size = $stream->getSize();
         }
 

src/Router/tests/CallableTest.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
         $router = $this->makeRouter();
         $router->setRoute(
             'action',
-            new Route('/something', function () {
+            new Route('/something', function (){
                 return 'hello world';
             })
         );

Braces +2 added lines, -1 removed lines

@@ -27,7 +27,8 @@
         $router = $this->makeRouter();
         $router->setRoute(
             'action',
-            new Route('/something', function () {
+            new Route('/something', function ()
+            {
                 return 'hello world';
             })
         );

src/Router/tests/TestCore.php

Spacing +1 added lines, -1 removed lines

@@ -24,6 +24,6 @@
 
     public function callAction(string $controller, string $action = null, array $parameters = [])
     {
-        return '@wrapped.' . $this->core->callAction($controller, $action, $parameters);
+        return '@wrapped.'.$this->core->callAction($controller, $action, $parameters);
     }
 }

src/Router/src/Target/Action.php

Spacing +4 added lines, -4 removed lines

@@ -39,7 +39,7 @@ 
      */
     public function __construct(string $controller, $action, int $options = 0)
     {
-        if (!is_string($action) && !is_array($action)) {
+        if (!is_string($action) && !is_array($action)){
             throw new InvalidArgumentException(
                 sprintf(
                     'Action parameter must type string or array, `%s` given',
@@ -51,13 +51,13 @@ 
         $this->controller = $controller;
         $this->action = $action;
 
-        if (is_string($action)) {
+        if (is_string($action)){
             parent::__construct(
                 ['action' => $action],
                 ['action' => new Autofill($action)],
                 $options
             );
-        } else {
+        }else{
             parent::__construct(
                 ['action' => null],
                 ['action' => $action],
@@ -80,7 +80,7 @@ 
     protected function resolveAction(array $matches): string
     {
         $action = $this->action;
-        if (is_string($action)) {
+        if (is_string($action)){
             return $action;
         }
 

Braces +9 added lines, -4 removed lines

@@ -39,7 +39,8 @@ 
      */
     public function __construct(string $controller, $action, int $options = 0)
     {
-        if (!is_string($action) && !is_array($action)) {
+        if (!is_string($action) && !is_array($action))
+        {
             throw new InvalidArgumentException(
                 sprintf(
                     'Action parameter must type string or array, `%s` given',
@@ -51,13 +52,16 @@ 
         $this->controller = $controller;
         $this->action = $action;
 
-        if (is_string($action)) {
+        if (is_string($action))
+        {
             parent::__construct(
                 ['action' => $action],
                 ['action' => new Autofill($action)],
                 $options
             );
-        } else {
+        }
+        else
+        {
             parent::__construct(
                 ['action' => null],
                 ['action' => $action],
@@ -80,7 +84,8 @@ 
     protected function resolveAction(array $matches): string
     {
         $action = $this->action;
-        if (is_string($action)) {
+        if (is_string($action))
+        {
             return $action;
         }
 

src/Router/src/Target/Namespaced.php

Braces +2 added lines, -1 removed lines

@@ -58,7 +58,8 @@
      */
     protected function resolveController(array $matches): string
     {
-        if (preg_match('/[^a-z_0-9\-]/i', $matches['controller'])) {
+        if (preg_match('/[^a-z_0-9\-]/i', $matches['controller']))
+        {
             throw new TargetException('Invalid namespace target, controller name not allowed.');
         }
 

Spacing +2 added lines, -2 removed lines

@@ -33,7 +33,7 @@ 
         string $namespace,
         string $postfix = 'Controller',
         int $options = 0
-    ) {
+    ){
         $this->namespace = rtrim($namespace, '\\');
         $this->postfix = ucfirst($postfix);
 
@@ -51,7 +51,7 @@ 
      */
     protected function resolveController(array $matches): string
     {
-        if (preg_match('/[^a-z_0-9\-]/i', $matches['controller'])) {
+        if (preg_match('/[^a-z_0-9\-]/i', $matches['controller'])){
             throw new TargetException('Invalid namespace target, controller name not allowed.');
         }
 

src/Router/src/Router.php

Braces +32 added lines, -14 removed lines

@@ -69,13 +69,17 @@ 
      */
     public function handle(ServerRequestInterface $request): ResponseInterface
     {
-        try {
+        try
+        {
             $route = $this->matchRoute($request, $routeName);
-        } catch (RouteException $e) {
+        }
+        catch (RouteException $e)
+        {
             throw new RouterException('Invalid route definition', $e->getCode(), $e);
         }
 
-        if ($route === null) {
+        if ($route === null)
+        {
             throw new RouteNotFoundException($request->getUri());
         }
 
@@ -120,7 +124,8 @@ 
      */
     public function getRoute(string $name): RouteInterface
     {
-        if (isset($this->routes[$name])) {
+        if (isset($this->routes[$name]))
+        {
             return $this->routes[$name];
         }
 
@@ -132,7 +137,8 @@ 
      */
     public function getRoutes(): array
     {
-        if (!empty($this->default)) {
+        if (!empty($this->default))
+        {
             return $this->routes + [null => $this->default];
         }
 
@@ -144,9 +150,12 @@ 
      */
     public function uri(string $route, $parameters = []): UriInterface
     {
-        try {
+        try
+        {
             return $this->getRoute($route)->uri($parameters);
-        } catch (UndefinedRouteException $e) {
+        }
+        catch (UndefinedRouteException $e)
+        {
             //In some cases route name can be provided as controller:action pair, we can try to
             //generate such route automatically based on our default/fallback route
             return $this->castRoute($route)->uri($parameters);
@@ -161,17 +170,20 @@ 
      */
     protected function matchRoute(ServerRequestInterface $request, string &$routeName = null): ?RouteInterface
     {
-        foreach ($this->routes as $name => $route) {
+        foreach ($this->routes as $name => $route)
+        {
             // Matched route will return new route instance with matched parameters
             $matched = $route->match($request);
 
-            if ($matched !== null) {
+            if ($matched !== null)
+            {
                 $routeName = $name;
                 return $matched;
             }
         }
 
-        if ($this->default !== null) {
+        if ($this->default !== null)
+        {
             return $this->default->match($request);
         }
 
@@ -187,7 +199,8 @@ 
      */
     protected function configure(RouteInterface $route): RouteInterface
     {
-        if ($route instanceof ContainerizedInterface && !$route->hasContainer()) {
+        if ($route instanceof ContainerizedInterface && !$route->hasContainer())
+        {
             // isolating route in a given container
             $route = $route->withContainer($this->container);
         }
@@ -220,11 +233,16 @@ 
             );
         }
 
-        if (!empty($matches['name'])) {
+        if (!empty($matches['name']))
+        {
             $routeObject = $this->getRoute($matches['name']);
-        } elseif ($this->default !== null) {
+        }
+        elseif ($this->default !== null)
+        {
             $routeObject = $this->default;
-        } else {
+        }
+        else
+        {
             throw new UndefinedRouteException("Unable to locate route candidate for `{$route}`");
         }
 

Spacing +16 added lines, -16 removed lines

@@ -51,7 +51,7 @@ 
 
     public function __construct(string $basePath, UriHandler $uriHandler, ContainerInterface $container)
     {
-        $this->basePath = '/' . ltrim($basePath, '/');
+        $this->basePath = '/'.ltrim($basePath, '/');
         $this->uriHandler = $uriHandler;
         $this->container = $container;
     }
@@ -64,13 +64,13 @@ 
      */
     public function handle(ServerRequestInterface $request): ResponseInterface
     {
-        try {
+        try{
             $route = $this->matchRoute($request, $routeName);
-        } catch (RouteException $e) {
+        }catch (RouteException $e){
             throw new RouterException('Invalid route definition', $e->getCode(), $e);
         }
 
-        if ($route === null) {
+        if ($route === null){
             throw new RouteNotFoundException($request->getUri());
         }
 
@@ -115,7 +115,7 @@ 
      */
     public function getRoute(string $name): RouteInterface
     {
-        if (isset($this->routes[$name])) {
+        if (isset($this->routes[$name])){
             return $this->routes[$name];
         }
 
@@ -127,7 +127,7 @@ 
      */
     public function getRoutes(): array
     {
-        if (!empty($this->default)) {
+        if (!empty($this->default)){
             return $this->routes + [null => $this->default];
         }
 
@@ -139,9 +139,9 @@ 
      */
     public function uri(string $route, $parameters = []): UriInterface
     {
-        try {
+        try{
             return $this->getRoute($route)->uri($parameters);
-        } catch (UndefinedRouteException $e) {
+        }catch (UndefinedRouteException $e){
             //In some cases route name can be provided as controller:action pair, we can try to
             //generate such route automatically based on our default/fallback route
             return $this->castRoute($route)->uri($parameters);
@@ -153,17 +153,17 @@ 
      */
     protected function matchRoute(ServerRequestInterface $request, string &$routeName = null): ?RouteInterface
     {
-        foreach ($this->routes as $name => $route) {
+        foreach ($this->routes as $name => $route){
             // Matched route will return new route instance with matched parameters
             $matched = $route->match($request);
 
-            if ($matched !== null) {
+            if ($matched !== null){
                 $routeName = $name;
                 return $matched;
             }
         }
 
-        if ($this->default !== null) {
+        if ($this->default !== null){
             return $this->default->match($request);
         }
 
@@ -176,7 +176,7 @@ 
      */
     protected function configure(RouteInterface $route): RouteInterface
     {
-        if ($route instanceof ContainerizedInterface && !$route->hasContainer()) {
+        if ($route instanceof ContainerizedInterface && !$route->hasContainer()){
             // isolating route in a given container
             $route = $route->withContainer($this->container);
         }
@@ -201,17 +201,17 @@ 
                 $route,
                 $matches
             )
-        ) {
+        ){
             throw new UndefinedRouteException(
                 "Unable to locate route or use default route with 'name/controller:action' pattern"
             );
         }
 
-        if (!empty($matches['name'])) {
+        if (!empty($matches['name'])){
             $routeObject = $this->getRoute($matches['name']);
-        } elseif ($this->default !== null) {
+        } elseif ($this->default !== null){
             $routeObject = $this->default;
-        } else {
+        }else{
             throw new UndefinedRouteException("Unable to locate route candidate for `{$route}`");
         }