LaravelFreelancerNL\Aranguent\Eloquent\Relations\Concerns\HasOneOrMany

Complexity

Total Complexity

4

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	firstOrNew()	0	11	2
A	firstOrCreate()	0	9	2

<?php

declare(strict_types=1);

namespace LaravelFreelancerNL\Aranguent\Eloquent\Relations\Concerns;

trait HasOneOrMany
{
    /**
     * Get the first related record matching the attributes or create it.
     *
     * @param  array  $attributes
     * @param  array  $values
     * @return \Illuminate\Database\Eloquent\Model
     */
    public function firstOrCreate(array $attributes = [], array $values = [])
    {
        /** @phpstan-ignore-next-line */
        $instance = $this->where(associativeFlatten($attributes))->first();

It seems like where() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        if (is_null($instance)) {
            $instance = $this->create(array_merge($attributes, $values));

It seems like create() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }

        return $instance;
    }

    /**
     * Get the first related model record matching the attributes or instantiate it.
     *
     * @param  array  $attributes
     * @param  array  $values
     * @return \Illuminate\Database\Eloquent\Model
     */
    public function firstOrNew(array $attributes = [], array $values = [])
    {
        /** @phpstan-ignore-next-line */
        $instance = $this->where(associativeFlatten($attributes))->first();
        if (is_null($instance)) {
            $instance = $this->related->newInstance(array_merge($attributes, $values));

            $this->setForeignAttributesForCreate($instance);

It seems like setForeignAttributesForCreate() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }

        return $instance;
    }
}