Thanks. I guess this needs a bit more rethinking, because of undeclared variable references.

Thanks, I was already afraid that removing the ampersand before the function name would be too easy and probably not be the definitive solution :-)

I have tested this item ✅ successfully on c5d5b8e

Tested successfully in Joomla 4.2.0-alpha3-dev of 27 April in Wampserver 3.2.8 using PHP 8.0.15.

I have tested this item ✅ successfully on c5d5b8e

Thanks, I was already afraid that removing the ampersand before the function name would be too easy and probably not be the definitive solution :-)

I guess removing the ampersand is fine, but we need to adapt the unit test

Can you replace the unit test function with this more real world scenario:

	public function testNotDeclaredVariable()
	{
		$model = new class(['dbo' => $this->createStub(DatabaseInterface::class)], $this->createStub(MVCFactoryInterface::class)) extends BaseDatabaseModel
		{
			public function cache($key, $value)
			{
				if (!isset($this->test[$key]))
				{
					$this->test[$key] = $value;
				}

				return $this->test[$key];
			}
		};

		$this->assertEquals('test', $model->cache(1, 'test'));
	}

I've added your unit test to my PR. Thanks @laoneo !

Thanks!