In NodeScopeResolver there are places that are interested in PassedByReference - @param-out should be handled there.

Also - require your version of phpdoc-parser with the support added in composer.json, so that the pipeline can actually get green.

Also, you have to guard any call to new method on PhpDocNode with method_exists so that Rector can do its downgrading job. See: #1799 (comment)

thanks for the tips.

I tried testing the PR with the following code

<?php
use function PHPStan\Testing\assertType;
class X {
	/**
	 * @param-out string $s
	 */
	function addFoo(?string &$s): void
	{
		if ($s === null) {
			$s = "hello";
		}
		$s .= "foo";
	}
}
function foo1(?string $s) {
	assertType('string|null', $s);
	$x = new X();
	$x->addFoo($s);
	assertType('string', $s);
}

but I currently struggle with php-doc resolving. 2 problems arise:

• (削除) in NodeScopeResolver->getParameterOutTypes(..) the phpdoc /** @param-out string $s */ gets always resolved to an empty-block from these lines (削除ここまで)

  phpstan-src/src/Type/FileTypeMapper.php

  Lines 111 to 113 in 49cd131

  if (!isset($this->inProcess[$fileName][$nameScopeKey])) { // wrong $fileName due to traits

  return ResolvedPhpDocBlock::createEmpty();

  }
• (削除) FunctionReflection does not yet contain a getDocComment method (like MethodReflection has)... it seems this is a missing part I need to implement with this PR? (削除ここまで)

I should write comments about my problems more often... sorry/thanks for rubberducking.. found the reason for doc-block resolving problem because I had mixed up caller and callee side of things - 48a2cf8.

I think the PR works now as expected for basic method examples. open todo is for regular functions. I would implement a getDocComment method on FunctionReflection now, to get it working... if we agree on this

(削除) for some reason the PR as is failling after rebase, because I am running into (削除ここまで) -> fixed by re-adding a recently added use-statement which got lost after manual merge conflict resolving

You updated a lot irrelevant deps. You need to run just composer update phpstan/phpdoc-parser instead.

And now you get relevant build errors finally 😊

I had a look into the psalm test-suite for some more advanced use-cases... I will push them as a followup after we got the basic support landed

fixed and rebased. thank you

Thank you!

btw: I opened a new issue on php-src, to add/discuss @param-out directly into php-src-stubs

php/php-src#9897

@staabm Could you describe and open another issue for flag constants added to the stubs too? Like all the valid constants for $flags on json_encode (https://www.php.net/manual/en/function.json-encode.php) could be there.

open another issue for flag constants added to the stubs too?

here we go: php/php-src#9900