App\Console\Commands\GitterRoomSyncCommand

Complexity

Total Complexity

3

Size/Duplication

Total Lines	58
Duplicated Lines	31.03 %

Coupling/Cohesion

Components	1
Dependencies	10

Importance

Changes	6
Bugs	0	Features	3

2 Methods

Rating	Name	Duplication	Size	Complexity
A	handle()	18	18	2
A	listen()	0	15	1

<?php
/**
 * This file is part of GitterBot package.
 *
 * @author Serafim <nesk@xakep.ru>
 * @date 26.01.2016 4:57
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */
namespace App\Console\Commands;

use App\Console\CircleProgress;
use App\User;
use Gitter\Client;
use React\EventLoop\LoopInterface;
use Illuminate\Container\Container;
use Gitter\Models\User as GitterUser;
use Gitter\Models\Room as GitterRoom;
use Gitter\Models\Message as GitterMessage;
use Illuminate\Contracts\Config\Repository;
use Gitter\Iterators\PromiseIterator\Controls;

/**
 * Class GitterRoomSyncCommand
 * @package App\Console\Commands
 */
class GitterRoomSyncCommand extends AbstractCommand
{
    /**
     * @var string
     */
    protected $signature = 'gitter:sync {room} {type=users}';

    /**
     * @var string
     */
    protected $description = 'Sync gitter room';

    /**
     * Execute the console command.
     *
     * @param Repository $config
     * @param Container $container
     *
     * @return mixed
     */
    public function handle(Repository $config, Container $container)

This method seems to be duplicated in your project.

    {
        /** @var Client $client */
        $client = $this->createClient($container, $config->get('gitter.token'));

        // Search room
        $this->findRoom($client, $this->argument('room'), function(GitterRoom $room) {

It seems like $this->argument('room') targeting Illuminate\Console\Command::argument() can also be of type array; however, App\Console\Commands\AbstractCommand::findRoom() does only seem to accept string, maybe add an additional type check?

            try {
                $this->listen($room);
            } catch (\Throwable $e) {
                $this->error($e->getMessage());
            }
        });

        /** @var LoopInterface $loop */
        $loop = $container->make(LoopInterface::class);
        $loop->run();
    }

    /**
     * @param GitterRoom $room
     */
    public function listen(GitterRoom $room)
    {
        /** @var CircleProgress $circle */
        $circle = new CircleProgress();
        $this->output->write('Fetching...');

        $room->getUsers()->fetch(function(GitterUser $gitterUser, Controls $controls) use ($circle) {
            $user = User::make($gitterUser)->login;

            $this->output->write(sprintf("\r[%s] %s %s", $circle->get(), $user, str_repeat(' ', 20)));
            flush();

            $controls->next();
        });
    }
}