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/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:
@@ -588,7 +588,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/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";
         }
     }
 }

src/Client/Connection.php

Doc Comments +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
     }

 

     /**

-     * @param \Exception|string $error

+     * @param \Exception $error

      */

     private function onError($error)

     {

Spacing +5 added lines, -5 removed lines

@@ -52,17 +52,17 @@ 
         $this->buffer = '';

         $this->handler = $handler;

 

-        $clientPromise->then(function (Stream $stream) {

+        $clientPromise->then(function(Stream $stream) {

             $this->stream = $stream;

             $this->onConnection($stream);

-        }, function (\Exception $error){

+        }, function(\Exception $error) {

             $this->onError($error);

         });

     }

 

     private function onConnection(Stream $stream)

     {

-        $stream->on('data', function (string $data) {

+        $stream->on('data', function(string $data) {

             $this->onMessage($data);

         });

 

@@ -114,7 +114,7 @@ 
 

             if ($this->currentMessage->isComplete()) {

                 // Sending the message through the woketo API.

-                switch($this->currentMessage->getOpcode()) {

+                switch ($this->currentMessage->getOpcode()) {

                     case Frame::OP_TEXT:

                         $this->getHandler()->onMessage($this->currentMessage->getContent(), $this);

                         break;

@@ -126,7 +126,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->getLogger()->notice('Connection to ' . $this->getIp() . ' timed out.');

                     $this->messageProcessor->timeout($this->stream);

                 });

Unused Use Statements +12 added lines, -12 removed lines

@@ -12,18 +12,18 @@
 namespace Nekland\Woketo\Client;

 

 

-use Nekland\Woketo\Core\AbstractConnection;

-use Nekland\Woketo\Exception\Http\IncompleteHttpMessageException;

-use Nekland\Woketo\Exception\RuntimeException;

-use Nekland\Woketo\Exception\WebsocketException;

-use Nekland\Woketo\Http\Response;

-use Nekland\Woketo\Http\Url;

-use Nekland\Woketo\Message\MessageHandlerInterface;

-use Nekland\Woketo\Rfc6455\Frame;

-use Nekland\Woketo\Rfc6455\Handshake\ClientHandshake;

-use Nekland\Woketo\Rfc6455\MessageProcessor;

-use React\EventLoop\LoopInterface;

-use React\Promise\PromiseInterface;

+use Nekland\Woketo\Core\AbstractConnection;
+use Nekland\Woketo\Exception\Http\IncompleteHttpMessageException;
+use Nekland\Woketo\Exception\RuntimeException;
+use Nekland\Woketo\Exception\WebsocketException;
+use Nekland\Woketo\Http\Response;
+use Nekland\Woketo\Http\Url;
+use Nekland\Woketo\Message\MessageHandlerInterface;
+use Nekland\Woketo\Rfc6455\Frame;
+use Nekland\Woketo\Rfc6455\Handshake\ClientHandshake;
+use Nekland\Woketo\Rfc6455\MessageProcessor;
+use React\EventLoop\LoopInterface;
+use React\Promise\PromiseInterface;
 use React\Stream\Stream;

 

 class Connection extends AbstractConnection

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/Server/WebSocketServer.php

Spacing +5 added lines, -5 removed lines

@@ -96,7 +96,7 @@ 
         $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
     }
 
@@ -106,13 +106,13 @@ 
      */
     public function setMessageHandler($messageHandler, $uri = '*')
     {
-        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) {
@@ -144,7 +144,7 @@ 
             $this->getLogger()->info('Enabled ssl');
         }
 
-        $this->server->on('connection', function ($socketStream) {
+        $this->server->on('connection', function($socketStream) {
             $this->onNewConnection($socketStream);
         });
         $this->server->listen($this->port);
@@ -159,7 +159,7 @@ 
      */
     private function onNewConnection(ConnectionInterface $socketStream)
     {
-        $connection = new Connection($socketStream, function ($uri, Connection $connection) {
+        $connection = new Connection($socketStream, function($uri, Connection $connection) {
             return $this->getMessageHandler($uri, $connection);
         }, $this->loop, $this->messageProcessor);
 

Unused Use Statements -1 removed lines

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

src/Server/Connection.php

Spacing +4 added lines, -4 removed lines

@@ -69,10 +69,10 @@ 
 
     private function initListeners()
     {
-        $this->stream->on('data', function ($data) {
+        $this->stream->on('data', function($data) {
             $this->processData($data);
         });
-        $this->stream->on('error', function ($data) {
+        $this->stream->on('error', function($data) {
             $this->error($data);
         });
     }
@@ -114,7 +114,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->getHandler()->onMessage($this->currentMessage->getContent(), $this);
                         break;
@@ -126,7 +126,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->stream);
                 });

src/Http/Response.php

Spacing +3 added lines, -3 removed lines

@@ -99,7 +99,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
@@ -124,7 +124,7 @@ 
      * @return Response
      * @throws HttpException
      */
-    public static function create(string &$data) : Response
+    public static function create(string & $data) : Response
     {
         if (!\preg_match('/\\r\\n\\r\\n/', $data)) {
             throw new IncompleteHttpMessageException();
@@ -162,7 +162,7 @@ 
     {
         $httpElements = \explode(' ', $firstLine);
 
-        if (!\preg_match('/HTTP\/[1-2\.]+/',$httpElements[0])) {
+        if (!\preg_match('/HTTP\/[1-2\.]+/', $httpElements[0])) {
             throw Response::createNotHttpException($firstLine);
         }
         $response->setHttpVersion($httpElements[0]);