Extend extbase QueryResult to fix count() of queries with a custom $statement

QueryResult.php

<?php
namespace AppZap\Tripshop\Persistence;

use TYPO3\CMS\Extbase\Persistence\Generic\Qom\Statement;

class QueryResult extends \TYPO3\CMS\Extbase\Persistence\Generic\QueryResult
{
    /**
     * Overwrites the original implementation of Extbase
     *
     * When the query contains a $statement the query is regularly executed and the number of results is counted
     * instead of the original implementation which tries to create a custom COUNT(*) query and delivers wrong results.
     *
     * @return int The number of matching objects
     */
    public function count()
    {
        if ($this->numberOfResults === null) {
            if (is_array($this->queryResult)) {
                $this->numberOfResults = count($this->queryResult);
            } else {
                $statement = $this->query->getStatement();
                if ($statement instanceof Statement) {
                    $this->initialize();
                    $this->numberOfResults = count($this->queryResult);
                } else {
                    return parent::count();
                }
            }
        }
        return $this->numberOfResults;
    }
}

In case you're wondering (as I did...): you'll also have to put some configuration into ext_typoscript_setup.txt to get this working:

config.tx_extbase {
  objects {
    TYPO3\CMS\Extbase\Persistence\QueryResultInterface {
      className = AppZap\Tripshop\Persistence\QueryResult
    }   
  }
}

BTW, great snippet! IMHO this should be in the core.

+1

Is this bug (it's a bug, right?) reported on forge? Couldn't find it there. This should at least be mentioned in the documentation.

Since march 2017 it's reported as a bug (#80464)