plunner\Console\Commands\Optimise\OptimiseCommand

Complexity

Total Complexity

10

Size/Duplication

Total Lines	88
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

12.12%

Importance

Changes	2
Bugs	1	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	handle()	0	11	2
A	syncAll()	0	14	4
A	makeBackground()	0	6	2
A	makeForeground()	0	6	1

<?php

namespace plunner\Console\Commands\Optimise;

use Illuminate\Console\Command;
use Illuminate\Console\Scheduling\Schedule;
use plunner\Company;

/**
 * Class OptimiseCommand
 * @package plunner\Console\Commands\Optimise
 * @author Claudio Cardinale <cardi@thecsea.it>
 * @copyright 2015 Claudio Cardinale
 * @version 1.0.0
 */
class OptimiseCommand extends Command
{
    const BACKGROUND_MOD_MEX = 'background mode';
    const BACKGROUND_COMPLETED_MEX = 'All background tasks started';
    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'optimise:meetings {companyId?}  {--background}';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Optimise meetings';

    /**
    * @var Schedule laravel schedule object needed to perform command in background
    */
    private $schedule;

    /**
     * Create a new command instance.
     * @param Schedule $schedule
     *
     */
    public function __construct(Schedule $schedule)
    {
        parent::__construct();
        $this->schedule = $schedule;
    }

    /**
     * Execute the console command.
     *
     * @return mixed
     */
    public function handle()
    {
        //
        //TODO insert a timeout
        //TODO try...catch with destruct
        $companyId = $this->argument('companyId');
        if(is_numeric($companyId))
            $this->makeForeground(Company::findOrFail($companyId));
        else
            $this->syncAll();
    }

    private function syncAll()
    {
        $calendars = Caldav::all();
        if($this->option('background')) {
            \Log::debug(self::BACKGROUND_MOD_MEX);
            $this->info(self::BACKGROUND_MOD_MEX);
            foreach ($calendars as $calendar)
                $this->makeBackground($calendar);
            \Log::debug(self::BACKGROUND_COMPLETED_MEX);
            $this->info(self::BACKGROUND_COMPLETED_MEX);
        }else
            foreach($calendars as $calendar)
                $this->makeForeground($calendar);
    }

    /**
     * optimise company via exec command
     * @param Company $company
     */
    private function makeBackground(Company $company)
    {
        $event = $this->schedule->command('optimise:meetings '.$company->id)->withoutOverlapping();
        if($event->isDue($this->laravel))
            $event->run($this->laravel);
    }

    /**
     * optimise company foreground
     * @param Company $company
     */
    private function makeForeground(Company $company)
    {
        $this->info('Optimisation company '. $company->id.' started');
        (new Optimise($company))->optimise()->save();

The call to Optimise::__construct() misses some required arguments starting with $schedule.

        $this->info('Optimisation '. $company->id.' completed');
    }
}