Skip to content

[CodeQuality] Skip used by other property hooks on InlineConstructorDefaultToPropertyRector#7920

Open
samsonasik wants to merge 2 commits intomainfrom
skip-found-in-any-property-hooks
Open

[CodeQuality] Skip used by other property hooks on InlineConstructorDefaultToPropertyRector#7920
samsonasik wants to merge 2 commits intomainfrom
skip-found-in-any-property-hooks

Conversation

@samsonasik
Copy link
Member

@samsonasik
Copy link
Member Author

Fixed 🎉 /cc @rensPols

@samsonasik
Copy link
Member Author

@TomasVotruba ready 👍

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.

InlineConstructorDefaultToPropertyRector doesn't check if the variable is being used (in a setter) before the default value is changed.

1 participant