Spatie\EventProjector\Console\Snapshots\CreateSnapshotCommand

Complexity

Total Complexity

3

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
A	handle()	0	14	2

<?php

namespace Spatie\EventProjector\Console\Snapshots;

use Illuminate\Console\Command;
use Spatie\EventProjector\EventProjectionist;
use Spatie\EventProjector\Snapshots\SnapshotFactory;
use Spatie\EventProjector\Snapshots\Snapshotter;

class CreateSnapshotCommand extends Command
{
    protected $signature = 'event-projector:create-snapshot {projectorName} {--name}';

    protected $description = 'Create new snapshots';

    /** @var \Spatie\EventProjector\EventProjectionist */
    protected $eventProjectionist;

    /** @var \Spatie\EventProjector\Snapshots\SnapshotFactory */
    protected $snapshotFactory;

    public function __construct(EventProjectionist $eventProjectionist, SnapshotFactory $snapshotFactory)
    {
        parent::__construct();

        $this->eventProjectionist = $eventProjectionist;

        $this->snapshotFactory = $snapshotFactory;
    }

    public function handle()
    {
        $projectorName = $this->argument('projectorName');

        $projector = $this->eventProjectionist->getProjector($projectorName);

It seems like $projectorName defined by $this->argument('projectorName') on line 33 can also be of type array; however, Spatie\EventProjector\EventProjectionist::getProjector() does only seem to accept string, maybe add an additional type check?

        if (! $projector) {
            $this->warn("No projector named `{$projectorName}` found!");

            return;
        }

        $this->snapshotFactory->createForProjector($projector, $this->option('name'));

It seems like $this->option('name') targeting Illuminate\Console\Command::option() can also be of type array; however, Spatie\EventProjector\Snapshots\SnapshotFactory::createForProjector() does only seem to accept string, maybe add an additional type check?

    }
}