ElggXMLElement - Code Metrics - gctools-outilsgc/gcconnex - Measure and Improve Code Quality continuously with Scrutinizer

ElggXMLElement A
last analyzed 2021-05-27 17:40 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	124
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

Importance

Changes

Metric	Value
dl	0
loc	124
ccs	0
cts	76
cp	0
rs	10
c	0
b	0
f	0
wmc	21
lcom	1
cbo	0

7 Methods

Rating	Name	Size	Complexity
A	__construct()	12	2
A	getName()	3	1
A	getAttributes()	21	5
A	getContent()	3	1
A	getChildren()	9	2
A	__get()	17	5
A	__isset()	17	5

<?php
/**
 * A parser for XML that uses SimpleXMLElement
 *
 * @package    Elgg.Core
 * @subpackage XML
 */
class ElggXMLElement {
	/**
	 * @var SimpleXMLElement
	 */
	private $_element;

	/**
	 * Creates an \ElggXMLParser from a string or existing SimpleXMLElement
	 * 
	 * @param string|SimpleXMLElement $xml The XML to parse
	 */
	public function __construct($xml) {
		if ($xml instanceof SimpleXMLElement) {
			$this->_element = $xml;
		} else {
			// do not load entities
			$disable_load_entities = libxml_disable_entity_loader(true);

			$this->_element = new SimpleXMLElement($xml);

			libxml_disable_entity_loader($disable_load_entities);
		}
	}

	/**
	 * @return string The name of the element
	 */
	public function getName() {
		return $this->_element->getName();
	}

	/**
	 * @return string[] The attributes
	 */
	public function getAttributes() {
		//include namespace declarations as attributes
		$xmlnsRaw = $this->_element->getNamespaces();
		$xmlns = array();
		foreach ($xmlnsRaw as $key => $val) {
			$label = 'xmlns' . ($key ? ":$key" : $key);
			$xmlns[$label] = $val;
		}
		//get attributes and merge with namespaces
		$attrRaw = $this->_element->attributes();
		$attr = array();
		foreach ($attrRaw as $key => $val) {
			$attr[$key] = $val;
		}
		$attr = array_merge((array) $xmlns, (array) $attr);
		$result = array();
		foreach ($attr as $key => $val) {
			$result[$key] = (string) $val;
		}
		return $result;
	}

	/**
	 * @return string CData
	 */
	public function getContent() {
		return (string) $this->_element;
	}

	/**
	 * @return \ElggXMLElement[] Child elements
	 */
	public function getChildren() {
		$children = $this->_element->children();
		$result = array();
		foreach ($children as $val) {
			$result[] = new \ElggXMLElement($val);
		}

		return $result;
	}

	/**
	 * Override ->
	 * 
	 * @param string $name Property name
	 * @return mixed
	 */
	public function __get($name) {
		switch ($name) {
			case 'name':
				return $this->getName();
				break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
			case 'attributes':
				return $this->getAttributes();
				break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
			case 'content':
				return $this->getContent();
				break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
			case 'children':
				return $this->getChildren();
				break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
		}
		return null;
	}

	/**
	 * Override isset
	 * 
	 * @param string $name Property name
	 * @return boolean
	 */
	public function __isset($name) {
		switch ($name) {
			case 'name':
				return $this->getName() !== null;
				break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
			case 'attributes':
				return $this->getAttributes() !== null;
				break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
			case 'content':
				return $this->getContent() !== null;
				break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
			case 'children':
				return $this->getChildren() !== null;
				break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
		}
		return false;
	}
}


1			<?php
2			/**
3			* A parser for XML that uses SimpleXMLElement
4			*
5			* @package Elgg.Core
6			* @subpackage XML
7			*/
8			class ElggXMLElement {
9			/**
10			* @var SimpleXMLElement
11			*/
12			private $_element;
13
14			/**
15			* Creates an \ElggXMLParser from a string or existing SimpleXMLElement
16			*
17			* @param string\|SimpleXMLElement $xml The XML to parse
18			*/
19			public function __construct($xml) {
20			if ($xml instanceof SimpleXMLElement) {
21			$this->_element = $xml;
22			} else {
23			// do not load entities
24			$disable_load_entities = libxml_disable_entity_loader(true);
25
26			$this->_element = new SimpleXMLElement($xml);
27
28			libxml_disable_entity_loader($disable_load_entities);
29			}
30			}
31
32			/**
33			* @return string The name of the element
34			*/
35			public function getName() {
36			return $this->_element->getName();
37			}
38
39			/**
40			* @return string[] The attributes
41			*/
42			public function getAttributes() {
43			//include namespace declarations as attributes
44			$xmlnsRaw = $this->_element->getNamespaces();
45			$xmlns = array();
46			foreach ($xmlnsRaw as $key => $val) {
47			$label = 'xmlns' . ($key ? ":$key" : $key);
48			$xmlns[$label] = $val;
49			}
50			//get attributes and merge with namespaces
51			$attrRaw = $this->_element->attributes();
52			$attr = array();
53			foreach ($attrRaw as $key => $val) {
54			$attr[$key] = $val;
55			}
56			$attr = array_merge((array) $xmlns, (array) $attr);
57			$result = array();
58			foreach ($attr as $key => $val) {
59			$result[$key] = (string) $val;
60			}
61			return $result;
62			}
63
64			/**
65			* @return string CData
66			*/
67			public function getContent() {
68			return (string) $this->_element;
69			}
70
71			/**
72			* @return \ElggXMLElement[] Child elements
73			*/
74			public function getChildren() {
75			$children = $this->_element->children();
76			$result = array();
77			foreach ($children as $val) {
78			$result[] = new \ElggXMLElement($val);
79			}
80
81			return $result;
82			}
83
84			/**
85			* Override ->
86			*
87			* @param string $name Property name
88			* @return mixed
89			*/
90			public function __get($name) {
91			switch ($name) {
92			case 'name':
93			return $this->getName();
94			break;
			0 ignored issues – show Unused Code introduced 2017-09-27 16:16 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
95			case 'attributes':
96			return $this->getAttributes();
97			break;
			0 ignored issues – show Unused Code introduced 2017-09-27 16:16 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
98			case 'content':
99			return $this->getContent();
100			break;
			0 ignored issues – show Unused Code introduced 2017-09-27 16:16 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
101			case 'children':
102			return $this->getChildren();
103			break;
			0 ignored issues – show Unused Code introduced 2017-09-27 16:16 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
104			}
105			return null;
106			}
107
108			/**
109			* Override isset
110			*
111			* @param string $name Property name
112			* @return boolean
113			*/
114			public function __isset($name) {
115			switch ($name) {
116			case 'name':
117			return $this->getName() !== null;
118			break;
			0 ignored issues – show Unused Code introduced 2017-09-27 16:16 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
119			case 'attributes':
120			return $this->getAttributes() !== null;
121			break;
			0 ignored issues – show Unused Code introduced 2017-09-27 16:16 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
122			case 'content':
123			return $this->getContent() !== null;
124			break;
			0 ignored issues – show Unused Code introduced 2017-09-27 16:16 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
125			case 'children':
126			return $this->getChildren() !== null;
127			break;
			0 ignored issues – show Unused Code introduced 2017-09-27 16:16 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
128			}
129			return false;
130			}
131			}
132

gctools-outilsgc / gcconnex

ElggXMLElement A last analyzed 2021-05-27 17:40 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

7 Methods

Duplication Side-by-Side

Filter issues like

ElggXMLElement A
last analyzed 2021-05-27 17:40 UTC