Typo3 extbase @param int无效

Question

I got this function in my ProductsRepository:

/**
 * @param int $ProductId
 * @return \Foo\Bar\Domain\Model\Products
 */
public function getProductById(int $ProductId) {
    $query = $this->createQuery();
    $query->matching(
        $query->equals('productId', $ProductId)
    );
    $query->setLimit(1);
    return $query->execute()->getFirst();
}

which I call like this:

$product = $this->productsRepository->getProductById($ProductId);

In Typo3 6.2.6 with PHP 5.6.3 this worked fine, but in Typo3 6.2.12 with PHP 5.6.12 it says the following in my error.log:

[Wed Sep 09 15:59:32.922153 2015] [:error] [pid 26601] [client 192.168.113.4:58686] PHP Catchable fatal error: Argument 1 passed to Foo\\Bar\\Domain\\Repository\\ProductsRepository::getProductById() must be an instance of Foo\\Bar\\Domain\\Repository\\int, integer given

Do you know why my repository wants Foo\\Bar\\Domain\\Repository\\int instead of int ?

Answer 1

As mentioned in comments by Jost and Viktor, this is not a TYPO3 error message, but a PHP error message. PHP does not support type hints for primitive types like int , bool and the like (it will, as of PHP 7 ).

What this code actually does : By using int as type hint, PHP assumes that you need an instance of a class int as an argument (note that this behaviour will change in PHP 7). Because no namespace is specified, PHP assumes this int class to be in your current namespace Foo\\Bar\\Domain\\Repository . This means when you call this method using $this->productRepository->getProductById(5) , you're passing an integer value ( 5 ) as parameter, where PHP expects an instance of the (probably non-existent) class Foo\\Bar\\Domain\\Repository\\int . This then triggers the "Argument 1 passed to ... must be an instance of ..., integer given" error.

To summarize: Using scalar type hints has never worked and will never work in PHP 5 (it will, in PHP 7). If it did appear to work at some point, chances are that it did not really work, but no error was triggered due to a bug in PHP that might have been fixed between versions.

Going back to TYPO3: In specific places, you can use scalar type hints in annotations. However, this is only possible in places where request parameters are mapped on method arguments (most notably controller actions in which you can use a @param annotation and domain entity attributes where you can use a @var annotation. This is a framework feature though, and will not work on regular method calls like $this->productRepository->getProductById(5) .

Answer 2

Try @param \\integer instead an remove the type hinting in the arguments. However, the type \\integer is what the Extension Builder generates by default in model classes.

I recommend using only the PHPDoc annotations instead of PHPs Type Hinting. Extbase validates the type where it is needed and you get in less trouble.