spatie / laravel-link-checker

src/CheckLinksCommand.php

<?php

namespace Spatie\LinkChecker;

use Exception;
use Illuminate\Console\Command;
use Spatie\Crawler\Crawler;
use Spatie\Crawler\CrawlObserver;
use Spatie\Crawler\CrawlProfile;

class CheckLinksCommand extends Command
{
    /**
     * The console command name.
     *
     * @var string
     */
    protected $signature = 'link-checker:run
                        {--url= : the url to be crawler}
                        {--profile= : The profiler to be used}
                        {--reporter= : The reporter to be used}
                        {--concurrency=10 : The amount of concurrent requests}
                        ';
    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Check all links';

    public function handle()
    {
        Crawler::create()
            ->setCrawlProfile($this->getProfile())
            ->setCrawlObserver($this->getReporter())
            ->startCrawling($this->getUrlToBeCrawled());

        $this->info('All done!');
    }

    /**
     * Determine the url to be crawled.
     *
     * @return null|string
     *
     * @throws \Exception
     */
    protected function getUrlToBeCrawled()
    {
        if (!is_null($this->option('url'))) {
            return $this->option('url');
        }

        if (config('laravel-link-checker.url') != '') {
            return config('laravel-link-checker.url');
        }

        if (config('app.url') != '') {
            return config('app.url');
        }

        throw new Exception('could not determine which url to be crawled.');
    }

    /**
     * Get the profile.
     */
    protected function getProfile(): CrawlProfile
    {
        if (!is_null($this->option('profile'))) {
            return app($this->option('profile'));

It seems like $this->option('profile') targeting Illuminate\Console\Command::option() can also be of type array; however, app() does only seem to accept string|null, maybe add an additional type check?

        }

        if (config('laravel-link-checker.default_profile') != '') {
            return app(config('laravel-link-checker.default_profile'));
        }

        throw new Exception('Could not determine the profile to be used');
    }

    /**
     * Get the reporter.
     */
    protected function getReporter(): CrawlObserver
    {
        if (!is_null($this->option('reporter'))) {
            return app($this->option('reporter'));

It seems like $this->option('reporter') targeting Illuminate\Console\Command::option() can also be of type array; however, app() does only seem to accept string|null, maybe add an additional type check?

        }

        if (config('laravel-link-checker.default_reporter') != '') {
            return app(config('laravel-link-checker.default_reporter'));
        }

        throw new Exception('Could not reporter the profile to be used');
    }
}