Skip to content

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

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

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

Conversation

@samsonasik

Copy link
Copy Markdown
Member

@samsonasik

Copy link
Copy Markdown
Member Author

Fixed 🎉 /cc @rensPols

@samsonasik

Copy link
Copy Markdown
Member Author

@TomasVotruba ready 👍

@rensPols

Copy link
Copy Markdown

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
Copy Markdown
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

Development

Successfully merging this pull request may close these issues.

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

3 participants