kadet1090 / nucleus

Module/TlsEnabler.php

Spacing +2 added lines, -2 removed lines

@@ -29,11 +29,11 @@
     {
         parent::setClient($client);
 
-        $client->on('features', function (Features $features) {
+        $client->on('features', function(Features $features) {
             return $this->handleFeatures($features);
         }, null, 10);
 
-        $client->on('element', function (XmlElement $element) {
+        $client->on('element', function(XmlElement $element) {
             $this->handleTls($element);
         }, with\xmlns(Features\StartTls::XMLNS));
     }

XmppClient.php

Spacing +4 added lines, -4 removed lines

@@ -110,11 +110,11 @@ 
 
         $this->applyOptions($options);
 
-        $this->_connector->on('connect', function (...$arguments) {
+        $this->_connector->on('connect', function(...$arguments) {
             return $this->emit('connect', $arguments);
         });
 
-        $this->on('element', function (Features $element) {
+        $this->on('element', function(Features $element) {
             $this->_features = $element;
             $this->emit('features', [$element]);
         }, Features::class);
@@ -133,7 +133,7 @@ 
             'password'  => -1
         ]);
 
-        if($options['default-modules']) {
+        if ($options['default-modules']) {
             $options['modules'] = array_merge([
                 TlsEnabler::class    => new TlsEnabler(),
                 Binding::class       => new Binding(),
@@ -218,7 +218,7 @@ 
             ));
         }
 
-        $this->_connector->on('connect', function ($stream) {
+        $this->_connector->on('connect', function($stream) {
             $this->handleConnect($stream);
         });
     }

Stanza/Stanza.php

Spacing +5 added lines, -5 removed lines

@@ -53,7 +53,7 @@ 
 
     public function getFrom()
     {
-        if($this->_from === false) {
+        if ($this->_from === false) {
             $this->_from = $this->hasAttribute('from') ? new Jid($this->_from) : null;
         }
 
@@ -62,7 +62,7 @@ 
 
     public function getTo()
     {
-        if($this->_to === false) {
+        if ($this->_to === false) {
             $this->_to = $this->hasAttribute('to') ? new Jid($this->_to) : null;
         }
 
@@ -86,7 +86,7 @@ 
 
     public function setFrom($from)
     {
-        if($from instanceof Jid) {
+        if ($from instanceof Jid) {
             $this->_from = $from instanceof Jid ? $from : new Jid($from);
         }
 
@@ -95,7 +95,7 @@ 
 
     public function setTo($to)
     {
-        if($to instanceof Jid) {
+        if ($to instanceof Jid) {
             $this->_to = $to instanceof Jid ? $to : new Jid($to);
         }
 
@@ -143,7 +143,7 @@ 
     public static function getXmlCollocations() : array
     {
         return [
-            [ Error::class, 'name' => 'error', 'uri' => 'jabber:client' ],
+            [Error::class, 'name' => 'error', 'uri' => 'jabber:client'],
         ];
     }
 }

Stanza/Error.php

Spacing +4 added lines, -4 removed lines

@@ -66,7 +66,7 @@ 
      */
     public function setType(string $type)
     {
-        if(!in_array($type, ['auth', 'cancel', 'continue', 'modify', 'wait'])) {
+        if (!in_array($type, ['auth', 'cancel', 'continue', 'modify', 'wait'])) {
             throw new InvalidArgumentException(
                 format("Error type must be 'auth', 'cancel', 'continue', 'modify' or 'wait', '{type}' given.", [
                     'type' => $type
@@ -112,7 +112,7 @@ 
      */
     public function setText(string $text)
     {
-        if(!$this->text) {
+        if (!$this->text) {
             $this->append(new XmlElement('text', self::XMLNS));
         }
 
@@ -134,7 +134,7 @@ 
             !$condition instanceof XmlElement ? $this->_definedCondition($condition) : $condition
         ];
 
-        if($description !== null) {
+        if ($description !== null) {
             $content[] = new XmlElement('text', self::XMLNS, ['content' => $description]);
         }
 
@@ -145,7 +145,7 @@ 
 
     private function _definedCondition(string $condition) : XmlElement
     {
-        if(!in_array($condition, [
+        if (!in_array($condition, [
             'bad-request', 'conflict', 'feature-not-implemented', 'forbidden',
             'gone', 'internal-server-error', 'item-not-found', 'jid-malformed',
             'not-acceptable', 'not-allowed', 'not-authorized', 'policy-violation',

Stream/Features.php

Spacing +2 added lines, -2 removed lines

@@ -38,7 +38,7 @@ 
      */
     public function __construct(array $features = [])
     {
-        parent::__construct('stream:features', XmlStream::NAMESPACE_URI, [ 'content' => $features ]);
+        parent::__construct('stream:features', XmlStream::NAMESPACE_URI, ['content' => $features]);
     }
 
     /**
@@ -51,7 +51,7 @@ 
 
     public function getMechanisms()
     {
-        return array_map(function (XmlElement $element) {
+        return array_map(function(XmlElement $element) {
             return $element->innerXml;
         }, $this->get(\Kadet\Xmpp\Utils\filter\tag('mechanisms'))->children ?? []);
     }

Xml/XmlParser.php

Spacing +11 added lines, -11 removed lines

@@ -71,13 +71,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);
         });
 
@@ -111,8 +111,8 @@ 
             return 'http://www.w3.org/2000/xmlns/';
         }
 
-        if (isset($namespaces[ $prefix ])) {
-            return $namespaces[ $prefix ];
+        if (isset($namespaces[$prefix])) {
+            return $namespaces[$prefix];
         }
 
         return !empty($this->_stack) ? end($this->_stack)->lookupUri($prefix) : null;
@@ -123,10 +123,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);
@@ -140,7 +140,7 @@ 
 
     private function _getCollocations()
     {
-        if(empty($this->_stack)) {
+        if (empty($this->_stack)) {
             return [];
         }
 
@@ -159,7 +159,7 @@ 
         if (count($this->_stack) > 1) {
             end($this->_stack)->append($element);
         }
-        $this->emit('parse.begin', [ $element ]);
+        $this->emit('parse.begin', [$element]);
 
         $this->_stack[] = $element;
     }
@@ -172,10 +172,10 @@ 
 
         $element = array_pop($this->_stack);
         if (count($this->_stack) == 1) {
-            $this->emit('element', [ $element ]);
+            $this->emit('element', [$element]);
         }
 
-        $this->emit('parse.end', [ $element ]);
+        $this->emit('parse.end', [$element]);
     }
 
     private function handleTextData($data)

Xml/XmlElement.php

Spacing +13 added lines, -13 removed lines

@@ -135,7 +135,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) {
@@ -180,7 +180,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)));
 
@@ -201,7 +201,7 @@ 
      */
     public function lookupPrefix(string $uri = null)
     {
-        return $this->getNamespaces()[ $uri ] ?? false;
+        return $this->getNamespaces()[$uri] ?? false;
     }
 
     /**
@@ -245,12 +245,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;
     }
 
     /**
@@ -262,7 +262,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;
     }
 
     /**
@@ -275,7 +275,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)]);
     }
 
     /**
@@ -284,7 +284,7 @@ 
      */
     public function getParents()
     {
-        return $this->_parent ? array_merge([ $this->_parent ], $this->_parent->getParents()) : [];
+        return $this->_parent ? array_merge([$this->_parent], $this->_parent->getParents()) : [];
     }
 
     /**
@@ -303,7 +303,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;
         }
 
@@ -326,7 +326,7 @@ 
             return false;
         }
 
-        if(is_array($element)) {
+        if (is_array($element)) {
             array_walk($element, [$this, 'append']);
             return $element;
         }
@@ -370,7 +370,7 @@ 
             $prefix = $this->_prefix;
         }
 
-        $this->_namespaces[ $uri ] = $prefix;
+        $this->_namespaces[$uri] = $prefix;
     }
 
     public function getName()
@@ -418,7 +418,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;
     }
 
     /**
@@ -492,7 +492,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));
 

Exception/Protocol/BindingException.php

Spacing +2 added lines, -2 removed lines

@@ -42,13 +42,13 @@
 
     private static function _conditionDescription(Error $error)
     {
-        if(!empty($error->text)) {
+        if (!empty($error->text)) {
             return $error->text;
         }
 
         return [
             'bad-request' => 'Bad Request: "{resource}" is not valid XMPP resource identifier.',
             'conflict'    => 'Conflict: {bare}/{resource} is already in use.'
-        ][$error->condition] ?? 'Unknown reason: '.$error->condition;
+        ][$error->condition] ?? 'Unknown reason: ' . $error->condition;
     }
 }

Utils/Filter/element.php

Doc Comments +2 added lines, -2 removed lines

@@ -67,7 +67,7 @@ 
  * $bar($element); // false, as element name is foo
  * ```
  *
- * @param string|\Closure $name Expected element name or predicate.
+ * @param string $name Expected element name or predicate.
  * @return \Closure
  */
 function name($name)
@@ -100,7 +100,7 @@ 
  * $bar($element); // false, as element name is foo
  * ```
  *
- * @param string|\Closure $name Expected element name or predicate.
+ * @param string $name Expected element name or predicate.
  * @param                 $value
  * @return \Closure
  */

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;
         }
@@ -108,7 +108,7 @@ 
 {
     $predicate = $value instanceof \Closure ? $value : \Kadet\Xmpp\Utils\filter\equals($value);
 
-    return function ($element) use ($name, $predicate) {
+    return function($element) use ($name, $predicate) {
         if (!$element instanceof XmlElement) {
             return false;
         }