App\Http\Controllers\Gedcom\Store

Complexity

Total Complexity

2

Size/Duplication

Total Lines	24
Duplicated Lines	0 %

Importance

Changes	1
Bugs	1	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	__invoke()	0	22	2

<?php

namespace App\Http\Controllers\Gedcom;

use App\Http\Controllers\Controller;
use App\Jobs\ImportGedcom;
use App\Tenant\Manager;
use Illuminate\Http\Request;
use Illuminate\Http\Response;

class Store extends Controller
{
    public function __invoke(Request $request): Response
    {
        if ($request->file('file')->getClientOriginalExtension() !== 'ged') {
            return response([
                'msg' => 'file type invalid',
            ], 422);
        }

        $request->validate([
            'slug' => 'required|string',
            'file' => 'required',
        ]);
        $slug = $request->slug;
        $file = $request->file('file');
        \Log::info(json_encode($request->user()->company(), JSON_THROW_ON_ERROR));
        $manager = Manager::fromModel($request->user()->company(), $request->user());
        $path = $manager->storage()->putFile('imports', $file);

        ImportGedcom::dispatch($request->user(), $manager->storagePath($path), $slug);

It seems like $path can also be of type false; however, parameter $path of App\Tenant\Manager::storagePath() does only seem to accept string, maybe add an additional type check?

        return response([
            'message' => 'Gedcom Import Dispatched',
        ]);
    }
}