First   A
last analyzed

Complexity

Total Complexity 3

Size/Duplication

Total Lines 30
Duplicated Lines 100 %

Coupling/Cohesion

Components 1
Dependencies 2

Test Coverage

Coverage 100%

Importance

Changes 1
Bugs 0 Features 0
Metric Value
wmc 3
c 1
b 0
f 0
lcom 1
cbo 2
dl 30
loc 30
ccs 8
cts 8
cp 1
rs 10

2 Methods

Rating   Name   Duplication   Size   Complexity  
A __construct() 4 4 1
A flow() 9 9 2

How to fix   Duplicated Code   

Duplicated Code

Duplicate code is one of the most pungent code smells. A rule that is often used is to re-structure code once it is duplicated in three or more places.

Common duplication problems, and corresponding solutions are:

1
<?php
2
/**
3
 * This file is part of graze/data-flow
4
 *
5
 * Copyright (c) 2016 Nature Delivered Ltd. <https://www.graze.com>
6
 *
7
 * For the full copyright and license information, please view the LICENSE
8
 * file that was distributed with this source code.
9
 *
10
 * @license https://github.com/graze/data-flow/blob/master/LICENSE.md
11
 * @link    https://github.com/graze/data-flow
12
 */
13
14
namespace Graze\DataFlow\Flow\Collection;
15
16
use Graze\DataFlow\Flow\AbstractFlow;
17
use Graze\DataNode\NodeCollectionInterface;
18
use Graze\DataNode\NodeInterface;
19
use InvalidArgumentException;
20
use Psr\Log\LogLevel;
21
22
/**
23
 * Return the first element in a NodeCollection based on an option callback
24
 */
25 View Code Duplication
class First extends AbstractFlow
0 ignored issues
show
Duplication introduced by
This class seems to be duplicated in your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
26
{
27
    /**
28
     * @var callable|null
29
     */
30
    private $callback;
31
32
    /**
33
     * First constructor.
34
     *
35
     * @param callable $callback
0 ignored issues
show
Documentation introduced by
Should the type for parameter $callback not be null|callable?

This check looks for @param annotations where the type inferred by our type inference engine differs from the declared type.

It makes a suggestion as to what type it considers more descriptive.

Most often this is a case of a parameter that can be null in addition to its declared types.

Loading history...
36
     */
37 7
    public function __construct(callable $callback = null)
38
    {
39 7
        $this->callback = $callback;
40 7
    }
41
42
    /**
43
     * @inheritDoc
44
     */
45 6
    public function flow(NodeInterface $node)
46
    {
47 6
        if (!($node instanceof NodeCollectionInterface)) {
48 1
            throw new InvalidArgumentException("The supplied $node is not a NodeCollectionInterface");
49
        }
50 5
        $this->log(LogLevel::DEBUG, "Selecting the first entry from {node}", ['node' => $node]);
51
52 5
        return $node->first($this->callback);
53
    }
54
}
55