App\Http\Controllers\Gedcom\Store

Complexity

Total Complexity

4

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	__invoke()	0	27	4

<?php

namespace App\Http\Controllers\Gedcom;

use App\Event;
use App\Family;
use App\Http\Controllers\Controller;
use App\Jobs\ImportGedcom;
use App\Note;
use App\Person;
use App\Source;
use App\Traits\ConnectionTrait;
use Auth;
use Illuminate\Http\Request;
use Illuminate\Support\Facades\Artisan;
use LaravelEnso\Multitenancy\Enums\Connections;
use ModularSoftware\LaravelGedcom\Facades\GedcomParserFacade;
use ModularSoftware\LaravelGedcom\Utils\GedcomParser;

class Store extends Controller
{
    use ConnectionTrait;
    /*
    * Api end-point for Gedcom api/gedcom/store
    * Saving uploaded file to storage and starting to read
    */

    public function __invoke(Request $request)
    {
        $slug = $request->get('slug');
        if ($request->hasFile('file')) {
            if ($request->file('file')->isValid()) {
                try {
                    $conn = $this->getConnection();
                    $db = $this->getDB();
                    $currentUser = Auth::user();
                    $_name = uniqid().'.ged';
                    $request->file->storeAs('gedcom', $_name);
                    define('STDIN', fopen('php://stdin', 'r'));
                    // $parser = new GedcomParser();
                    // $parser->parse($request->file('file'), $slug, true);
                    $filename = 'app/gedcom/'.$_name;
                    ImportGedcom::dispatch($filename, $slug, $currentUser->id, $conn, $db);

Accessing id on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

                    // Artisan::call('queue:work');
                    return ['File uploaded'];
                } catch (Exception $e) {

The type App\Http\Controllers\Gedcom\Exception was not found. Did you mean Exception? If so, make sure to prefix the type with \.

                    return ['Not uploaded'];
                }
            }

            return ['File corrupted'];
        }

        return ['Not uploaded'];
    }
}