PersistentProperty, EntityAccess, EntityReflector formalities
- Dominant language
- Groovy
- Stars
- 2.9k
- Forks
- 975
- Avg merge
- 1d 22h
- Merged PRs (30d)
- 92
Description
Formal issues in key interfaces.
- Interface org.grails.datastore.mapping.model.PersistentProperty has a misspelled method name. `getCapitilizedName` should be deprecated, clone to `getCapitalizedName` (corrected spelling).
- Interface org.grails.datastore.mapping.engine.EntityAccess has a superfluous method. Deprecate either of `getPropertyValue` or `getProperty`.
- org.grails.datastore.mapping.reflect.EntityReflector has a misspelled method name. `getPersitentEntity` should be deprecated, clone to `getPersistentEntity` (corrected spelling).
Contributor guide
Research direction
Start by inspecting PersistentProperty, EntityAccess, and EntityReflector, then search their method usages to understand the compatibility impact. Verify that the corrected method names are available, the redundant EntityAccess method is deprecated, and the existing project tests still pass.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- groovy
- Domain
- backend-api-design
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 35/100