Longman\LaravelLodash\Eloquent\UsesUuidAsPrimary

Complexity

Total Complexity

9

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

0%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	isUuidBinary()	0	4	2
A	getIncrementing()	0	4	1
A	getKeyType()	0	4	1
A	bootUsesUuidAsPrimary()	0	20	5

<?php

declare(strict_types=1);

namespace Longman\LaravelLodash\Eloquent;

use Ramsey\Uuid\Uuid;

use function preg_match;

/**
 * @mixin \Illuminate\Database\Eloquent\Model
 */
trait UsesUuidAsPrimary
{
    public function isUuidBinary(string $value): bool
    {
        return isset($value[0]) && ! preg_match('//u', $value);
    }

    public function getIncrementing(): bool
    {
        return false;
    }

    public function getKeyType(): string
    {
        return 'string';
    }

    protected static function bootUsesUuidAsPrimary(): void
    {
        static::creating(static function (UuidAsPrimaryContract $model): void {
            $keyName = $model->getKeyName();

            if (! empty($model->{$keyName})) {
                // Do some validation
                $model->isUuidBinary($model->{$keyName})
                    ? Uuid::fromBytes($model->{$keyName})
                    : Uuid::fromString($model->{$keyName});
            } elseif (! empty($model->attributes[$keyName])) {

Accessing attributes on the interface Longman\LaravelLodash\Eloquent\UuidAsPrimaryContract suggest that you code against a concrete implementation. How about adding an instanceof check?

                // Do some validation
                $model->isUuidBinary($model->attributes[$keyName])

Accessing attributes on the interface Longman\LaravelLodash\Eloquent\UuidAsPrimaryContract suggest that you code against a concrete implementation. How about adding an instanceof check?

                    ? Uuid::fromBytes($model->attributes[$keyName])

Accessing attributes on the interface Longman\LaravelLodash\Eloquent\UuidAsPrimaryContract suggest that you code against a concrete implementation. How about adding an instanceof check?

                    : Uuid::fromString($model->attributes[$keyName]);

Accessing attributes on the interface Longman\LaravelLodash\Eloquent\UuidAsPrimaryContract suggest that you code against a concrete implementation. How about adding an instanceof check?

            } else {
                $model->{$keyName} = Uuid::uuid1()->getBytes();
            }
        });
    }
}