Description::setProperty() - Code Metrics - Inspection of "issue #55 test nodetype" - alexdebril/feed-io - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — issue/55 ( 2550af )

by Alex

created 2016-11-17 17:22 UTC

Description::setProperty() A

↳ Parent: Description

Complexity

Conditions	3
Paths	2

Size

Total Lines	15
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	3.0175

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	15
ccs	7
cts	8
cp	0.875
rs	9.4285
cc	3
eloc	9
nc	2
nop	2
crap	3.0175

<?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->nodeType == XML_CDATA_SECTION_NODE ) {
            $string = $element->firstChild->textContent;
        } else {
            foreach($element->childNodes as $childNode) {
                $string .= $element->ownerDocument->saveXML($childNode);
            }
        }

        $node->setDescription($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());

        return $document->createElement($this->getNodeName(), $description);
    }
}


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
16		const NODE_NAME = 'description';
17
18		/**
19		* @param NodeInterface $node
20		* @param \DOMElement $element
21		* @return $this
22		*/
23	7	public function setProperty(NodeInterface $node, \DOMElement $element)
24		{
25	7	$string = '';
26	7	if ( $element->firstChild->nodeType == XML_CDATA_SECTION_NODE ) {
27		$string = $element->firstChild->textContent;
28		} else {
29	7	foreach($element->childNodes as $childNode) {
30	7	$string .= $element->ownerDocument->saveXML($childNode);
31		}
32		}
33
34	7	$node->setDescription($string);
35
36	7	return $this;
37		}
38
39		/**
40		* creates the accurate DomElement content according to the $item's property
41		*
42		* @param \DOMDocument $document
43		* @param NodeInterface $node
44		* @return \DOMElement
45		*/
46	3	public function createElement(\DOMDocument $document, NodeInterface $node)
47		{
48	3	$description = htmlspecialchars($node->getDescription());
49
50	3	return $document->createElement($this->getNodeName(), $description);
51		}
52		}
53

alexdebril / feed-io

Push — issue/55 ( 2550af )

Description::setProperty() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like