MockCommandExtractor::extractFromRequest() - Code Metrics - Inspection of "[#6] Attach path parameters" - eps90/req2cmd-bundle - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#12)

by Jakub

created 2017-06-23 16:14 UTC

MockCommandExtractor::extractFromRequest() A

↳ Parent: MockCommandExtractor

Complexity

Conditions	2
Paths	2

Size

Total Lines	9
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	9
c	0
b	0
f	0
rs	9.6666
cc	2
eloc	5
nc	2
nop	3

<?php
declare(strict_types=1);

namespace Eps\Req2CmdBundle\CommandExtractor;

use Symfony\Component\HttpFoundation\Request;

class MockCommandExtractor implements CommandExtractorInterface
{
    private $toReturn;
    private $toReturnForArgs;
    private $lastArgsHash;

    /**
     * MockCommandExtractor constructor.
     */
    public function __construct()
    {
        $this->toReturnForArgs = [];
    }

    /**
     * {@inheritdoc}
     */
    public function extractFromRequest(Request $request, string $commandClass, array $additionalProps = [])
    {
        $paramsHash = serialize(func_get_args());
        if (array_key_exists($paramsHash, $this->toReturnForArgs)) {
            return $this->toReturnForArgs[$paramsHash];
        }

        return $this->toReturn;
    }

    public function willReturn($command): self
    {
        if ($this->lastArgsHash !== null) {
            $this->toReturnForArgs[$this->lastArgsHash] = $command;
        } else {
            $this->toReturn = $command;
        }

        return $this;
    }

    public function andThen(): self
    {
        return $this;
    }

    public function forArguments(Request $request, string $cmdClass, array $additionalProps = []): self

    {
        $this->lastArgsHash = serialize(func_get_args());

        return $this;
    }
}


1			<?php
2			declare(strict_types=1);
3
4			namespace Eps\Req2CmdBundle\CommandExtractor;
5
6			use Symfony\Component\HttpFoundation\Request;
7
8			class MockCommandExtractor implements CommandExtractorInterface
9			{
10			private $toReturn;
11			private $toReturnForArgs;
12			private $lastArgsHash;
13
14			/**
15			* MockCommandExtractor constructor.
16			*/
17			public function __construct()
18			{
19			$this->toReturnForArgs = [];
20			}
21
22			/**
23			* {@inheritdoc}
24			*/
25			public function extractFromRequest(Request $request, string $commandClass, array $additionalProps = [])
26			{
27			$paramsHash = serialize(func_get_args());
28			if (array_key_exists($paramsHash, $this->toReturnForArgs)) {
29			return $this->toReturnForArgs[$paramsHash];
30			}
31
32			return $this->toReturn;
33			}
34
35			public function willReturn($command): self
36			{
37			if ($this->lastArgsHash !== null) {
38			$this->toReturnForArgs[$this->lastArgsHash] = $command;
39			} else {
40			$this->toReturn = $command;
41			}
42
43			return $this;
44			}
45
46			public function andThen(): self
47			{
48			return $this;
49			}
50
51			public function forArguments(Request $request, string $cmdClass, array $additionalProps = []): self
			0 ignored issues – show Unused Code introduced 2017-06-23 16:20 UTC by Report Bug Copy Issue Report The parameter `$request` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Unused Code introduced 2017-06-23 16:20 UTC by Report Bug Copy Issue Report The parameter `$cmdClass` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Unused Code introduced 2017-06-23 16:20 UTC by Report Bug Copy Issue Report The parameter `$additionalProps` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
52			{
53			$this->lastArgsHash = serialize(func_get_args());
54
55			return $this;
56			}
57			}
58

eps90 / req2cmd-bundle

Pull Request — master (#12)

MockCommandExtractor::extractFromRequest() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like