-
Notifications
You must be signed in to change notification settings - Fork 5
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
Quick possible refactor? #1
Comments
No because of use having to make private property members accessible. |
It's being done via a trait in the implementing class though, so all properties will be visible in scope to |
I've just tested this and it is correct, as long as the trait is used in the highest most implementing class. |
It sure does, TIL :D |
@johnrobertporter I've tested, Please feel free to contribute! |
A little later today :) |
Love this example code!
Looking at: https://github.com/lzakrzewski/prooph-saga-playground/blob/master/src/Messaging/ReturnsPayload.php
Would it not be simpler to use
get_object_vars($this);
instead of the massive overhead of reflection?The text was updated successfully, but these errors were encountered: