CompositeTargetOrigin::__construct() - Code Metrics - Inspection of "Merge pull request #1 from moufmouf/improvements" - thecodingmachine/csrf-header-check-middleware - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( c44d00...13b3c2 )

by David

created 2017-06-12 15:01 UTC

CompositeTargetOrigin::__construct() A

↳ Parent: CompositeTargetOrigin

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php


namespace TheCodingMachine\Middlewares\OriginFetchers;

use Psr\Http\Message\ServerRequestInterface;

class CompositeTargetOrigin implements TargetOriginInterface
{
    /**
     * @var TargetOriginInterface[]
     */
    private $targetOrigins;

    /**
     * @param TargetOriginInterface[] ...$targetOrigins
     */
    public function __construct(TargetOriginInterface ...$targetOrigins)
    {
        $this->targetOrigins = $targetOrigins;

    }

    /**
     * Returns an array of allowed domain names.
     * If the "source" origin matches one of these origins, the request is valid.
     *
     * @return string[]
     */
    public function __invoke(ServerRequestInterface $request): array
    {
        $targetOrigins = [];

        foreach ($this->targetOrigins as $targetOrigin) {
            $targetOrigins = array_merge($targetOrigins, $targetOrigin($request));
        }

        return $targetOrigins;
    }
}


1			<?php
2
3
4			namespace TheCodingMachine\Middlewares\OriginFetchers;
5
6			use Psr\Http\Message\ServerRequestInterface;
7
8			class CompositeTargetOrigin implements TargetOriginInterface
9			{
10			/**
11			* @var TargetOriginInterface[]
12			*/
13			private $targetOrigins;
14
15			/**
16			* @param TargetOriginInterface[] ...$targetOrigins
17			*/
18			public function __construct(TargetOriginInterface ...$targetOrigins)
19			{
20			$this->targetOrigins = $targetOrigins;
			0 ignored issues – show Documentation Bug introduced 2017-06-12 09:58 UTC by Report Bug Copy Issue Report It seems like `$targetOrigins` of type `array<integer,array<inte...argetOriginInterface>>>` is incompatible with the declared type `array<integer,object<The...TargetOriginInterface>>` of property `$targetOrigins`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
21			}
22
23			/**
24			* Returns an array of allowed domain names.
25			* If the "source" origin matches one of these origins, the request is valid.
26			*
27			* @return string[]
28			*/
29			public function __invoke(ServerRequestInterface $request): array
30			{
31			$targetOrigins = [];
32
33			foreach ($this->targetOrigins as $targetOrigin) {
34			$targetOrigins = array_merge($targetOrigins, $targetOrigin($request));
35			}
36
37			return $targetOrigins;
38			}
39			}
40

thecodingmachine / csrf-header-check-middleware

Push — master ( c44d00...13b3c2 )

CompositeTargetOrigin::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like