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

Fix GH-9420: Attributes on promoted properties (PHP 8.1) #9887

Open
wants to merge 1 commit into
base: PHP-8.1
Choose a base branch
from

Conversation

kooldev
Copy link
Contributor

@kooldev kooldev commented Nov 3, 2022

Backported #9661 (fixes #9420) to PHP 8.1 branch.

@iluuu1994
Copy link
Member

#[SensitiveParameter] is only available in PHP 8.2, do we really need this for PHP 8.1?

@kooldev
Copy link
Contributor Author

kooldev commented Nov 3, 2022

@iluuu1994 You are right but some extension might support PHP 8.1 and provide internal attributes. I think that backporting the fix makes sense in this regard since it clearly fixes an existing bug in a maintained version of PHP.

Plus it was basically just copy-paste. 😉

@iluuu1994
Copy link
Member

In that case, let's see what the PHP 8.1 release managers @krakjoe, @ramsey and @patrickallaert think 🙂

@ramsey
Copy link
Member

ramsey commented Dec 20, 2022

I think this would be good to have in 8.1. @patrickallaert, what do you think?

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.

None yet

3 participants