kadet1090 / nucleus

Xml/XmlElement.php

Doc Comments +4 added lines, -1 removed lines

@@ -259,7 +259,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)
     {
@@ -335,6 +335,9 @@ 
         return $this->appendChild($element);
     }
 
+    /**
+     * @param XmlElement $element
+     */
     public function remove($element)
     {
         $this->_children = array_filter($this->_children, not(filter\same($element)));

Spacing +14 added lines, -14 removed lines

@@ -136,7 +136,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) {
@@ -181,7 +181,7 @@ 
         $attributes = $this->attributes();
 
         $result = "<{$this->name}";
-        $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)));
 
@@ -202,7 +202,7 @@ 
      */
     public function lookupPrefix(string $uri = null)
     {
-        return $this->getNamespaces()[ $uri ] ?? false;
+        return $this->getNamespaces()[$uri] ?? false;
     }
 
     /**
@@ -246,12 +246,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;
     }
 
     /**
@@ -263,7 +263,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;
     }
 
     /**
@@ -276,7 +276,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)]);
     }
 
     /**
@@ -285,7 +285,7 @@ 
      */
     public function getParents()
     {
-        return $this->_parent ? array_merge([ $this->_parent ], $this->_parent->getParents()) : [];
+        return $this->_parent ? array_merge([$this->_parent], $this->_parent->getParents()) : [];
     }
 
     /**
@@ -304,7 +304,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;
         }
 
@@ -327,7 +327,7 @@ 
             return false;
         }
 
-        if(is_array($element)) {
+        if (is_array($element)) {
             array_walk($element, [$this, 'appendChild']);
             return $element;
         }
@@ -338,7 +338,7 @@ 
     public function remove($element)
     {
         $this->_children = array_filter($this->_children, not(filter\same($element)));
-        if($element instanceof XmlElement) {
+        if ($element instanceof XmlElement) {
             $element->_parent = null;
         }
     }
@@ -388,7 +388,7 @@ 
             $prefix = $this->_prefix;
         }
 
-        $this->_namespaces[ $uri ] = $prefix;
+        $this->_namespaces[$uri] = $prefix;
     }
 
     public function getName()
@@ -436,7 +436,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;
     }
 
     /**
@@ -517,7 +517,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));
 

XmppClient.php

Doc Comments +1 added lines, -1 removed lines

@@ -200,7 +200,7 @@
      * Registers module in client's dependency container.
      *
      * @param ComponentInterface $module    Module to be registered
-     * @param bool|string|array  $alias     Module alias, class name by default.
+     * @param string|boolean  $alias     Module alias, class name by default.
      *                                      `true` for aliasing interfaces and parents too,
      *                                      `false` for aliasing as class name only
      *                                      array for multiple aliases,

Spacing +8 added lines, -8 removed lines

@@ -132,12 +132,12 @@ 
 
         $this->applyOptions($options);
 
-        $this->on('element', function (Features $element) {
+        $this->on('element', function(Features $element) {
             $this->_features = $element;
             $this->emit('features', [$element]);
         }, Features::class);
 
-        $this->on('close', function (Features $element) {
+        $this->on('close', function(Features $element) {
             $this->state = 'disconnected';
         }, Features::class);
     }
@@ -189,7 +189,7 @@ 
         $this->state = 'bound';
 
         $queue = new \SplQueue();
-        $this->emit('init', [ $queue ]);
+        $this->emit('init', [$queue]);
 
         \React\Promise\all(iterator_to_array($queue))->then(function() {
             $this->state = 'ready';
@@ -213,10 +213,10 @@ 
             $this->_container->set(get_class($module), $module);
 
             $this->_addToContainer($module, array_merge(class_implements($module), array_slice(class_parents($module), 1)));
-        } elseif(is_array($alias)) {
+        } elseif (is_array($alias)) {
             $this->_addToContainer($module, $alias);
         } else {
-            $this->_addToContainer($module, [ $alias === false ? get_class($module) : $alias ]);
+            $this->_addToContainer($module, [$alias === false ? get_class($module) : $alias]);
         }
     }
 
@@ -239,7 +239,7 @@ 
         $deferred = new Deferred();
 
         $this->once('element', function(Stanza $stanza) use ($deferred) {
-            if($stanza->type === "error") {
+            if ($stanza->type === "error") {
                 $deferred->reject($stanza);
             } else {
                 $deferred->resolve($stanza);
@@ -275,7 +275,7 @@ 
     //region Parser
     public function setParser(XmlParser $parser)
     {
-        if($this->state !== "disconnected") {
+        if ($this->state !== "disconnected") {
             throw new \BadMethodCallException('Parser can be changed only when client is disconnected.');
         }
 
@@ -303,7 +303,7 @@ 
             ));
         }
 
-        $this->_connector->on('connect', function ($stream) {
+        $this->_connector->on('connect', function($stream) {
             return $this->handleConnect($stream);
         });
     }

Xml/XmlElementFactory.php

Spacing +2 added lines, -2 removed lines

@@ -33,7 +33,7 @@ 
         }
 
         foreach ($lookup['<predicate>'] as list($class, $predicate)) {
-            if($predicate($tag, $namespace)) {
+            if ($predicate($tag, $namespace)) {
                 return $class;
             }
         }
@@ -64,7 +64,7 @@ 
     {
         $result = ['<predicate>' => []];
         foreach ($dictionary as $element) {
-            if($element['name'] instanceof \Closure || $element['uri'] instanceof \Closure) {
+            if ($element['name'] instanceof \Closure || $element['uri'] instanceof \Closure) {
                 $result['<predicate>'][] = [$element[0], \Kadet\Xmpp\Utils\filter\consecutive(
                     \Kadet\Xmpp\Utils\filter\predicate($element['name']),
                     \Kadet\Xmpp\Utils\filter\predicate($element['uri'])

Utils/BetterEmitter.php

Spacing +4 added lines, -4 removed lines

@@ -56,7 +56,7 @@ 
                     return false;
                 }
             } catch (\Throwable $exception) {
-                if($this->emit('exception', [ $exception, $event ])) {
+                if ($this->emit('exception', [$exception, $event])) {
                     throw $exception;
                 }
                 return false;
@@ -90,7 +90,7 @@ 
 
         $condition = with\predicate($condition);
 
-        return function (...$arguments) use ($listener, $condition) {
+        return function(...$arguments) use ($listener, $condition) {
             if ($condition(...$arguments)) {
                 return $listener(...$arguments);
             }
@@ -101,8 +101,8 @@ 
 
     private function getOnceCallable(callable $listener, $event) : callable
     {
-        return $onceListener = function (...$arguments) use (&$onceListener, $event, $listener) {
-            if(($result = $listener(...$arguments)) !== INF) {
+        return $onceListener = function(...$arguments) use (&$onceListener, $event, $listener) {
+            if (($result = $listener(...$arguments)) !== INF) {
                 $this->removeListener($event, $onceListener);
                 return $result;
             }

Stanza/Iq.php

Spacing +2 added lines, -2 removed lines

@@ -48,7 +48,7 @@ 
 
     protected function appendChild($element)
     {
-        if(count($this->children) > 0) {
+        if (count($this->children) > 0) {
             throw new \RuntimeException('Iq stanzas cannot have more than one child.');
         }
 
@@ -77,7 +77,7 @@ 
     public static function getXmlCollocations() : array
     {
         return array_merge(
-            [[ Query::class, 'name' => pass(), 'uri' => pass() ]],
+            [[Query::class, 'name' => pass(), 'uri' => pass()]],
             parent::getXmlCollocations()
         );
     }