Completed
Push — master ( 45904d...9fbe45 )
by Kevin
03:18
created

Tr::doClean()   C

Complexity

Conditions 13
Paths 3

Size

Total Lines 41
Code Lines 24

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 26
CRAP Score 13

Importance

Changes 1
Bugs 0 Features 1
Metric Value
c 1
b 0
f 1
dl 0
loc 41
ccs 26
cts 26
cp 1
rs 5.1234
cc 13
eloc 24
nc 3
nop 1
crap 13

How to fix   Complexity   

Long Method

Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.

For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.

Commonly applied refactorings include:

1
<?php
2
3
namespace Groundskeeper\Tokens\Elements;
4
5
use Groundskeeper\Configuration;
6
use Groundskeeper\Tokens\ElementTypes\OpenElement;
7
use Groundskeeper\Tokens\ElementTypes\ScriptSupporting;
8
use Groundskeeper\Tokens\Token;
9
use Psr\Log\LoggerInterface;
10
11
/**
12
 * "tr" element
13
 *
14
 * https://html.spec.whatwg.org/multipage/semantics.html#the-tr-element
15
 */
16
class Tr extends OpenElement
17
{
18 3
    protected function doClean(LoggerInterface $logger)
19
    {
20 3
        if ($this->configuration->get('clean-strategy') != Configuration::CLEAN_STRATEGY_LENIENT) {
21
            // "table" must be parent.
22 3
            $parent = $this->getParent();
23 3
            if ($parent !== null &&
24 3
                $parent->getName() != 'thead' &&
25 3
                $parent->getName() != 'tbody' &&
26 3
                $parent->getName() != 'tfoot' &&
27 3
                $parent->getName() != 'table') {
28 1
                $logger->debug('Element "tr" must be a child of the "thead", "tbody", "tfoot", or "table" elements.');
29
30 1
                return false;
31
            }
32
33
            // Children can be "td", "th", and script supporting elements.
34 3
            foreach ($this->children as $child) {
35 3
                if ($child->getType() == Token::COMMENT) {
36 1
                    continue;
37
                }
38
39 2
                if ($child->getType() !== Token::ELEMENT) {
40 1
                    $logger->debug('Removing ' . $child . '. Only elements allowed as children of "tr" element.');
41 1
                    $this->removeChild($child);
42
43 1
                    continue;
44
                }
45
46 2
                if ($child->getName() == 'td' ||
47 2
                    $child->getName() == 'th' ||
48 2
                    $child instanceof ScriptSupporting) {
49 2
                    continue;
50
                }
51
52 1
                $logger->debug('Removing ' . $child . '. Only "td", "th", and script supporting elements allowed as children of "tr" element.');
53 1
                $this->removeChild($child);
54 3
            }
55 3
        }
56
57 3
        return true;
58
    }
59
}
60