-
-
Notifications
You must be signed in to change notification settings - Fork 344
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
Add ExceptionList property for LongParameterList #559
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@andrewwheal thank you for your enhancement.
Please add some documentation. This way, other's will find this nice enhancement, too.
@@ -42,6 +42,12 @@ public function apply(AbstractNode $node) | |||
return; | |||
} | |||
|
|||
$exceptions = $this->getExceptionsList(); | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove white line to move these two related code parts together. As they belong toghether and do a special thing, they can be extracted into a separate protected method.
For example:
if ($this->parameterIsInExceptionList()) {
return;
}
@andrewwheal are you still intersted in pursuing this? |
Remove the empty line as requested by @ravage84
I took inspiration/code from the ShortVariable rule which already had an exception list property and added it to the LongParameterList rule.