-
-
Notifications
You must be signed in to change notification settings - Fork 190
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
Admit no prefix as getter #289
Conversation
As an alternative to reflection or closure hacking to properly get the private property whenever not using `getXXX()` `isXXX()` accessors, just added blank prefix '' to call DDD oriented notation (ex. `id()` ) Is adverted to who want to use this in a future, that if the method returns a valueObject, to use ::CONTAINS operator that casts to string, although the VO must be stringable
Interesting addition prevent classic getter names. Can you please also extend the tests for this naming? |
Please fix the issues listed in the coding standards results of the build. |
Looks great. Please adapt your 2 last commit message as described in our contribution guideline. https://www.doctrine-project.org/contribute/index.html#working-on-topic-branches |
Added a test for using blank accessor ('id' instead 'getId')
2b04a75
to
597892c
Compare
Do we consider this to be a bugfix? Otherwise, we should target 1.7.x, shouldn't we? |
This is considered a feature and needs to be merged into the next unreleased version branch
You're right. This is a feature. |
As an alternative to reflection or closure hacking to properly get the private property whenever not using
getXXX()
isXXX()
accessors,just added blank prefix '' to call DDD oriented notation (ex.
id()
)Is adverted to who want to use this in a future, that if the method returns a valueObject, to use ::CONTAINS operator that casts to string, although the VO must be stringable