-
Notifications
You must be signed in to change notification settings - Fork 34
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
Refactor to include ReflectinoClass to support better inheritance model #6
base: master
Are you sure you want to change the base?
Refactor to include ReflectinoClass to support better inheritance model #6
Conversation
…ance model" This reverts commit d7a8a04.
@schmittjoh this is a WIP, I will now work on the JMSAopBundle If you have any comments, please go quick, I have a limited time window to contribute to this code! |
Maybe I'm missing something, but why do we need to duplicate so much code? :) |
* | ||
* @return array<MethodInterceptorInterface> | ||
*/ | ||
function loadInterceptors(\ReflectionClass $class, \ReflectionMethod $method); |
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.
This function's signature forces to create these codes
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.
also you don't want BC breaks
If you are not sure if it can be simplified more, just leave it as is, and I will take a look at it. |
you are welcome to simplify this code ;) but without changing method's signatures this is going to be hard. |
No description provided.