Detector   A
last analyzed

Complexity

Total Complexity 11

Size/Duplication

Total Lines 78
Duplicated Lines 0 %

Coupling/Cohesion

Components 1
Dependencies 2

Test Coverage

Coverage 84.85%

Importance

Changes 0
Metric Value
wmc 11
lcom 1
cbo 2
dl 0
loc 78
ccs 28
cts 33
cp 0.8485
rs 10
c 0
b 0
f 0

4 Methods

Rating   Name   Duplication   Size   Complexity  
A testAll() 0 16 3
A __construct() 0 4 1
A findEmail() 0 10 3
A testRulesAgainst() 0 21 4
1
<?php
2
/*
3
 * This file is part of Bounce-Detection.
4
 *
5
 * (c)2016 cwd.at GmbH <[email protected]>
6
 *
7
 * For the full copyright and license information, please view the LICENSE
8
 * file that was distributed with this source code.
9
 */
10
11
namespace Cwd\BounceDetection;
12
use PhpMimeMailParser\Parser;
13
14
/**
15
 * Class Detector
16
 *
17
 * @package Cwd\BounceDetection
18
 * @author  Ludwig Ruderstaller <[email protected]>
19
 */
20
class Detector
21
{
22
    /**
23
     * @var RuleConfig|null
24
     */
25
    protected $ruleConfig = null;
26
27
    public function __construct($ruleFile = null)
28
    {
29
        $this->ruleConfig = RuleFactory::getRules($ruleFile);
30
    }
31
32
    /**
33
     * @param string $mail
34
     *
35
     * @return array
36
     */
37 247
    public function testAll($mail)
38
    {
39 247
        $result = $this->testRulesAgainst($this->ruleConfig->getDSNRules(), $mail);
40
41 247
        if ($result === null) {
42 18
            $result = $this->testRulesAgainst($this->ruleConfig->getBodyRules(), $mail);
43 18
        }
44
45 247
        if ($result === null) {
0 ignored issues
show
Unused Code introduced by
This if statement is empty and can be removed.

This check looks for the bodies of if statements that have no statements or where all statements have been commented out. This may be the result of changes for debugging or the code may simply be obsolete.

These if bodies can be removed. If you have an empty if but statements in the else branch, consider inverting the condition.

if (rand(1, 6) > 3) {
//print "Check failed";
} else {
    print "Check succeeded";
}

could be turned into

if (rand(1, 6) <= 3) {
    print "Check succeeded";
}

This is much more concise to read.

Loading history...
46
            // @todo
47 18
        }
48
49 247
        $result['email'] = $this->findEmail($mail);
50
51 247
        return $result;
52
    }
53
54
    /**
55
     * @param $mail
56
     *
57
     * @return string
58
     */
59 247
    public function findEmail($mail)
60
    {
61 247
        foreach ($this->ruleConfig->getEmailRules() as $rule) {
62 247
            if (preg_match($rule['regexp'], $mail, $match)) {
63 134
                return $match[1];
64
            }
65 216
        }
66
67 113
        return null;
68
    }
69
70
    /**
71
     * @param array  $rules
72
     * @param string $mail
73
     *
74
     * @return array
75
     */
76 247
    public function testRulesAgainst(array $rules, $mail)
77
    {
78 247
        $parser = new Parser();
79 247
        $parser->setText($mail);
80 247
        $data = $parser->getData();
81
82 247
        foreach ($rules as $rule) {
83 247
            if (preg_match($rule['regexp'], $data, $match)) {
84
                $result = [
85 229
                    'rule_cat' => $rule['category'],
86 229
                    'rule_no'  => $rule['number'],
87 229
                ];
88
89 229
                if (isset($rule['idx_email'])) {
90
                    $result['email'] = $match[$rule['idx_email']];
91
                }
92
93 229
                return $result;
94
            }
95 242
        }
96 18
    }
97
}
98