Skip to content

Property from interface is abstract property#1478

Merged
Ocramius merged 1 commit intoRoave:6.53.xfrom
ondrejmirtes:fix-abstract-property
Jan 24, 2025
Merged

Property from interface is abstract property#1478
Ocramius merged 1 commit intoRoave:6.53.xfrom
ondrejmirtes:fix-abstract-property

Conversation

@ondrejmirtes
Copy link
Contributor

Days without bug in property hooks: 0

PHP proof: https://3v4l.org/B8iGK

I took the fix from ReflectionMethod because the logic is the same and it works properly there.

@ondrejmirtes
Copy link
Contributor Author

Kukulich couldn't do this in his skiing gloves so I took over this time 😂

@Ocramius
Copy link
Member

Days without bug in property hooks: 0

This new tech is keeping us employed (unpaid)!

@Ocramius Ocramius self-assigned this Jan 24, 2025
@Ocramius Ocramius merged commit c1326ba into Roave:6.53.x Jan 24, 2025
30 checks passed
@Ocramius
Copy link
Member

Thanks @ondrejmirtes!

@ondrejmirtes
Copy link
Contributor Author

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants