W2w\Laravel\Apie\Console\DumpOpenApiSpecCommand::handle()

Complexity

Conditions	3
Paths	4

Size

Total Lines	15
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

<?php
    public function handle(OpenApiSpecGenerator $generator)
    {
        $filename = $this->argument('filename');
        if (preg_match('/\.y[a]{0,1}ml$/i', $filename)) {

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

            $content = $this->createYamlContent($generator);
        } else {
            $content = $this->createJsonContent($generator);
        }
        if (@file_put_contents($this->argument('filename'), $content)) {

It seems like $this->argument('filename') can also be of type string[]; however, parameter $filename of file_put_contents() does only seem to accept string, maybe add an additional type check?

            $this->output->writeln('Created file "' . $this->argument('filename') . '" successfully!');

Are you sure $this->argument('filename') of type null|string|string[] can be used in concatenation?

        } else {
            $this->output->error('Could not write file "' . $this->argument('filename') . '"!');
            return false;
        }
        return true;
    }