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
extract clone duplication logic in exceptionlistener #8716
Conversation
You broke tests as you changed type of |
Good point. I am wondering whether adding Request in the use statements is right, as this kind of add a new depedency of the http kernel on the http foundation. Maybe this is already the case, so that is not a problem. |
Yep. Already the case:
|
/** | ||
* Clone the request for the exception. | ||
* | ||
* @param \Exception $exception |
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.
All phpdoc should in one line, the only exception is when description is really long.
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.
Thank you! There we go
Looks good to me. |
Thank you for the review ... fixed the codestyle issues. |
This PR was merged into the master branch. Discussion ---------- extract clone duplication logic in exceptionlistener Drupal needs potentially to adapt the exception listener to add more information from the original request. (see http://drupal.org/node/2057607) so what about extracing some of the logic into a new method. | Q | A | ------------- | --- | Bug fix? | no | New feature? | yes | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | | License | MIT Commits ------- 6af2801 extract method
Drupal needs potentially to adapt the exception listener to add more information from the original request. (see http://drupal.org/node/2057607) so what about extracing some of the logic into a new method.