App\Utils\SourceCreator

Complexity

Total Complexity

3

Size/Duplication

Total Lines	23
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	generateKey()	0	8	2
A	create()	0	9	1

<?php

namespace App\Utils;

use App\Models\Source;
use Illuminate\Support\Str;

class SourceCreator
{
    const SOURCE_KEY_MAXLENGTH = 6;

    public static function create(array $data): Source
    {
        $source = new Source();
        $source->name = $data['name'];
        $source->recruitment_id = $data['recruitment_id'];

The property recruitment_id does not exist on App\Models\Source. Did you mean recruitment?

        $source->key = self::generateKey();
        $source->save();

        return Source::find($source->id);

The expression return App\Models\Source::find($source->id) could return the type null which is incompatible with the type-hinted return App\Models\Source. Consider adding an additional type-check to rule them out.

    }

    private static function generateKey()
    {
        do {
            $key = strtoupper(Str::random(self::SOURCE_KEY_MAXLENGTH));

The method random() does not exist on Illuminate\Support\Str.

            $sources = Source::where('key', $key)->count();
        } while ($sources > 0);

        return $key;
    }
}