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 8.3] Polyfill new non-overloaded variants of functions #437

Merged
merged 1 commit into from
Aug 2, 2023

Conversation

IonBazan
Copy link
Contributor

@IonBazan IonBazan commented Jul 28, 2023

This PR aims to add new PHP 8.3 functions introduced with RFC: https://wiki.php.net/rfc/deprecate_functions_with_overloaded_signatures

Comments and feedback welcome 🙏🏻

  • array_keys_filter (rejected)
  • ldap_connect_wallet
  • ldap_exop_sync
  • stream_context_set_options

Fixes #436

@IonBazan IonBazan force-pushed the feature/no-overload branch 4 times, most recently from 95c0916 to 7ad2dc3 Compare July 31, 2023 07:59
@IonBazan IonBazan marked this pull request as ready for review July 31, 2023 08:03
src/Php83/bootstrap.php Outdated Show resolved Hide resolved
src/Php83/bootstrap81.php Outdated Show resolved Hide resolved
@nicolas-grekas
Copy link
Member

Can you also please update the changelog?

@IonBazan
Copy link
Contributor Author

IonBazan commented Aug 1, 2023

Updated, @nicolas-grekas but I noticed that changelog mentions v1.28 too which hasn't been released 🤔. Is this expected?

@nicolas-grekas
Copy link
Member

Thank you @IonBazan.

@nicolas-grekas nicolas-grekas merged commit 2b18ad0 into symfony:main Aug 2, 2023
4 checks passed
@nicolas-grekas
Copy link
Member

I noticed that changelog mentions v1.28 too which hasn't been released thinking. Is this expected?

Good catch, fixed in e204dfd

Anything else we should add to 1.28 before releasing it?

@IonBazan IonBazan deleted the feature/no-overload branch August 2, 2023 10:13
@IonBazan
Copy link
Contributor Author

IonBazan commented Aug 2, 2023

I think we're good. Maybe you can try to test it in a real-life application? I didn't have a chance to test the LDAP-related functionality 🙈

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

Successfully merging this pull request may close these issues.

Missing polyfills for PHP 8.3
2 participants