App\Service\enso\Localisation\Storer

Complexity

Total Complexity

2

Size/Duplication

Total Lines	21
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	create()	0	10	1
A	__construct()	0	4	1

<?php

namespace App\Service\enso\Localisation;

use Illuminate\Support\Facades\DB;
use App\Models\enso\Localisation\Language;
use App\Service\enso\Localisation\Json\Storer as JsonStorer;
use LaravelEnso\Localisation\Services\Legacy\Storer as LegacyStorer;

class Storer
{
    private array $request;
    private ?string $flagSuffix;

    public function __construct(array $request, $flagSuffix)
    {
        $this->request = $request;
        $this->flagSuffix = $flagSuffix;
    }

    public function create()
    {
        return DB::transaction(function () {
            $language = (new Language())
                ->storeWithFlagSufix($this->request, $this->flagSuffix);

It seems like $this->flagSuffix can also be of type null; however, parameter $sufix of App\Models\enso\Localisation\Language::storeWithFlagSufix() does only seem to accept string, maybe add an additional type check?

            (new LegacyStorer($this->request['name']))->create();
            (new JsonStorer($this->request['name']))->create();

            return $language;
        });
    }
}