App\Http\Controllers\Gedcom\Store::__invoke()

Complexity

Conditions	2
Paths	2

Size

Total Lines	22
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	1	Features	0

<?php
    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',
        ]);