LaravelFreelancerNL\Aranguent\Eloquent\Relations\Concerns\InteractsWithPivotTable

Complexity

Total Complexity

1

Size/Duplication

Total Lines	24
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	formatAttachRecord()	0	13	1

<?php

declare(strict_types=1);

namespace LaravelFreelancerNL\Aranguent\Eloquent\Relations\Concerns;

trait InteractsWithPivotTable
{
    /**
     * Create a full attachment record payload.
     *
     * @param  int  $key
     * @param  mixed  $value
     * @param  array  $attributes
     * @param  bool  $hasTimestamps
     * @return array
     */
    protected function formatAttachRecord($key, $value, $attributes, $hasTimestamps)
    {
        [$id, $attributes] = $this->extractAttachIdAndAttributes($key, $value, $attributes);

        /**
         * When attaching multiple models with pivot data PHP will cast a numeric-string array key to an integer
         * By setting the type here we recast it to the keyType set on the model.
         */
        settype($id, $this->related->getKeyType());

        return array_merge(
            $this->baseAttachRecord($id, $hasTimestamps),

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

            $this->castAttributes($attributes)

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

        );
    }
}