LaravelFreelancerNL\Aranguent\Eloquent\Relations\Concerns\InteractsWithPivotTable::formatAttachRecord()

Complexity

Conditions	1
Paths	1

Size

Total Lines	13
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	1

Importance

Changes	1
Bugs	0	Features	0

<?php
    protected function formatAttachRecord($key, $value, $attributes, $hasTimestamps)
    {
        [$id, $attributes] = $this->extractAttachIdAndAttributes($key, $value, $attributes);

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

        /**
         * 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?

        );