Nekland / Woketo

src/Http/HttpHeadersBag.php

Doc Comments +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
 
     /**
      * @param string $name
-     * @param mixed  $value
+     * @param string  $value
      * @return self
      */
     public function add(string $name, $value)

Spacing +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
     public function add(string $name, $value)
     {
         if (!empty($this->headers[$name])) {
-            if (!\is_array($this->headers[$name])){
+            if (!\is_array($this->headers[$name])) {
                 $this->headers[$name] = [$this->headers[$name]];
             }
             $this->headers[$name][] = $value;

src/Http/Request.php

Spacing +1 added lines, -1 removed lines

@@ -96,7 +96,7 @@
 
             if (count($explodingHeader)) {
                 unset($explodingHeader[0]); // removing the name of the extension
-                foreach($explodingHeader as $variable) {
+                foreach ($explodingHeader as $variable) {
                     $explodeVariable = explode('=', $variable);
 
                     // The value can be with or without quote. We need to remove extra quotes.

src/Http/Response.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
         $stringResponse = $this->getHttpVersion() . ' ' . $this->httpResponse . "\r\n";
 
         foreach ($this->getHeaders() as $name => $content) {
-            $stringResponse .= $name . ': '. $content . "\r\n";
+            $stringResponse .= $name . ': ' . $content . "\r\n";
         }
         
         // No content to concatenate

src/Utils/BitManipulation.php

Spacing +3 added lines, -3 removed lines

@@ -8,7 +8,7 @@ 
  * For the full license, take a look to the LICENSE file
  * on the root directory of this project
  */
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Nekland\Woketo\Utils;
 
@@ -58,7 +58,7 @@ 
         if (is_string($frame)) {
             $len = BitManipulation::frameSize($frame);
 
-            if ($byteNumber < 0 || $byteNumber > ($len-1)) {
+            if ($byteNumber < 0 || $byteNumber > ($len - 1)) {
                 throw new \InvalidArgumentException(
                     sprintf('The frame is only %s bytes larges but you tried to get the %sth byte.', $len, $byteNumber)
                 );
@@ -166,7 +166,7 @@ 
             $res = 0;
 
             // for each byte, getting ord
-            for($i = 0; $i < $subStringLength; $i++) {
+            for ($i = 0; $i < $subStringLength; $i++) {
                 $res <<= 8;
                 $res += ord($subString[$i]);
             }

src/Rfc6455/Frame.php

Doc Comments +1 added lines, -1 removed lines

@@ -140,7 +140,7 @@
     /**
      * It also run checks on data.
      *
-     * @param string|int $rawData Probably more likely a string than an int, but well... why not.
+     * @param string $rawData Probably more likely a string than an int, but well... why not.
      * @return self
      * @throws IncompleteFrameException
      */

Spacing +7 added lines, -7 removed lines

@@ -8,7 +8,7 @@ 
  * on the root directory of this project
  */
 
-declare(strict_types=1);
+declare(strict_types = 1);
 namespace Nekland\Woketo\Rfc6455;
 
 use Nekland\Woketo\Exception\Frame\ControlFrameException;
@@ -26,11 +26,11 @@ 
  */
 class Frame
 {
-    const OP_CONTINUE =  0;
-    const OP_TEXT     =  1;
-    const OP_BINARY   =  2;
-    const OP_CLOSE    =  8;
-    const OP_PING     =  9;
+    const OP_CONTINUE = 0;
+    const OP_TEXT     = 1;
+    const OP_BINARY   = 2;
+    const OP_CLOSE    = 8;
+    const OP_PING     = 9;
     const OP_PONG     = 10;
 
     // To understand codes, please refer to RFC:
@@ -586,7 +586,7 @@ 
     {
         $this->config = array_merge([
             'maxPayloadSize' => Frame::$defaultMaxPayloadSize,
-        ],$config);
+        ], $config);
 
         return $this;
     }

src/Rfc6455/MessageProcessor.php

Spacing +2 added lines, -2 removed lines

@@ -95,7 +95,7 @@ 
                         // Data is now stored in the message, let's clean the variable to stop both loops.
                         $data = null;
                     }
-                } while(!$message->isComplete() && !empty($data));
+                } while (!$message->isComplete() && !empty($data));
 
                 if ($message->isComplete()) {
                     $this->processHelper($message, $socket);
@@ -118,7 +118,7 @@ 
                 $socket->end();
                 $data = '';
             }
-        } while(!empty($data));
+        } while (!empty($data));
     }
 
     /**

src/Server/WebSocketServer.php

Spacing +4 added lines, -4 removed lines

@@ -83,19 +83,19 @@ 
         $this->buildMessageProcessor();
 
         // Some optimization
-        \gc_enable();       // As the process never stops, the garbage collector will be usefull, you may need to call it manually sometimes for performance purpose
+        \gc_enable(); // As the process never stops, the garbage collector will be usefull, you may need to call it manually sometimes for performance purpose
         \set_time_limit(0); // It's by default on most server for cli apps but better be sure of that fact
     }
 
     public function setMessageHandler($messageHandler)
     {
-        if (!$messageHandler instanceof MessageHandlerInterface &&  !\is_string($messageHandler)) {
+        if (!$messageHandler instanceof MessageHandlerInterface && !\is_string($messageHandler)) {
             throw new \InvalidArgumentException('The message handler must be an instance of MessageHandlerInterface or a string.');
         }
         if (\is_string($messageHandler)) {
             try {
                 $reflection = new \ReflectionClass($messageHandler);
-                if(!$reflection->implementsInterface('Nekland\Woketo\Message\MessageHandlerInterface')) {
+                if (!$reflection->implementsInterface('Nekland\Woketo\Message\MessageHandlerInterface')) {
                     throw new \InvalidArgumentException('The messageHandler must implement MessageHandlerInterface');
                 }
             } catch (\ReflectionException $e) {
@@ -114,7 +114,7 @@ 
         $this->loop = \React\EventLoop\Factory::create();
 
         $socket = new \React\Socket\Server($this->loop);
-        $socket->on('connection', function ($socketStream) {
+        $socket->on('connection', function($socketStream) {
             $this->onNewConnection($socketStream);
         });
         $socket->listen($this->port);

Unused Use Statements -1 removed lines

@@ -23,7 +23,6 @@
 use Nekland\Woketo\Rfc6455\ServerHandshake;
 use Nekland\Woketo\Utils\SimpleLogger;
 use Psr\Log\LoggerInterface;
-use Psr\Log\LogLevel;
 use React\EventLoop\LoopInterface;
 use React\Socket\ConnectionInterface;
 

src/Server/Connection.php

Spacing +4 added lines, -4 removed lines

@@ -91,10 +91,10 @@ 
 
     private function initListeners()
     {
-        $this->socketStream->on('data', function ($data) {
+        $this->socketStream->on('data', function($data) {
             $this->processData($data);
         });
-        $this->socketStream->on('error', function ($data) {
+        $this->socketStream->on('error', function($data) {
             $this->error($data);
         });
     }
@@ -133,7 +133,7 @@ 
             $this->currentMessage = $message;
             if ($this->currentMessage->isComplete()) {
                 // Sending the message through the woketo API.
-                switch($this->currentMessage->getOpcode()) {
+                switch ($this->currentMessage->getOpcode()) {
                     case Frame::OP_TEXT:
                         $this->handler->onMessage($this->currentMessage->getContent(), $this);
                         break;
@@ -145,7 +145,7 @@ 
 
             } else {
                 // We wait for more data so we start a timeout.
-                $this->timeout = $this->loop->addTimer(Connection::DEFAULT_TIMEOUT, function () {
+                $this->timeout = $this->loop->addTimer(Connection::DEFAULT_TIMEOUT, function() {
                     $this->logger->notice('Connection to ' . $this->getIp() . ' timed out.');
                     $this->messageProcessor->timeout($this->socketStream);
                 });

src/Utils/SimpleLogger.php

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
     {
         // Doesn't log everything in not debug context
         if ($this->debug || \in_array($level, [LogLevel::CRITICAL, LogLevel::ERROR])) {
-            echo '[' . date('Y-m-d H:i:s') . '][' . $level . '] ' . $message ."\n";
+            echo '[' . date('Y-m-d H:i:s') . '][' . $level . '] ' . $message . "\n";
         }
     }
 }