Doctrine\ORM\Internal\Hydration\IterableResult

Complexity

Total Complexity

10

Size/Duplication

Total Lines	74
Duplicated Lines	0 %

Test Coverage

Coverage

85.71%

Importance

Changes

0

6 Methods

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

<?php

declare(strict_types=1);

namespace Doctrine\ORM\Internal\Hydration;

use Iterator;
use function array_key_exists;
use function count;
use function is_array;

/**
 * Represents a result structure that can be iterated over, hydrating row-by-row
 * during the iteration. An IterableResult is obtained by AbstractHydrator#iterate().
 */
class IterableResult 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) && array_key_exists(0, $this->current) && count($this->current) === 1) {

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

            $this->current = $this->current[0];
        }

        $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;
    }
}