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.

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

Fix of UnusedPrivateMethod chained method calls #151 #241

Merged
merged 1 commit into from Sep 21, 2015
Merged

Fix of UnusedPrivateMethod chained method calls #151 #241

merged 1 commit into from Sep 21, 2015

Conversation

sas05
Copy link

@sas05 sas05 commented Jan 8, 2015

No description provided.

@sas05
Copy link
Author

sas05 commented Jan 8, 2015

if ($owner->isInstanceOf('MethodPostfix')) {
            $owner = $owner->getParent()->getParent()->getChild(0);
}

This code assume that there would be any chain method call which more than 2 level. it will only work for 2 level chain method call. ex

$this->bar()->baz();

@manuelpichler manuelpichler added this to the 2.3.0 milestone Sep 21, 2015
@manuelpichler manuelpichler self-assigned this Sep 21, 2015
manuelpichler pushed a commit that referenced this pull request Sep 21, 2015
Fix of UnusedPrivateMethod chained method calls #151
@manuelpichler manuelpichler merged commit f5da8fa into phpmd:master Sep 21, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

2 participants