verify constants via array_key_exists

[voku]

issue: phpstan/phpstan#4174

I looked into ArrayKeyExistsFunctionTypeSpecifyingExtension, but it seems like I didn't fully understand the logic of the specifying classes. In this case we want to specify parameter 1 or/and 2, can we currently do that?

[voku]

@ondrejmirtes is there already a option to run multiple FunctionTypeSpecifyingExtension per function? I added a simple skip method, but I don't know if there is a better way.

[ondrejmirtes]

@voku Hi, I don't think it's necessary to support multiple FunctionTypeSpecifyingExtension for the same function. Everything can be written in a single extension.

[ondrejmirtes]

Please verify that this also fixes: https://phpstan.org/r/2561c442-f952-470d-a6a5-1bc7a74b99ad

[ondrejmirtes]

Or probably doesn't have to, maybe it's a totally different issue.

[voku]

@voku Hi, I don't think it's necessary to support multiple FunctionTypeSpecifyingExtension for the same function. Everything can be written in a single extension.

@ondrejmirtes I really tried it, but I failed. How can I combine different SpecifiedTypes for different parameters in one FunctionTypeSpecifyingExtension class? Can you please give me a hint?

[ondrejmirtes]

@voku There's unionWith method on SpecifiedTypes:

phpstan-src/src/Analyser/SpecifiedTypes.php

Lines 103 to 104 in 7340657

	/** @api */
	public function unionWith(SpecifiedTypes $other): self

[voku]

@voku There's unionWith method on SpecifiedTypes:

phpstan-src/src/Analyser/SpecifiedTypes.php

Lines 103 to 104 in 7340657

	/** @api */
	public function unionWith(SpecifiedTypes $other): self

@ondrejmirtes Thanks for the hint. I tried it, but it's not working as expected for different parameters?! But I re-think about it and for a simple check for array-shapes, we do not need multiple specified types at all: If we have array-shapes, then we do not need to specify the $array anymore, and we can specify the value of $key of array_key_exists($key, $array).

[voku]

@ondrejmirtes Do you have some extra tests in mind?

[voku]

@ondrejmirtes this week I had again a bug in my code, where this fix could prevent it.

Can you maybe give feedback here, thanks. 😊

[herndlm]

I did not read through all comments, but I think we need to be careful that this doesn't become another in_array in regard of impossiblechecktype adaptions needed. Preferably none should be needed I guess :/

[ondrejmirtes]

Yes, that's what I worry about here. ImpossibleCheckTypeHelper should get thinner over time, not thicker.

[voku]

Maybe we can only use the "array_key_exists"-extension improvement? 🤔

[ondrejmirtes]

Hi, I'm cleaning up old and stale PRs. Please send a new PR if you're still interested, thanks.

I haven't merged this because I don't particularly feel great about the changes here.