RunServer::getTransportProvider() - Code Metrics - Inspection of "Merge branch 'master' of https://github.com/sonrac..." - sonrac/laravel-wamp - Measure and Improve Code Quality continuously with Scrutinizer

Completed
Push — master ( 7235dd...b44091 )

by Sergii
created 2017-11-07 16:20 UTC
RunServer::getTransportProvider() B

↳ Parent: RunServer
Complexity

Conditions	5
Paths	3
Size

Total Lines	15
Code Lines	7
Duplication

Lines	15
Ratio	100 %
Importance

Changes
Metric	Value
cc	5
eloc	7
nc	3
nop	0
dl	15
loc	15
rs	8.8571
c	0
b	0
f	0
<?php
/**
 * Created by PhpStorm.
 *
 * @author Donii Sergii <[email protected]>
 * Date: 10/24/17
 * Time: 10:33 AM
 */

namespace sonrac\WAMP\Commands;

use Illuminate\Console\Command;
use sonrac\WAMP\Exceptions\InvalidWampTransportProvider;

/**
 * Class RunServer
 * Run WAMP server command.
 */
class RunServer extends Command
{
    use WAMPCommandTrait;
    /**
     * WAMP router host.
     *
     * @var string
     */
    protected $host = '127.0.0.1';

    /**
     * {@inheritdoc}
     */
    protected $name = 'wamp:run-server {--realm=?} {--host=?} {--port=?} {--tls?} {--transport-provider=?}
    {--no-loop?} {--no-debug?} {--in-background?} {--client-transport-provider=?} {--route-path=?}';

    /**
     * {@inheritdoc}
     */
    protected $signature = 'wamp:run-server
                                {--realm= : Specify WAMP realm to be used}
                                {--host= : Specify the router host}
                                {--port= : Specify the router port}
                                {--tls : Specify the router protocol as wss}
                                {--no-debug : Disable debug mode.}
                                {--no-loop : Disable loop runner}
                                {--transport-provider : Transport provider class}
                                {--route-path= : Path to routes config}
                                {--client-transport-provider= : Client transport provider class}
                                {--in-background : Run task in background}
                                ';

    /**
     * {@inheritdoc}
     */
    protected $description = 'Run wamp server';

    /**
     * Wamp server.
     *
     * @var \Thruway\Peer\ClientInterface|\sonrac\WAMP\Client
     */
    protected $WAMPServer = null;

    /**
     * Client transport provider class.
     *
     * @var null|string
     *
     * @author Donii Sergii <[email protected]>
     */
    protected $clientTransportProvider = null;

    /**
     * Run server handle.
     *
     * @throws \Exception
     */
    public function handle()
    {
        $this->parseOptions();
        $this->changeWampLogger();

        $clientCommand = 'php artisan wamp:register-routes' . $this->getCommandLineOptions();

        if ($this->clientTransportProvider) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
            $clientCommand .= ' --transport-provider=' . $this->clientTransportProvider;
        }

        RunCommandInBackground::factory($clientCommand)->runInBackground();

        if (!$this->runInBackground) {
            $this->WAMPServer = app()->wampRouter;
            $this->WAMPServer = /** @scrutinizer ignore-call */ app()->wampRouter;
            $this->WAMPServer->registerModule($this->getTransportProvider());
            $this->WAMPServer->start(!$this->runOnce);
        } else {
            $serverCommand = 'php artisan wamp:run-server ' . $this->getCommandLineOptions();

            if ($this->clientTransportProvider) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
                $serverCommand .= ' --client-transport-provider=' . $this->clientTransportProvider;
            }

            $this->addPidToLog(RunCommandInBackground::factory($serverCommand)->runInBackground(), 'servers.pids');
        }
    }

    /**
     * Get commandline options for background command
     *
     * @return string
     *
     * @author Donii Sergii <[email protected]>
     */
    protected function getCommandLineOptions()
    {
        $command = ' --port=' . $this->port .
            ' --host=' . $this->host .
            ' --realm=' . $this->realm;

        if ($this->clientTransportProvider) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
            $command .= ' --transport-provider=' . $this->clientTransportProvider;
        }

        if ($this->noDebug) {
            $command .= ' --no-debug';
        }

        if ($this->tls) {
            $command .= ' --tls';
        }

        if ($this->routePath) {
            $command .= ' --route-path=' . $this->routePath;
        }

        if ($this->noLoop) {
            $command .= ' --no-loop';
        }

        return $command;
    }

    /**
     * Run server handle
     *
     * @throws \Exception
     */
    public function fire()
    {
        return $this->handle();
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
    }

    /**
     * Merge config & input options.
     */
    protected function parseOptions()
    {
        $this->clientTransportProvider = $this->getOptionFromInput('client-transport-provider');
class Id
{
    public $id;

    public function __construct($id)
    {
        $this->id = $id;
    }

}

class Account
{
    /** @var  Id $id */
    public $id;
}

$account_id = false;

if (starsAreRight()) {
    $account_id = new Id(42);
}

$account = new Account();
if ($account instanceof Id)
{
    $account->id = $account_id;
}
        $this->parseBaseOptions();
    }

    /**
     * Get WAMP server transport provider.
     *
     * @return null|string|\Thruway\Transport\RatchetTransportProvider
     *
     * @throws \sonrac\WAMP\Exceptions\InvalidWampTransportProvider
     *
     * @author Donii Sergii <[email protected]>
     */
    protected function getTransportProvider()

    {
        if (is_object($this->transportProvider)) {
            return $this->transportProvider;
        }

        if (is_null($this->transportProvider) || empty($this->transportProvider)) {
            $this->transportProvider = 'Thruway\Transport\RatchetTransportProvider';
        }

        if (is_string($this->transportProvider)) {
            return $this->transportProvider = new $this->transportProvider($this->host, $this->port);
        }

        throw new InvalidWampTransportProvider();
    }
}

sonrac / laravel-wamp

Push — master ( 7235dd...b44091 )

RunServer::getTransportProvider() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like
