gomoob/php-websocket-server

Method onMessage has 32 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function onMessage(ConnectionInterface $connection, $strMessage)
    {
        // In development mode we log a message
        $this->logger->debug(
            'Receiving message on WebSocket connection with resource id \'{resourceId}\'.',

Missing class import via use statement (line '137', column '23').

Open

            throw new \InvalidArgumentException($errorMessage, -1, $iaex);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Missing class import via use statement (line '152', column '23').

Open

            throw new \InvalidArgumentException($errorMessage);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Missing class import via use statement (line '97', column '23').

Open

            throw new \InvalidArgumentException('Connection opening is not authorized !');

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Avoid using static access to class '\Gomoob\WebSocket\Server\QueryStringTagsParser' in method 'onOpen'.

Open

        $tags = QueryStringTagsParser::parse($connection->WebSocket->request->getQuery());

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class '\Gomoob\WebSocket\Request\WebSocketRequest' in method 'onMessage'.

Open

            $webSocketRequest = WebSocketRequest::createFromJSON($strMessage, $this->messageParser);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}