Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Use Reflection before invoking the method #48

Open
railken opened this issue Mar 19, 2019 · 1 comment
Open

Use Reflection before invoking the method #48

railken opened this issue Mar 19, 2019 · 1 comment
Labels
good first issue Good for newcomers

Comments

@railken
Copy link

railken commented Mar 19, 2019 •
edited
Loading

Invoking directly the method code can lead to unexpected code execution. Before executing the method a great filter could be the use of ReflectionFunctionAbstract::getReturnType. If not mandatory i think it sould be at least configurable.

Anyway, great package!
@railken railken changed the title Use Reflection instead of invoking the method Use Reflection before invoking the method Mar 20, 2019
@mpociot
Copy link
Member

mpociot commented Sep 4, 2019

Good idea, can you create a PR for this?

@mechelon mechelon added the good first issue Good for newcomers label Mar 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@mpociot @mechelon @railken