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
Free up the getDescription method to prevent it overriding db fields #785
base: master
Are you sure you want to change the base?
Free up the getDescription method to prevent it overriding db fields #785
Conversation
Unfortunately we can't break SemVer justified by the likelihood of someone (not) using the API, as much as I wish that were true. This will have to be a change targeted at |
This pull request hasn't had any activity for a while. Are you going to be doing further work on it, or would you prefer to close it now? |
Yes @emteknetnz, I'll rebase and retarget the PR. |
ffc01e7
to
9e69a02
Compare
@michalkleiner are you still interested in this one? I have no idea when the next major version of elemental will be released |
@emteknetnz can we merge as is since it now targets master? At least that won't get lost. I may then raise a separate PR deprecating the method. |
I'm OK with that, link through the deprecation PR and I'll merge this as is |
The
getDescription
method is potentially overriding a Description db field on any extending class.The method was used only to get an addition to getTypeNice to display in the gridfield, possibly translated.
Let me know if this needs to be targetted at any other branch as it might be seen as a BC issue, should someone use this public, though otherwise quite narrow-focused, method somewhere else.
Issue
getDescription
method fromBaseElement
#1106