Doctrine\ORM\Internal\Hydration\IterableNewResult

Complexity

Total Complexity

8

Size/Duplication

Total Lines	74
Duplicated Lines	0 %

Test Coverage

Coverage

95.24%

Importance

Changes	2
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	current()	0	3	1
A	valid()	0	3	1
A	next()	0	10	2
A	__construct()	0	3	1
A	rewind()	0	8	2
A	key()	0	3	1

<?php

declare(strict_types=1);

namespace Doctrine\ORM\Internal\Hydration;

use Iterator;
use function array_key_exists;

Type array_key_exists is not used in this file.

use function count;

Type count is not used in this file.

use function is_array;

/**
 * TODO: temp name
 *
 * Represents a result structure that can be iterated over, hydrating row-by-row
 * during the iteration. An IterableResult is obtained by AbstractHydrator#getIterable().
 */
final class IterableNewResult implements Iterator
{
    /** @var AbstractHydrator */
    private $hydrator;

    /** @var bool */
    private $rewinded = false;

    /** @var int */
    private $key = -1;

    /** @var object|null */
    private $current;

    /**
     * @param AbstractHydrator $hydrator
     */
    public function __construct($hydrator)
    {
        $this->hydrator = $hydrator;
    }

    /**
     * @throws HydrationException
     */
    public function rewind()
    {
        if ($this->rewinded === true) {
            throw new HydrationException('Can only iterate a Result once.');
        }

        $this->current  = $this->next();

It seems like $this->next() of type false is incompatible with the declared type null|object of property $current.

        $this->rewinded = true;
    }

    /**
     * Gets the next set of results.
     *
     * @return mixed|false
     */
    public function next()
    {
        $this->current = $this->hydrator->hydrateRow();

It seems like $this->hydrator->hydrateRow() of type false is incompatible with the declared type null|object of property $current.

        if (is_array($this->current)) {

The condition is_array($this->current) is always false.

            $this->current = array_values($this->current)[0];

Function array_values() should not be referenced via a fallback global name, but via a use statement.

        }

        $this->key++;

        return $this->current;

The expression return $this->current returns the type false which is incompatible with the return type mandated by Iterator::next() of void.

    }

    /**
     * @return mixed
     */
    public function current()
    {
        return $this->current;
    }

    /**
     * @return int
     */
    public function key()
    {
        return $this->key;
    }

    /**
     * @return bool
     */
    public function valid()
    {
        return $this->current!==false;
    }
}