Bhaktaraz\RSSGenerator\SimpleXMLElement

Complexity

Total Complexity

5

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes	3
Bugs	1	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	addChild()	0	8	3
A	addChildCData()	0	5	1
A	addCData()	0	6	1

<?php

namespace Bhaktaraz\RSSGenerator;

use SimpleXMLElement as XMLElement;

class SimpleXMLElement extends XMLElement
{

    /**
     * @param string $name
     * @param string $value
     * @param string $namespace
     * @return XMLElement
     */
    public function addChild($name, $value = null, $namespace = null)
    {
        if ($value !== null and is_string($value) === true) {

Using logical operators such as and instead of && is generally not recommended.

            $value = str_replace('&', '&', $value);
        }

        return parent::addChild($name, $value, $namespace);
    }

    /**
     * Create a child with CDATA value
     * @param string $name The name of the child element to add.
     * @param string $cdata_text The CDATA value of the child element.
     */
    public function addChildCData($name, $cdata_text)
    {
        $child = $this->addChild($name);
        $child->addCData($cdata_text);
    }

    /**
     * Add CDATA text in a node
     * @param string $cdata_text The CDATA value  to add
     */
    private function addCData($cdata_text)

This method is not used, and could be removed.

    {
        $node = dom_import_simplexml($this);
        $no = $node->ownerDocument;
        $node->appendChild($no->createCDATASection($cdata_text));
    }
}