App\Jobs\ExportGedCom

Complexity

Total Complexity

3

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	handle()	0	19	2
A	__construct()	0	4	1

<?php

namespace App\Jobs;

use Illuminate\Bus\Queueable;
use Illuminate\Contracts\Queue\ShouldQueue;
use Illuminate\Foundation\Bus\Dispatchable;
use Illuminate\Queue\InteractsWithQueue;
use Illuminate\Queue\SerializesModels;
use Illuminate\Http\Request;
use ModularSoftware\LaravelGedcom\Utils\GedcomGenerator;

The type ModularSoftware\LaravelGedcom\Utils\GedcomGenerator was not found. Maybe you did not declare it correctly or list all dependencies?

use File;
use Auth;
class ExportGedCom implements ShouldQueue
{
    use Dispatchable, InteractsWithQueue, Queueable, SerializesModels;

The trait Illuminate\Queue\SerializesModels requires some properties which are not provided by App\Jobs\ExportGedCom: $id, $relations, $class, $keyBy

    protected $family_id;
    /**
     * Create a new job instance.
     *
     * @return void
     */
    public function __construct($family_id, Request $request)

The parameter $request is not used and could be removed.

    {
        //
        $this->family_id = $family_id;
    }

    /**
     * Execute the job.
     *
     * @return void
     */
    public function handle()
    {
        //
        $headers = [

The assignment to $headers is dead and can be removed.

            'Content-type' => 'text/txt',
        ];
        $p_id = 7;
        $f_id = $this->family_id;
        $up_nest = 3;
        $down_nest = 3;
        $writer = new GedcomGenerator($p_id, $f_id, $up_nest, $down_nest);
        $content = $writer->getGedcomPerson();
        // $user_id = Auth::user()->id;
        $ts = microtime(true);
        $file = env('APP_NAME').date('_Ymd_').$ts.'.GED';
        $destinationPath=public_path()."/upload/";
        if (!is_dir($destinationPath)) {  mkdir($destinationPath,0777,true);  }
        File::put($destinationPath.$file,$content);
        return 0;

The expression return 0 returns the type integer which is incompatible with the documented return type void.

    }
}