kadet1090 / nucleus

Xml/XmlElement.php

Doc Comments +2 added lines, -2 removed lines

@@ -261,7 +261,7 @@ 
      *
      * @param string      $attribute Attribute name, optionally with prefix
      * @param string|null $uri       XML Namespace URI of attribute, prefix will be automatically looked up
-     * @return bool|mixed
+     * @return string
      */
     public function getAttribute(string $attribute, string $uri = null)
     {
@@ -483,7 +483,7 @@ 
      *
      * @param callable|string $predicate Predicate or class name
      *
-     * @return XmlElement|false
+     * @return XmlElement|null
      */
     public function get($predicate)
     {

Spacing +15 added lines, -15 removed lines

@@ -138,7 +138,7 @@ 
      */
     public function getInnerXml()
     {
-        return implode('', array_map(function ($element) {
+        return implode('', array_map(function($element) {
             if (is_string($element)) {
                 return htmlspecialchars($element);
             } elseif ($element instanceof XmlElement) {
@@ -183,7 +183,7 @@ 
         $attributes = $this->attributes();
 
         $result = "<{$this->fullName}";
-        $result .= ' ' . implode(' ', array_map(function ($key, $value) {
+        $result .= ' ' . implode(' ', array_map(function($key, $value) {
             return $key . '="' . htmlspecialchars($value, ENT_QUOTES) . '"';
         }, array_keys($attributes), array_values($attributes)));
 
@@ -204,7 +204,7 @@ 
      */
     public function lookupPrefix(string $uri = null)
     {
-        return $this->getNamespaces()[ $uri ] ?? array_search($uri, XmlParser::$predefined) ?: false;
+        return $this->getNamespaces()[$uri] ?? array_search($uri, XmlParser::$predefined) ?: false;
     }
 
     /**
@@ -248,12 +248,12 @@ 
     {
         $attribute = $this->_prefix($attribute, $uri);
         if ($value === null) {
-            unset($this->_attributes[ $attribute ]);
+            unset($this->_attributes[$attribute]);
 
             return;
         }
 
-        $this->_attributes[ $attribute ] = $value;
+        $this->_attributes[$attribute] = $value;
     }
 
     /**
@@ -265,7 +265,7 @@ 
      */
     public function getAttribute(string $attribute, string $uri = null)
     {
-        return $this->_attributes[ $this->_prefix($attribute, $uri) ] ?? false;
+        return $this->_attributes[$this->_prefix($attribute, $uri)] ?? false;
     }
 
     /**
@@ -278,7 +278,7 @@ 
      */
     public function hasAttribute(string $attribute, string $uri = null)
     {
-        return isset($this->_attributes[ $this->_prefix($attribute, $uri) ]);
+        return isset($this->_attributes[$this->_prefix($attribute, $uri)]);
     }
 
     /**
@@ -287,7 +287,7 @@ 
      */
     public function getParents()
     {
-        return $this->_parent ? array_merge([ $this->_parent ], $this->_parent->getParents()) : [];
+        return $this->_parent ? array_merge([$this->_parent], $this->_parent->getParents()) : [];
     }
 
     /**
@@ -306,7 +306,7 @@ 
     protected function setParent(XmlElement $parent)
     {
         if (!$this->_prefix && ($prefix = $parent->lookupPrefix($this->namespace)) !== false) {
-            $this->_namespaces[ $this->namespace ] = $prefix;
+            $this->_namespaces[$this->namespace] = $prefix;
             $this->_prefix                         = $prefix;
         }
 
@@ -329,7 +329,7 @@ 
             return false;
         }
 
-        if(is_array($element)) {
+        if (is_array($element)) {
             array_walk($element, [$this, 'appendChild']);
             return $element;
         }
@@ -339,14 +339,14 @@ 
 
     public function remove($element)
     {
-        if(!$element instanceof \Closure) {
+        if (!$element instanceof \Closure) {
             $element = is_array($element) ? filter\in($element) : filter\same($element);
         }
         $old = $this->_children;
         $this->_children = array_filter($this->_children, not($element));
 
         foreach (array_diff($old, $this->_children) as $removed) {
-            if($removed instanceof XmlElement) {
+            if ($removed instanceof XmlElement) {
                 $removed->_parent = null;
             }
         }
@@ -397,7 +397,7 @@ 
             $prefix = $this->_prefix;
         }
 
-        $this->_namespaces[ $uri ] = $prefix;
+        $this->_namespaces[$uri] = $prefix;
     }
 
     public function getFullName()
@@ -445,7 +445,7 @@ 
      */
     public function element(string $name, string $uri = null, int $index = 0)
     {
-        return array_values($this->elements($name, $uri))[ $index ] ?? false;
+        return array_values($this->elements($name, $uri))[$index] ?? false;
     }
 
     /**
@@ -526,7 +526,7 @@ 
     private function attributes(): array
     {
         $namespaces = $this->getNamespaces(false);
-        $namespaces = array_map(function ($prefix, $uri) {
+        $namespaces = array_map(function($prefix, $uri) {
             return [$prefix ? "xmlns:{$prefix}" : 'xmlns', $uri];
         }, array_values($namespaces), array_keys($namespaces));
 

Xml/XmlParser.php

Spacing +12 added lines, -12 removed lines

@@ -76,13 +76,13 @@ 
         xml_parser_set_option($this->_parser, XML_OPTION_SKIP_WHITE, 1);
         xml_parser_set_option($this->_parser, XML_OPTION_CASE_FOLDING, 0);
 
-        xml_set_element_handler($this->_parser, function ($parser, $name, $attrs) {
+        xml_set_element_handler($this->_parser, function($parser, $name, $attrs) {
             $this->handleElementStart($name, $attrs);
-        }, function () {
+        }, function() {
             $this->handleElementEnd();
         });
 
-        xml_set_character_data_handler($this->_parser, function ($parser, $data) {
+        xml_set_character_data_handler($this->_parser, function($parser, $data) {
             $this->handleTextData($data);
         });
 
@@ -112,12 +112,12 @@ 
 
     private function _lookup($prefix, $namespaces)
     {
-        if(isset(static::$predefined[$prefix])) {
+        if (isset(static::$predefined[$prefix])) {
             return static::$predefined[$prefix];
         }
 
-        if (isset($namespaces[ $prefix ])) {
-            return $namespaces[ $prefix ];
+        if (isset($namespaces[$prefix])) {
+            return $namespaces[$prefix];
         }
 
         return !empty($this->_stack) ? end($this->_stack)->lookupUri($prefix) : null;
@@ -128,10 +128,10 @@ 
         list($attributes, $namespaces) = $this->_attributes($attrs);
         list($tag, $prefix)            = XmlElement::resolve($name);
 
-        $uri   = $this->_lookup($prefix, $namespaces);
+        $uri = $this->_lookup($prefix, $namespaces);
 
         /** @var XmlElement $element */
-        $element = $this->factory->create($uri, $tag, [ $name, $uri ], $this->_getCollocations());
+        $element = $this->factory->create($uri, $tag, [$name, $uri], $this->_getCollocations());
 
         foreach ($namespaces as $prefix => $uri) {
             $element->setNamespace($uri, $prefix);
@@ -145,7 +145,7 @@ 
 
     private function _getCollocations()
     {
-        if(empty($this->_stack)) {
+        if (empty($this->_stack)) {
             return [];
         }
 
@@ -164,7 +164,7 @@ 
         if (count($this->_stack) > 0) {
             end($this->_stack)->append($element);
         }
-        $this->emit('parse.begin', [ $element, count($this->_stack) ]);
+        $this->emit('parse.begin', [$element, count($this->_stack)]);
 
         $this->_stack[] = $element;
     }
@@ -177,10 +177,10 @@ 
 
         $element = array_pop($this->_stack);
         if (count($this->_stack) == 1) {
-            $this->emit('element', [ $element, count($this->_stack) ]);
+            $this->emit('element', [$element, count($this->_stack)]);
         }
 
-        $this->emit('parse.end', [ $element, count($this->_stack) ]);
+        $this->emit('parse.end', [$element, count($this->_stack)]);
     }
 
     private function handleTextData($data)

Utils/filter/element.php

Spacing +3 added lines, -3 removed lines

@@ -41,7 +41,7 @@ 
 {
     $predicate = $uri instanceof \Closure ? $uri : \Kadet\Xmpp\Utils\filter\same($uri);
 
-    return function ($element) use ($predicate) {
+    return function($element) use ($predicate) {
         if (!$element instanceof XmlElement) {
             return false;
         }
@@ -74,7 +74,7 @@ 
 {
     $predicate = $name instanceof \Closure ? $name : \Kadet\Xmpp\Utils\filter\same($name);
 
-    return function ($element) use ($predicate) {
+    return function($element) use ($predicate) {
         if (!$element instanceof XmlElement) {
             return false;
         }
@@ -110,7 +110,7 @@ 
 {
     $predicate = $value instanceof \Closure ? $value : \Kadet\Xmpp\Utils\filter\equals($value);
 
-    return function ($element) use ($name, $predicate, $namespace) {
+    return function($element) use ($name, $predicate, $namespace) {
         if (!$element instanceof XmlElement) {
             return false;
         }

Stanza/Message.php

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
 
     private function bodyPredicate($language) {
         $predicate = name('body');
-        if($language !== null) {
+        if ($language !== null) {
             $predicate = all($predicate, attribute('lang', $language, XmlElement::XML));
         }