agoalofalife\Reports\Console\HandleReportCommand::handle()

Complexity

Conditions	4
Paths	4

Size

Total Lines

27

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	15
CRAP Score	4

Importance

Changes

0

<?php
    public function handle() : void
    {
        $classReport = $this->argument('classReport');

        if (class_exists($classReport) === false) {
             throw new \Exception($classReport . ' class not exist');
        }

        if (is_subclass_of($classReport, Report::class) === false) {

Due to PHP Bug #53727, is_subclass_of might return inconsistent results on some PHP versions if \agoalofalife\Reports\Report::class can be an interface. If so, you could instead use ReflectionClass::implementsInterface.

            throw new \Exception($classReport . ' class is not subclass ' . Report::class);
        }

        $report = app()->make($classReport);
        $excelWriter = Excel::create($report->getFileName(), $resultHandler = [$report, 'handler']);

The method create() does not exist on Maatwebsite\Excel\Facades\Excel. Did you maybe mean createFreshMockInstance()?

        // if method handler return true, meaning 'success' result
        if ($resultHandler) {

The expression $resultHandler of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            $excelWriter->store($report->extension, Storage::disk($report->disk)
                ->getDriver()->getAdapter()->getPathPrefix());

            $report->getReportModel()->update([
                'status' => ReportModel::STATUS_COMPLETED,
                'is_completed' => true,
            ]);
            // send notification
            $report->send();
        }
    }
}