Thead::removeInvalidSelf() - Code Metrics - Inspection of "Refactored how elements are cleaned and removed." - kevintweber/Groundskeeper - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 9fbe45...817727 )

by Kevin

created 2016-05-13 03:01 UTC

Thead::removeInvalidSelf() A

↳ Parent: Thead

Complexity

Conditions	3
Paths	2

Size

Total Lines	12
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	3

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	12
ccs	6
cts	6
cp	1
rs	9.4285
cc	3
eloc	6
nc	2
nop	1
crap	3

<?php

namespace Groundskeeper\Tokens\Elements;

use Groundskeeper\Configuration;
use Groundskeeper\Tokens\ElementTypes\OpenElement;
use Groundskeeper\Tokens\ElementTypes\ScriptSupporting;
use Groundskeeper\Tokens\Token;
use Psr\Log\LoggerInterface;

/**
 * "thead" element
 *
 * https://html.spec.whatwg.org/multipage/semantics.html#the-thead-element
 */
class Thead extends OpenElement
{
    protected function removeInvalidChildren(LoggerInterface $logger)
    {
        // Children can be "tr" and script supporting elements.
        foreach ($this->children as $child) {
            if ($child->getType() == Token::COMMENT) {
                continue;
            }

            if ($child->getType() !== Token::ELEMENT) {
                $logger->debug('Removing ' . $child . '. Only elements allowed as children of "thead" element.');
                $this->removeChild($child);

                continue;
            }

            if ($child->getName() == 'tr' ||
                $child instanceof ScriptSupporting) {
                continue;
            }

            $logger->debug('Removing ' . $child . '. Only "tr" and script supporting elements allowed as children of "thead" element.');
            $this->removeChild($child);
        }
    }

    protected function removeInvalidSelf(LoggerInterface $logger)
    {
        // "table" must be parent.
        $parent = $this->getParent();
        if ($parent !== null && $parent->getName() != 'table') {
            $logger->debug($this . ' must be a child of the "table" element.');

            return true;
        }

        return false;
    }
}


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		* "thead" element
13		*
14		* https://html.spec.whatwg.org/multipage/semantics.html#the-thead-element
15		*/
16		class Thead extends OpenElement
17		{
18	3	protected function removeInvalidChildren(LoggerInterface $logger)
19		{
20		// Children can be "tr" and script supporting elements.
21	3	foreach ($this->children as $child) {
22	3	if ($child->getType() == Token::COMMENT) {
23	1	continue;
24		}
25
26	3	if ($child->getType() !== Token::ELEMENT) {
27	1	$logger->debug('Removing ' . $child . '. Only elements allowed as children of "thead" element.');
28	1	$this->removeChild($child);
29
30	1	continue;
31		}
32
33	2	if ($child->getName() == 'tr' \|\|
34	2	$child instanceof ScriptSupporting) {
35	1	continue;
36		}
37
38	1	$logger->debug('Removing ' . $child . '. Only "tr" and script supporting elements allowed as children of "thead" element.');
39	1	$this->removeChild($child);
40	3	}
41	3	}
42
43	4	protected function removeInvalidSelf(LoggerInterface $logger)
44		{
45		// "table" must be parent.
46	4	$parent = $this->getParent();
47	4	if ($parent !== null && $parent->getName() != 'table') {
48	1	$logger->debug($this . ' must be a child of the "table" element.');
49
50	1	return true;
51		}
52
53	3	return false;
54		}
55		}
56

kevintweber / Groundskeeper

Push — master ( 9fbe45...817727 )

Thead::removeInvalidSelf() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like