Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

php deprication notices in 8.1 #52

Closed
janw-me opened this issue Sep 26, 2022 · 7 comments
Closed

php deprication notices in 8.1 #52

janw-me opened this issue Sep 26, 2022 · 7 comments

Comments

@janw-me
Copy link
Contributor

janw-me commented Sep 26, 2022

When running php8.1 to run phpstan I get 24 Deprication notices.
I have not tested 8.0, these notices do not appear on 7.4

These can be fixed. And I'm up for that.
I've checked a couple and these fixed will cause errors when lower then running php7.0.

So would you be open for a PR with these fixes of 8+ notices at the cost of dropping support of 5.6 and lower.

To be clear, I'm not talking about the version you set in phpstan.neon.dist I'm talking about the php cli version.

@simonhammes
Copy link

Are you referring to these deprecation notices? #41

@janw-me
Copy link
Contributor Author

janw-me commented Sep 27, 2022

Yes

@janw-me janw-me closed this as completed Sep 27, 2022
@szepeviktor
Copy link
Member

I would gladly include a bunch of sed commands.

@janw-me
Copy link
Contributor Author

janw-me commented Sep 27, 2022

That's not a bad idea. I'll see if I can fix that this week.

@janw-me janw-me reopened this Sep 27, 2022
@szepeviktor
Copy link
Member

Like this one.
https://github.com/php-stubs/acf-pro-stubs/blob/7e44b653bd72e5eebf725f3af6b5afaae3039dcf/generate.sh#L30-L43

It is easier to work on WordPress source before stubs generation.

@szepeviktor
Copy link
Member

szepeviktor commented Sep 27, 2022

You find the Requests v1 package in wp-includes/Requests/

sed -i -e 's|^\s*public function unserialize(|#\t[\\ReturnTypeWillChange]\n&|' source/wordpress/wp-includes/Requests/Utility/FilteredIterator.php something ....

@janw-me
Copy link
Contributor Author

janw-me commented Sep 27, 2022

👍 Thanks I'll take a look.
I'll manage

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants