-
-
Notifications
You must be signed in to change notification settings - Fork 160
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
PublicVisibility mutator failing due to missing reflection #501
Labels
Comments
sanmai
added a commit
to sanmai/infection
that referenced
this issue
Oct 12, 2018
…reflection It seems that PHP-Parser can't provide reflection for anonymous classes yet. Fixes infection#501 for time being until nikic/PHP-Parser#543 gets fixed.
2 tasks
maks-rafalko
pushed a commit
that referenced
this issue
Oct 13, 2018
…reflection (#502) * Update ProtectedVisibility/PublicVisibility to guard against missing reflection It seems that PHP-Parser can't provide reflection for anonymous classes yet. Fixes #501 for time being until nikic/PHP-Parser#543 gets fixed. * PublicVisibility: test for presence of reflection in one place * Update type hint to include null
BackEndTea
pushed a commit
to BackEndTea/infection
that referenced
this issue
Oct 22, 2018
…reflection (infection#502) * Update ProtectedVisibility/PublicVisibility to guard against missing reflection It seems that PHP-Parser can't provide reflection for anonymous classes yet. Fixes infection#501 for time being until nikic/PHP-Parser#543 gets fixed. * PublicVisibility: test for presence of reflection in one place * Update type hint to include null
raphaelstolt
pushed a commit
to raphaelstolt/infection
that referenced
this issue
Nov 2, 2018
…reflection (infection#502) * Update ProtectedVisibility/PublicVisibility to guard against missing reflection It seems that PHP-Parser can't provide reflection for anonymous classes yet. Fixes infection#501 for time being until nikic/PHP-Parser#543 gets fixed. * PublicVisibility: test for presence of reflection in one place * Update type hint to include null
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Complete output
The issue seems to be caused by that following piece of code:
The text was updated successfully, but these errors were encountered: