hiqdev\hiart\ExtendedCollection::performOperation()

Complexity

Conditions	4
Paths	5

Size

Total Lines	18
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	1	Features	1

<?php
    protected function performOperation(string $command, array $data, array $queryOptions = []) : array
    {
        if ($this->isConsistent()) {
            return $this->first->batchQuery($command, $data, $queryOptions);
        }

        foreach ($data as $key => $record) {
            $records[$record['recordModel']->className()][$key] = $record;
        }

        $results = [];
        foreach ($records as $className => $d) {

The variable $records seems to be defined by a foreach iteration on line 59. Are you sure the iterator is never empty, otherwise this variable is not defined?

            $row = reset($d);

The assignment to $row is dead and can be removed.

            $model = $d['recordModel'];
            $results = array_merge($results, $model->batchQuery($command, $d, $queryOptions));
        }

        return $results;
    }