Cycle\ORM\Relation\Traits\HasSomeTrait::deleteChild()

Complexity

Conditions	5
Paths	3

Size

Total Lines	16
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	11
CRAP Score	5

Importance

Changes	1
Bugs	0	Features	0

<?php
    protected function deleteChild(Pool $pool, Tuple $tuple, object $child): Tuple
    {
        if ($this->isNullable()) {

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

            $rTuple = $pool->attachStore($child, false);
            $relName = $this->getTargetRelationName();

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

            // Related state
            $state = $rTuple->state;
            if (!$state->hasRelation($relName) || $state->getRelation($relName) === $tuple->entity) {
                foreach ($this->outerKeys as $outerKey) {
                    $state->register($outerKey, null);
                }
                $state->setRelation($relName, null);
            }
            return $rTuple;
        }
        return $pool->attachDelete($child, $this->isCascade());

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

    }