Skip to content

[Php80] Skip property hook with indirect set on ClassPropertyAssignToConstructorPromotionRector#7887

Merged
TomasVotruba merged 2 commits intomainfrom
skip-prop-hook-multi
Feb 10, 2026
Merged

[Php80] Skip property hook with indirect set on ClassPropertyAssignToConstructorPromotionRector#7887
TomasVotruba merged 2 commits intomainfrom
skip-prop-hook-multi

Conversation

@samsonasik
Copy link
Member

@samsonasik
Copy link
Member Author

Fixed 🎉 /cc @rensPols

@samsonasik
Copy link
Member Author

@TomasVotruba ready 👍

@rensPols
Copy link

Fixed 🎉 /cc @rensPols

@samsonasik That's great to hear!

When do you think the next release with this fix will be?

@TomasVotruba TomasVotruba merged commit 57b37ca into main Feb 10, 2026
59 checks passed
@TomasVotruba TomasVotruba deleted the skip-prop-hook-multi branch February 10, 2026 10:44
@TomasVotruba
Copy link
Member

Thank you @samsonasik 👍

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.

ClassPropertyAssignToConstructorPromotionRector doesn't check dependencies in setter hooks. Resulting in a UninitializedPropertyError

3 participants