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

ReflectionClass getMethods() and getProperties() do not filter #103

Closed
asgrim opened this issue Aug 17, 2015 · 1 comment
Closed

ReflectionClass getMethods() and getProperties() do not filter #103

asgrim opened this issue Aug 17, 2015 · 1 comment
Assignees
@Ocramius
Labels
bug easypick reflection compatibility
Milestone
2.0.0

Comments

@asgrim
Copy link
Member

asgrim commented Aug 17, 2015

http://php.net/manual/en/reflectionclass.getmethods.php
http://php.net/manual/en/reflectionclass.getproperties.php

The getMethods() and getProperties() both do not actually filter at all - they need to accept an integer parameter.

// Fetch all private methods
$classInfo->getMethods(ReflectionMethod::IS_PRIVATE);
@asgrim asgrim added this to the 1.0.0 milestone Aug 17, 2015
@asgrim asgrim changed the title ReflectionClass getMethods and getProperties do not filter ReflectionClass getMethods() and getProperties() do not filter Aug 17, 2015
@asgrim asgrim removed this from the 1.0.0 milestone Aug 26, 2015
@asgrim asgrim added this to the 1.0.0 milestone Nov 2, 2015
@asgrim asgrim removed this from the 1.0.0 milestone Dec 21, 2015
@asgrim asgrim added the easypick label Feb 4, 2016
@marcosh marcosh mentioned this issue May 17, 2017
Merged
@Ocramius Ocramius added this to the 2.0.0 milestone May 18, 2017
@Ocramius Ocramius self-assigned this May 18, 2017
@Ocramius
Copy link
Member

Handled in #275

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

@Ocramius Ocramius

Labels
bug easypick reflection compatibility
Projects
None yet
Milestone
2.0.0
Development

No branches or pull requests
2 participants
@Ocramius @asgrim