Li::removeInvalidSelf() - Code Metrics - Inspection of "Simplified clean and fix functions for elements." - kevintweber/Groundskeeper - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 817727...d8e3ee )

by Kevin

created 2016-05-14 00:46 UTC

Li::removeInvalidSelf() C

↳ Parent: Li

Complexity

Conditions	7
Paths	4

Size

Total Lines	22
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	11
CRAP Score	7

Importance

Changes	2
Bugs	1	Features	0

Metric	Value
c	2
b	1
f	0
dl	0
loc	22
ccs	11
cts	11
cp	1
rs	6.9811
cc	7
eloc	10
nc	4
nop	1
crap	7

<?php

namespace Groundskeeper\Tokens\Elements;

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

/**
 * "li" element
 *
 * https://html.spec.whatwg.org/multipage/semantics.html#the-li-element
 */
class Li extends OpenElement
{
    protected function getAllowedAttributes()
    {
        // "value" attribute for "li" which are children of "ol" elements.
        if ($this->getParent() instanceof Ol) {
            $liAllowedAttributes = array(
                '/^value$/i' => Element::ATTR_INT
            );

            return array_merge(
                $liAllowedAttributes,
                parent::getAllowedAttributes()
            );
        }

        return parent::getAllowedAttributes();
    }

    protected function removeInvalidSelf(LoggerInterface $logger)
    {
        // Only allowed:
        // Inside ol elements.
        // Inside ul elements.
        // Inside menu elements whose type attribute is in the toolbar state.
        $parent = $this->getParent();
        if ($parent === null || $parent instanceof Ol || $parent instanceof Ul) {
            return false;
        }

        if ($parent instanceof Menu && $parent->hasAttribute('type')) {
            $typeAttributeValue = $parent->getAttribute('type');
            if ($typeAttributeValue == 'toolbar') {
                return false;
            }
        }

        $logger->debug('Removing ' . $this . '. Only allowed inside the "li" element are "ol", "ul" and "menu[type=toolbar]" elements.');

        return true;
    }
}


1		<?php
2
3		namespace Groundskeeper\Tokens\Elements;
4
5		use Groundskeeper\Configuration;
6		use Groundskeeper\Tokens\Element;
7		use Groundskeeper\Tokens\ElementTypes\OpenElement;
8		use Groundskeeper\Tokens\Token;
9		use Psr\Log\LoggerInterface;
10
11		/**
12		* "li" element
13		*
14		* https://html.spec.whatwg.org/multipage/semantics.html#the-li-element
15		*/
16		class Li extends OpenElement
17		{
18	2	protected function getAllowedAttributes()
19		{
20		// "value" attribute for "li" which are children of "ol" elements.
21	2	if ($this->getParent() instanceof Ol) {
22		$liAllowedAttributes = array(
23		'/^value$/i' => Element::ATTR_INT
24	1	);
25
26	1	return array_merge(
27	1	$liAllowedAttributes,
28	1	parent::getAllowedAttributes()
29	1	);
30		}
31
32	1	return parent::getAllowedAttributes();
33		}
34
35	4	protected function removeInvalidSelf(LoggerInterface $logger)
36		{
37		// Only allowed:
38		// Inside ol elements.
39		// Inside ul elements.
40		// Inside menu elements whose type attribute is in the toolbar state.
41	4	$parent = $this->getParent();
42	4	if ($parent === null \|\| $parent instanceof Ol \|\| $parent instanceof Ul) {
43	2	return false;
44		}
45
46	2	if ($parent instanceof Menu && $parent->hasAttribute('type')) {
47	1	$typeAttributeValue = $parent->getAttribute('type');
48	1	if ($typeAttributeValue == 'toolbar') {
49	1	return false;
50		}
51	1	}
52
53	2	$logger->debug('Removing ' . $this . '. Only allowed inside the "li" element are "ol", "ul" and "menu[type=toolbar]" elements.');
54
55	2	return true;
56		}
57		}
58

kevintweber / Groundskeeper

Push — master ( 817727...d8e3ee )

Li::removeInvalidSelf() C

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like