App\Console\Commands\BunqImport::handle()

Complexity

Conditions	10
Paths	13

Size

Total Lines	54
Code Lines	36

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function handle(): int
    {
        $access = $this->haveAccess();
        if (false === $access) {
            $this->error('Could not connect to your local Firefly III instance.');

            return 1;
        }

        $this->info(sprintf('Welcome to the Firefly III bunq importer, v%s', config('bunq.version')));
        app('log')->debug(sprintf('Now in %s', __METHOD__));
        $config = $this->argument('config');

        if (!file_exists($config) || (file_exists($config) && !is_file($config))) {

It seems like $config can also be of type string[]; however, parameter $filename of is_file() does only seem to accept string, maybe add an additional type check?

It seems like $config can also be of type string[]; however, parameter $filename of file_exists() does only seem to accept string, maybe add an additional type check?

            $message = sprintf('The importer can\'t import: configuration file "%s" does not exist or could not be read.', $config);

It seems like $config can also be of type string[]; however, parameter $args of sprintf() does only seem to accept string, maybe add an additional type check?

            $this->error($message);
            app('log')->error($message);

            return 1;
        }
        $jsonResult = $this->verifyJSON($config);

It seems like $config can also be of type null and string[]; however, parameter $file of App\Console\Commands\BunqImport::verifyJSON() does only seem to accept string, maybe add an additional type check?

        if (false === $jsonResult) {
            $message = 'The importer can\'t import: could not decode the JSON in the config file.';
            $this->error($message);

            return 1;
        }
        $configuration = json_decode(file_get_contents($config), true, 512, JSON_THROW_ON_ERROR);

It seems like $config can also be of type string[]; however, parameter $filename of file_get_contents() does only seem to accept string, maybe add an additional type check?

        $this->line('The import routine is about to start.');
        $this->line('This is invisible and may take quite some time.');
        $this->line('Once finished, you will see a list of errors, warnings and messages (if applicable).');
        $this->line('--------');
        $this->line('Running...');
        $result = $this->startDownload($configuration);
        if (0 === $result) {
            $this->line('Download from bunq complete.');
        }
        if (0 !== $result) {
            $this->warn('Download from bunq resulted in errors.');

            return $result;
        }
        $secondResult = $this->startSync($configuration);
        if (0 === $secondResult) {
            $this->line('Sync to Firefly III complete.');
        }
        if (0 !== $secondResult) {
            $this->warn('Sync to Firefly III resulted in errors.');

            return $secondResult;
        }

        return 0;
    }