Don't support changes in strip_tags from php 7.4

[lompi]

From php 7.4 as second argument is possible to use array- psalm see that as error.

https://psalm.dev/r/6dca71d4dc

[psalm-github-bot]

I found these snippets:

https://psalm.dev/r/6dca71d4dc

<?php

echo strip_tags('<h1>header</h1>', ['<h1>']);

Psalm output (using commit bb7d713):

ERROR: InvalidArgument - 3:36 - Argument 2 of strip_tags expects null|string, array{string(<h1>)} provided

[orklah]

@weirdan strip_tags is present in CoreGenericsFunctions.phpstub . Is there a way to tell psalm the second parameter is an array only from PHP 7.4 in there? Maybe the CallMap_74_Delta is enough? It's still blury for me how Psalm behaves when a function has more than one description

[weirdan]

That's a very good question. I don't know.

[othercorey]

If this is CoreGenericFunctions, does adding it to a versioned stub override this file or is this file assumed to be latest?

[orklah]

I'm not really sure, we may need to test that

[jawira]

Hi, I'm still having this false-positive.

Argument 2 of strip_tags expects null|string, but array<array-key, string> provided (see https://psalm.dev/004)

PHP 8.0.28
Psalm 5.13.1@086b94371304750d1c673315321a55d15fc59015

There's something I can do to fix this ? Thanks.

[weirdan]

Versioned stubs do indeed override things in CoreGenericFunctions.