Completed
Pull Request — master (#13)
by
unknown
04:03
created

RestrictionIdentifierFactory   A

Complexity

Total Complexity 4

Size/Duplication

Total Lines 34
Duplicated Lines 0 %

Coupling/Cohesion

Components 0
Dependencies 3

Test Coverage

Coverage 100%

Importance

Changes 0
Metric Value
wmc 4
lcom 0
cbo 3
dl 0
loc 34
ccs 14
cts 14
cp 1
rs 10
c 0
b 0
f 0

1 Method

Rating   Name   Duplication   Size   Complexity  
B getRestrictionIdentifier() 0 24 4
1
<?php
2
namespace Aoe\FeloginBruteforceProtection\Domain\Service;
3
4
/***************************************************************
5
 *  Copyright notice
6
 *
7
 *  (c) 2015 AOE GmbH <[email protected]>
8
 *
9
 *  All rights reserved
10
 *
11
 *  This script is part of the TYPO3 project. The TYPO3 project is
12
 *  free software; you can redistribute it and/or modify
13
 *  it under the terms of the GNU General Public License as published by
14
 *  the Free Software Foundation; either version 3 of the License, or
15
 *  (at your option) any later version.
16
 *
17
 *  The GNU General Public License can be found at
18
 *  http://www.gnu.org/copyleft/gpl.html.
19
 *
20
 *  This script is distributed in the hope that it will be useful,
21
 *  but WITHOUT ANY WARRANTY; without even the implied warranty of
22
 *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
23
 *  GNU General Public License for more details.
24
 *
25
 *  This copyright notice MUST APPEAR in all copies of the script!
26
 ***************************************************************/
27
28
use TYPO3\CMS\Frontend\Authentication\FrontendUserAuthentication;
29
use Aoe\FeloginBruteforceProtection\System\Configuration;
30
31
/**
32
 * Class RestrictionIdentifierFactory
33
 * @package Aoe\FeloginBruteforceProtection\Domain\Service
34
 */
35
class RestrictionIdentifierFactory
36
{
37
    /**
38
     * Restriction identifier factory method
39
     *
40
     * @param Configuration $configuration
41
     * @param FrontendUserAuthentication $frontendUserAuthentication
42
     * @return RestrictionIdentifierInterface
43
     **/
44 33
    public function getRestrictionIdentifier(
45
        Configuration $configuration,
46
        FrontendUserAuthentication $frontendUserAuthentication = null
47
    ) {
48
49 33
        $identificationIdentifier = $configuration->getIdentificationIdentifier();
50
        switch ($identificationIdentifier) {
51 33
            case RestrictionIdentifierFrontendName::IDENTIFIER:
52 12
                if ($frontendUserAuthentication !== null) {
53 11
                    new RestrictionIdentifierFrontendName();
54 11
                    $restrictionIdentifier = new RestrictionIdentifierFrontendName();
55 11
                    $restrictionIdentifier->setFrontendUserAuthentication($frontendUserAuthentication);
56 11
                    return $restrictionIdentifier;
57
                    break;
0 ignored issues
show
Unused Code introduced by
break is not strictly necessary here and could be removed.

The break statement is not necessary if it is preceded for example by a return statement:

switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}

If you would like to keep this construct to be consistent with other case statements, you can safely mark this issue as a false-positive.

Loading history...
58
                }
59
                // no break
60 22
            case RestrictionIdentifierClientIp::IDENTIFIER:
61 22
            default:
62 22
                $restrictionIdentifier = new RestrictionIdentifierClientIp();
63 22
                $restrictionIdentifier->setConfiguration($configuration);
64 22
                return $restrictionIdentifier;
65
                break;
0 ignored issues
show
Unused Code introduced by
break is not strictly necessary here and could be removed.

The break statement is not necessary if it is preceded for example by a return statement:

switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}

If you would like to keep this construct to be consistent with other case statements, you can safely mark this issue as a false-positive.

Loading history...
66 3
        }
67
    }
68
}
69