jasny / controller

src/Middleware/Slim.php

Braces +2 added lines, -1 removed lines

@@ -27,7 +27,8 @@
      * @param bool $useSlimErrors  Throw Slim exceptions for error responses.
      */
     public function __construct(public bool $useSlimErrors = false)
-    { }
+    {
+}
 
     public function process(ServerRequestInterface $request, RequestHandlerInterface $handler): ResponseInterface
     {

Spacing +1 added lines, -1 removed lines

@@ -97,7 +97,7 @@
         }
     }
 
-    protected function getBody(ResponseInterface $response): string|null
+    protected function getBody(ResponseInterface $response): string | null
     {
         $body = (string)$response->getBody();
         return $body !== '' ? $body : null;

src/Guardian.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
      * Run a set of guards.
      */
     public function guard(
-        \ReflectionObject|\ReflectionMethod $subject,
+        \ReflectionObject | \ReflectionMethod $subject,
         ServerRequestInterface $request,
         ResponseInterface $response,
     ): ?ResponseInterface {

src/Traits/Guarded.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
         return $this->guardian;
     }
 
-    protected function guard(\ReflectionObject|\ReflectionMethod $subject): ?ResponseInterface
+    protected function guard(\ReflectionObject | \ReflectionMethod $subject): ?ResponseInterface
     {
         return $this->getGuardian()->guard($subject, $this->getRequest(), $this->getResponse());
     }

src/Traits/Header.php

Spacing +3 added lines, -3 removed lines

@@ -24,7 +24,7 @@ 
      *
      * @return $this
      */
-    protected function header(string $header, string|int|\Stringable $value, bool $add = false): static
+    protected function header(string $header, string | int | \Stringable $value, bool $add = false): static
     {
         $response = $add
             ? $this->getResponse()->withAddedHeader($header, (string)$value)
@@ -47,7 +47,7 @@ 
      * @param int|string $status
      * @return $this
      */
-    protected function status(int|string $status): static
+    protected function status(int | string $status): static
     {
         if (is_string($status)) {
             [$status, $phrase] = explode(' ', $status, 2) + [1 => ''];
@@ -134,7 +134,7 @@ 
      * @param string     $url
      * @param int|string $status  301 (Moved Permanently), 302 (Found), 303 (See Other) or 307 (Temporary Redirect)
      */
-    protected function redirect(string $url, int|string $status = 303): static
+    protected function redirect(string $url, int | string $status = 303): static
     {
         if ($status < 300 || $status >= 400) {
             throw new \DomainException("Invalid status code $status for redirect");

tests/ControllerTest.php

Braces +31 added lines, -13 removed lines

@@ -35,9 +35,12 @@ 
         $this->initialResponse = $this->createMock(ResponseInterface::class);
         $this->finalResponse = $this->createMock(ResponseInterface::class);
 
-        $this->controller = new class ($this) extends Controller {
+        $this->controller = new class ($this) extends Controller
+        {
             public $called;
-            public function __construct(protected ControllerTest $test) { }
+            public function __construct(protected ControllerTest $test)
+            {
+}
 
             public function process(int $foo, string $bar = 'red'): static {
                 $this->called = __FUNCTION__;
@@ -124,14 +127,19 @@ 
 
     public function testBefore()
     {
-        $controller = new class ($this) extends Controller {
-            function __construct(protected ControllerTest $test) { }
+        $controller = new class ($this) extends Controller
+        {
+            function __construct(protected ControllerTest $test)
+            {
+}
 
-            function before() {
+            function before()
+            {
                 return $this->paymentRequired();
             }
 
-            function process() {
+            function process()
+            {
                 $this->test->fail("Process should not be called");
             }
         };
@@ -149,9 +157,12 @@ 
 
     public function testClassGuard()
     {
-        $controller = new #[PaymentRequired] class ($this) extends Controller {
+        $controller = new #[PaymentRequired] class ($this) extends Controller
+        {
             public $called;
-            public function __construct(protected ControllerTest $test) { }
+            public function __construct(protected ControllerTest $test)
+            {
+}
 
             public function process(): static {
                 $this->called = __FUNCTION__;
@@ -174,12 +185,16 @@ 
 
     public function testMethodGuard()
     {
-        $controller = new class ($this) extends Controller {
+        $controller = new class ($this) extends Controller
+        {
             public $called;
-            function __construct(protected ControllerTest $test) { }
+            function __construct(protected ControllerTest $test)
+            {
+}
 
             #[PaymentRequired]
-            function process() {
+            function process()
+            {
                 $this->called = __FUNCTION__;
                 return $this->ok();
             }
@@ -200,9 +215,12 @@ 
 
     public function testNopGuard()
     {
-        $controller = new #[NopGuard] class ($this) extends Controller {
+        $controller = new #[NopGuard] class ($this) extends Controller
+        {
             public $called;
-            public function __construct(protected ControllerTest $test) { }
+            public function __construct(protected ControllerTest $test)
+            {
+}
 
             public function process(): static {
                 $this->called = __FUNCTION__;

tests/InContextOf.php

Braces +2 added lines, -1 removed lines

@@ -4,7 +4,8 @@
 
 trait InContextOf
 {
-    public function inContextOf(object $object, \Closure $function) {
+    public function inContextOf(object $object, \Closure $function)
+    {
         return $function->bindTo($this, $object)();
     }
 }

tests/Traits/CheckResponseTest.php

Braces +7 added lines, -3 removed lines

@@ -61,14 +61,18 @@
         $response = $this->createMock(ResponseInterface::class);
         $response->method('getStatusCode')->willReturn($code);
 
-        $controller = new class ($this, $response) extends Controller {
-            public function __construct(public TestCase $test, protected ResponseInterface $response) {}
+        $controller = new class ($this, $response) extends Controller
+        {
+            public function __construct(public TestCase $test, protected ResponseInterface $response)
+            {
+}
 
             protected function getResponse(): ResponseInterface {
                 return $this->response;
             }
 
-            public function assertResponseStatus(string $type) {
+            public function assertResponseStatus(string $type)
+            {
                 $this->test->assertSame($type === 'informational', $this->isInformational(), 'isInformational');
                 $this->test->assertSame($type === 'successful', $this->isSuccessful(), 'isSuccessful');
                 $this->test->assertSame($type === 'redirect', $this->isRedirection(), 'isRedirection');

tests/Traits/OutputTest.php

Spacing +2 added lines, -2 removed lines

@@ -43,7 +43,7 @@ 
         $response = $this->createMock(ResponseInterface::class);
         $response->method('getBody')->willReturn($stream);
         $response->expects($this->once())->method('withHeader')
-            ->with( 'Content-Type', $contentType)
+            ->with('Content-Type', $contentType)
             ->willReturnSelf();
 
         $controller = $this->createPartialMock(Controller::class, ['getResponse']);
@@ -99,7 +99,7 @@ 
         $response = $this->createMock(ResponseInterface::class);
         $response->method('getBody')->willReturn($stream);
         $response->expects($this->once())->method('withHeader')
-            ->with( 'Content-Type', 'application/json')
+            ->with('Content-Type', 'application/json')
             ->willReturnSelf();
 
         $controller = $this->createPartialMock(Controller::class, ['getResponse']);

tests/Traits/HeaderTest.php

Spacing +2 added lines, -2 removed lines

@@ -27,7 +27,7 @@ 
     /**
      * @dataProvider statusProvider
      */
-    public function testStatus(int|string $status, int $code, string $phrase)
+    public function testStatus(int | string $status, int $code, string $phrase)
     {
         $response = $this->createMock(ResponseInterface::class);
         $response->expects($this->once())->method('withStatus')->with($code, $phrase)->willReturnSelf();
@@ -205,7 +205,7 @@ 
      * Test 'redirect' function
      * @dataProvider redirectStatusProvider
      */
-    public function testRedirect(int|string $status)
+    public function testRedirect(int | string $status)
     {
         $controller = $this->createPartialMock(Controller::class, ['status', 'header', 'output']);