Description::setProperty() - Code Metrics - Inspection of "Close #161" - alexdebril/feed-io - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — issue/161 ( d32a7d )

by Alex

created 2018-01-12 22:16 UTC

Description::setProperty() A

↳ Parent: Description

Complexity

Conditions	4
Paths	2

Size

Total Lines	15
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	4.128

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	15
ccs	8
cts	10
cp	0.8
rs	9.2
cc	4
eloc	9
nc	2
nop	2
crap	4.128

<?php
/**
 * Created by PhpStorm.
 * User: alex
 * Date: 26/10/14
 * Time: 00:26
 */
namespace FeedIo\Rule;

use FeedIo\Feed\NodeInterface;
use FeedIo\RuleAbstract;

class Description extends RuleAbstract
{
    const NODE_NAME = 'description';

    /**
     * @param  NodeInterface $node
     * @param  \DOMElement   $element
     * @return $this
     */
    public function setProperty(NodeInterface $node, \DOMElement $element)
    {
        $string = '';
        if ($element->firstChild && $element->firstChild->nodeType == XML_CDATA_SECTION_NODE) {
            $string = $element->firstChild->textContent;
        } else {
            foreach ($element->childNodes as $childNode) {
                $string .= $element->ownerDocument->saveXML($childNode);
            }
        }

        $node->setDescription(htmlspecialchars_decode($string));

        return $this;
    }

    /**
     * creates the accurate DomElement content according to the $item's property
     *
     * @param  \DOMDocument  $document
     * @param  NodeInterface $node
     * @return \DOMElement
     */
    public function createElement(\DOMDocument $document, NodeInterface $node)
    {
        $description = htmlspecialchars($node->getDescription());
        $element = $document->createElement($this->getNodeName(), $description);

        if ($description !== $node->getDescription() && $this->getNodeName() != 'description') {
            $element->setAttribute('type', 'html');
        }

        return $element;
    }
}


1		<?php
2		/**
3		* Created by PhpStorm.
4		* User: alex
5		* Date: 26/10/14
6		* Time: 00:26
7		*/
8		namespace FeedIo\Rule;
9
10		use FeedIo\Feed\NodeInterface;
11		use FeedIo\RuleAbstract;
12
13		class Description extends RuleAbstract
14		{
15		const NODE_NAME = 'description';
16
17		/**
18		* @param NodeInterface $node
19		* @param \DOMElement $element
20		* @return $this
21		*/
22	9	public function setProperty(NodeInterface $node, \DOMElement $element)
23		{
24	9	$string = '';
25	9	if ($element->firstChild && $element->firstChild->nodeType == XML_CDATA_SECTION_NODE) {
26		$string = $element->firstChild->textContent;
27		} else {
28	9	foreach ($element->childNodes as $childNode) {
29	9	$string .= $element->ownerDocument->saveXML($childNode);
30	9	}
31		}
32
33	9	$node->setDescription(htmlspecialchars_decode($string));
34
35	9	return $this;
36		}
37
38		/**
39		* creates the accurate DomElement content according to the $item's property
40		*
41		* @param \DOMDocument $document
42		* @param NodeInterface $node
43		* @return \DOMElement
44		*/
45	3	public function createElement(\DOMDocument $document, NodeInterface $node)
46		{
47	3	$description = htmlspecialchars($node->getDescription());
48	3	$element = $document->createElement($this->getNodeName(), $description);
49
50	3	if ($description !== $node->getDescription() && $this->getNodeName() != 'description') {
51		$element->setAttribute('type', 'html');
52		}
53
54	3	return $element;
55		}
56		}
57

alexdebril / feed-io

Push — issue/161 ( d32a7d )

Description::setProperty() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like