kadet1090 / KeyLighter

Tests/Samples/php/huge.php

Braces +100 added lines, -56 removed lines

@@ -204,8 +204,9 @@ 
      */
     public function __construct(Jid $jid, $password, AbstractConnector $connector = null)
     {
-        if(!$connector)
-            $connector = new TcpConnector($jid->server);
+        if(!$connector) {
+                    $connector = new TcpConnector($jid->server);
+        }
 
         $this->connector = $connector;
         $this->connector->client = $this;
@@ -262,27 +263,31 @@ 
         $this->connector->onSend->add(function ($socket, $packet) use ($settings) {
             $len = strlen($packet);
 
-            if (function_exists('tidy_repair_string'))
-                $packet = trim(tidy_repair_string($packet, $settings));
+            if (function_exists('tidy_repair_string')) {
+                            $packet = trim(tidy_repair_string($packet, $settings));
+            }
 
-            if (isset($this->logger))
-                $this->logger->debug("Sent {length} bytes: \n{packet}", [
+            if (isset($this->logger)) {
+                            $this->logger->debug("Sent {length} bytes: \n{packet}", [
                     'length' => $len,
                     'packet' => $packet
                 ]);
+            }
         });
 
         $this->connector->onReceive->add(function ($socket, $packet) use ($settings) {
             $len = strlen($packet);
 
-            if (function_exists('tidy_repair_string'))
-                $packet = trim(tidy_repair_string($packet, $settings));
+            if (function_exists('tidy_repair_string')) {
+                            $packet = trim(tidy_repair_string($packet, $settings));
+            }
 
-            if (isset($this->logger))
-                $this->logger->debug("Received {length} bytes: \n{packet}", [
+            if (isset($this->logger)) {
+                            $this->logger->debug("Received {length} bytes: \n{packet}", [
                     'length' => $len,
                     'packet' => $packet
                 ]);
+            }
         });
 
         XmlBranch::$bind['iq']       = 'Kadet\\Xmpp\\Stanza\\Iq';
@@ -375,7 +380,9 @@ 
 
     private function startTls()
     {
-        if ($this->logger) $this->logger->notice('Starting TLS negotiation.');
+        if ($this->logger) {
+            $this->logger->notice('Starting TLS negotiation.');
+        }
 
         $xml = new XmlBranch('starttls');
         $xml->addAttribute('xmlns', 'urn:ietf:params:xml:ns:xmpp-tls');
@@ -389,10 +396,11 @@ 
      */
     private function auth()
     {
-        if ($this->logger)
-            $this->logger->notice('SASL Auth, available mechanisms: {mechanisms}', [
+        if ($this->logger) {
+                    $this->logger->notice('SASL Auth, available mechanisms: {mechanisms}', [
                 'mechanisms' => implode(', ', array_map(function (XmlBranch $item) {
                     return $item->content;
+        }
                 }, (array)$this->connector->features->mechanisms[0]->mechanism))
             ]);
 
@@ -402,15 +410,18 @@ 
         $mechanism = null;
         $this->connector->features->mechanisms[0]->mechanism->getIterator();
         foreach ($this->connector->features->mechanisms[0]->mechanism as $current) {
-            if ($mechanism = SaslFactory::get($current->content, $this->jid, $this->password))
-                break;
+            if ($mechanism = SaslFactory::get($current->content, $this->jid, $this->password)) {
+                            break;
+            }
         }
 
-        if (!$mechanism)
-            throw new \RuntimeException('This client is not supporting any of server auth mechanisms.');
+        if (!$mechanism) {
+                    throw new \RuntimeException('This client is not supporting any of server auth mechanisms.');
+        }
 
-        if ($this->logger)
-            $this->logger->notice('Chosen mechanism: {mechanism}', ['mechanism' => $current->content]);
+        if ($this->logger) {
+                    $this->logger->notice('Chosen mechanism: {mechanism}', ['mechanism' => $current->content]);
+        }
 
         $this->_mechanism = $mechanism;
 
@@ -433,13 +444,15 @@ 
     public function _onAuth(XmppClient $client, Stanza $result)
     {
         if ($result->xml->getName() == 'success') {
-            if ($this->logger)
-                $this->logger->info('SASL Auth successful.');
+            if ($this->logger) {
+                            $this->logger->info('SASL Auth successful.');
+            }
 
             $this->connector->streamRestart($this->jid);
             $this->_bind();
-        } elseif ($this->logger)
-            $this->logger->error('SASL Auth failed, reason: {reason}', ['reason' => $result->text[0]]);
+        } elseif ($this->logger) {
+                    $this->logger->error('SASL Auth failed, reason: {reason}', ['reason' => $result->text[0]]);
+        }
     }
 
     /**
@@ -476,13 +489,15 @@ 
     public function _onTls(XmppClient $client, Stanza $result)
     {
         if ($result->tag == 'proceed') {
-            if ($this->logger)
-                $this->logger->info('TLS Connection established.');
+            if ($this->logger) {
+                            $this->logger->info('TLS Connection established.');
+            }
 
             $this->connector->startTls();
             $this->connector->streamRestart($this->jid);
-        } else
-            throw new \RuntimeException('Tls negotiation failed.');
+        } else {
+                    throw new \RuntimeException('Tls negotiation failed.');
+        }
     }
 
     /**
@@ -504,8 +519,9 @@ 
             $this->write($iq->asXml());
             $this->isReady = true;
             $this->onReady->run($this);
-        } else
-            throw new \RuntimeException('Resource binding error.');
+        } else {
+                    throw new \RuntimeException('Resource binding error.');
+        }
     }
 
     /**
@@ -593,10 +609,11 @@ 
             case 'success':
             case 'failure':
             case 'proceed':
-                if ($stanza['xmlns'] == 'urn:ietf:params:xml:ns:xmpp-sasl')
-                    $this->onAuth->run($this, $stanza);
-                elseif ($stanza['xmlns'] == 'urn:ietf:params:xml:ns:xmpp-tls')
-                    $this->onTls->run($this, $stanza);
+                if ($stanza['xmlns'] == 'urn:ietf:params:xml:ns:xmpp-sasl') {
+                                    $this->onAuth->run($this, $stanza);
+                } elseif ($stanza['xmlns'] == 'urn:ietf:params:xml:ns:xmpp-tls') {
+                                    $this->onTls->run($this, $stanza);
+                }
 
                 break;
             case 'challenge':
@@ -635,7 +652,9 @@ 
         $channelJid = $packet->from->bare();
         $jid        = new Jid($channelJid);
 
-        if (!$jid->isChannel()) return;
+        if (!$jid->isChannel()) {
+            return;
+        }
 
         if ($packet->type != 'unavailable') {
             if (isset($this->rooms[$channelJid]->users[$packet->from->resource])) {
@@ -645,7 +664,9 @@ 
                 if (
                     (string)$user->jid == (string)$this->jid ||
                     $this->rooms[$channelJid]->nick == $packet->from->resource
-                ) $user->self = true;
+                ) {
+                    $user->self = true;
+                }
 
                 $this->onJoin->run(
                     $this,
@@ -659,7 +680,9 @@ 
         } elseif (isset($this->rooms[$channelJid])) {
             $item = $packet->xpath('//user:item', ['user' => 'http://jabber.org/protocol/muc#user']);
 
-            if (!isset($item[0])) return;
+            if (!isset($item[0])) {
+                return;
+            }
             $item = $item[0];
 
             // Nickname change
@@ -700,13 +723,18 @@ 
      */
     public function _onMessage(XmppClient $client, Message $packet)
     {
-        if ($packet->type != 'groupchat' || !isset($this->rooms[$packet->from->bare()])) return;
+        if ($packet->type != 'groupchat' || !isset($this->rooms[$packet->from->bare()])) {
+            return;
+        }
 
-        if (isset($packet->subject))
-            $this->rooms[$packet->from->bare()]->subject = $packet->subject;
+        if (isset($packet->subject)) {
+                    $this->rooms[$packet->from->bare()]->subject = $packet->subject;
+        }
 
-        if (!isset($packet->delay) && $this->rooms[$packet->from->bare()]->subject === false)
-            $this->rooms[$packet->from->bare()]->subject = ''; // Some strange workaround, servers doesn't meet specification... ;(
+        if (!isset($packet->delay) && $this->rooms[$packet->from->bare()]->subject === false) {
+                    $this->rooms[$packet->from->bare()]->subject = '';
+        }
+        // Some strange workaround, servers doesn't meet specification... ;(
     }
 
     /**
@@ -714,8 +742,9 @@ 
      */
     public function process()
     {
-        if ($this->isReady)
-            $this->onTick->run($this);
+        if ($this->isReady) {
+                    $this->onTick->run($this);
+        }
 
         Timer::update();
         $this->connector->read();
@@ -738,7 +767,9 @@ 
      */
     public function getUserByJid(Jid $user)
     {
-        if (!$user->fromChannel()) return null;
+        if (!$user->fromChannel()) {
+            return null;
+        }
 
         return isset($this->rooms[$user->bare()]->users[$user->resource]) ?
             $this->rooms[$user->bare()]->users[$user->resource] :
@@ -819,7 +850,10 @@ 
      */
     public function join(Jid $room, $nick)
     {
-        if (!$room->isChannel()) throw new \InvalidArgumentException('room'); // YOU SHALL NOT PASS
+        if (!$room->isChannel()) {
+            throw new \InvalidArgumentException('room');
+        }
+        // YOU SHALL NOT PASS
 
         $xml = new xmlBranch("presence");
         $xml->addAttribute("from", $this->jid->__toString())
@@ -845,7 +879,9 @@ 
      */
     public function leave(Jid $room)
     {
-        if (!$room->isChannel() || !isset($this->rooms[$room->bare()])) throw new \InvalidArgumentException('room');
+        if (!$room->isChannel() || !isset($this->rooms[$room->bare()])) {
+            throw new \InvalidArgumentException('room');
+        }
 
         $xml = new xmlBranch("presence");
         $xml->addAttribute("from", $this->jid->__toString())
@@ -875,8 +911,9 @@ 
      */
     public function role(Jid $room, $nick, $role, $reason = '')
     {
-        if (!in_array($role, array('visitor', 'none', 'participant', 'moderator')))
-            throw new \InvalidArgumentException('role');
+        if (!in_array($role, array('visitor', 'none', 'participant', 'moderator'))) {
+                    throw new \InvalidArgumentException('role');
+        }
 
         $xml = new xmlBranch("iq");
         $xml->addAttribute("type", "set")
@@ -889,7 +926,9 @@ 
         $xml->query[0]->item[0]->addAttribute("nick", $nick);
         $xml->query[0]->item[0]->addAttribute("role", $role);
 
-        if (!empty($reason)) $xml->query[0]->item[0]->addChild(new xmlBranch("reason"))->setContent($reason);
+        if (!empty($reason)) {
+            $xml->query[0]->item[0]->addChild(new xmlBranch("reason"))->setContent($reason);
+        }
 
         $this->write($xml->asXml());
     }
@@ -909,8 +948,9 @@ 
      */
     public function affiliate(Jid $room, Jid $user, $affiliation, $reason = '')
     {
-        if (!in_array($affiliation, array('none', 'outcast', 'member', 'admin', 'owner')))
-            throw new \InvalidArgumentException('affiliation');
+        if (!in_array($affiliation, array('none', 'outcast', 'member', 'admin', 'owner'))) {
+                    throw new \InvalidArgumentException('affiliation');
+        }
 
         $xml = new xmlBranch("iq");
         $xml->addAttribute("type", "set")
@@ -923,7 +963,9 @@ 
         $xml->query[0]->item[0]->addAttribute("jid", $user->bare());
         $xml->query[0]->item[0]->addAttribute("affiliation", $affiliation);
 
-        if (!empty($reason)) $xml->query[0]->item[0]->addChild(new xmlBranch("reason"))->setContent($reason);
+        if (!empty($reason)) {
+            $xml->query[0]->item[0]->addChild(new xmlBranch("reason"))->setContent($reason);
+        }
 
         $this->write($xml->asXml());
     }
@@ -960,8 +1002,9 @@ 
      */
     public function affiliationList(Jid $room, $affiliation, callable $delegate)
     {
-        if (!in_array($affiliation, array('none', 'outcast', 'member', 'admin', 'owner')))
-            throw new \InvalidArgumentException('affiliation');
+        if (!in_array($affiliation, array('none', 'outcast', 'member', 'admin', 'owner'))) {
+                    throw new \InvalidArgumentException('affiliation');
+        }
 
         $xml = new xmlBranch("iq");
         $id  = uniqid('affiliate_');
@@ -1009,7 +1052,8 @@ 
     }
 
     public function __destruct() {
-        if($this->connector->connected)
-            $this->disconnect();
+        if($this->connector->connected) {
+                    $this->disconnect();
+        }
     }
 }
\ No newline at end of file