Cors::methods() - Code Metrics - Inspection of "Add cors handler (#2)" - controlabs/slim-handlers - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 25c522...337ea3 )

by Breno

created 2018-10-30 12:28 UTC

Cors::methods() A

↳ Parent: Cors

Complexity

Conditions	4
Paths	2

Size

Total Lines	18
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	4
eloc	10
nc	2
nop	1
dl	0
loc	18
rs	9.9332
c	0
b	0
f	0

<?php

namespace Controlabs\Handler\Slim;

use Psr\Container\ContainerInterface;
use Slim\Http\Request;
use Slim\Http\Response;

class Cors
{
    const HEADERS = [
        'Content-Type',
        'Accept',
        'Origin',
        'Authorization',
        'X-Requested-With'
    ];

    protected $container;

    public function __construct(ContainerInterface $container)
    {
        $this->container = $container;
    }

    public function __invoke(Request $request, Response $response, $next)
    {
        if ($next && is_callable($next)) {
            $response = $next($request, $response);
        }

        $methods = $this->methods($request);

        return $response
            ->withHeader('Access-Control-Allow-Origin', '*')
            ->withHeader('Access-Control-Allow-Headers', implode(',', self::HEADERS))
            ->withHeader('Access-Control-Allow-Methods', implode(',', $methods));
    }

    private function methods(Request $request): array
    {
        $methods = [];

        if ($route = $request->getAttribute('route')) {
            $pattern = $route->getPattern();

            $router = $this->container->get('router');
            foreach ($router->getRoutes() as $route) {
                if ($pattern === $route->getPattern()) {
                    $methods = array_merge_recursive($methods, $route->getMethods());
                }
            }
        } else {
            // Methods holds all of the HTTP Verbs that a particular route handles.
            $methods[] = $request->getMethod();
        }
        return $methods;
    }
}


1			<?php
2
3			namespace Controlabs\Handler\Slim;
4
5			use Psr\Container\ContainerInterface;
6			use Slim\Http\Request;
7			use Slim\Http\Response;
8
9			class Cors
10			{
11			const HEADERS = [
12			'Content-Type',
13			'Accept',
14			'Origin',
15			'Authorization',
16			'X-Requested-With'
17			];
18
19			protected $container;
20
21			public function __construct(ContainerInterface $container)
22			{
23			$this->container = $container;
24			}
25
26			public function __invoke(Request $request, Response $response, $next)
27			{
28			if ($next && is_callable($next)) {
29			$response = $next($request, $response);
30			}
31
32			$methods = $this->methods($request);
33
34			return $response
35			->withHeader('Access-Control-Allow-Origin', '*')
36			->withHeader('Access-Control-Allow-Headers', implode(',', self::HEADERS))
37			->withHeader('Access-Control-Allow-Methods', implode(',', $methods));
38			}
39
40			private function methods(Request $request): array
41			{
42			$methods = [];
43
44			if ($route = $request->getAttribute('route')) {
45			$pattern = $route->getPattern();
46
47			$router = $this->container->get('router');
48			foreach ($router->getRoutes() as $route) {
49			if ($pattern === $route->getPattern()) {
50			$methods = array_merge_recursive($methods, $route->getMethods());
51			}
52			}
53			} else {
54			// Methods holds all of the HTTP Verbs that a particular route handles.
55			$methods[] = $request->getMethod();
56			}
57			return $methods;
58			}
59			}
60

controlabs / slim-handlers

Push — master ( 25c522...337ea3 )

Cors::methods() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like