Completed
Push — master ( 40e412...07993c )
by Tobias
02:30
created

GroupValidationTrait   A

Complexity

Total Complexity 3

Size/Duplication

Total Lines 19
Duplicated Lines 0 %

Coupling/Cohesion

Components 0
Dependencies 0

Importance

Changes 1
Bugs 0 Features 0
Metric Value
wmc 3
c 1
b 0
f 0
lcom 0
cbo 0
dl 0
loc 19
rs 10

1 Method

Rating   Name   Duplication   Size   Complexity  
A includeBasedOnGroup() 0 10 3
1
<?php
2
3
namespace Happyr\SerializerBundle\Normalizer;
4
5
/**
6
 *
7
 *
8
 * @author Tobias Nyholm <[email protected]>
9
 */
10
trait GroupValidationTrait
11
{
12
    /**
13
     * @param array $context
14
     * @param array $groups
15
     *
16
     * @return bool
17
     */
18
    protected function includeBasedOnGroup(array $context, array $groups)
19
    {
20
        foreach ($context['groups'] as $group) {
21
            if (in_array($group, $groups)) {
22
                return $included = true;
0 ignored issues
show
Unused Code introduced by
$included is not used, you could remove the assignment.

This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently.

$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

Both the $myVar assignment in line 1 and the $higher assignment in line 2 are dead. The first because $myVar is never used and the second because $higher is always overwritten for every possible time line.

Loading history...
23
            }
24
        }
25
26
        return false;
27
    }
28
}