spring-projects / spring-projects/spring-data-rest
Configurable ResourceInformationHandlerMethodArgumentResolver [DATAREST-1011]
@odrotbohm is already working on this.
Since Dec 31, 2020.
- Dominant language
- Java
- Stars
- 958
- Forks
- 568
- PR merge metrics
- No merged PRs in 30d
Description
Joerg Bellmann opened DATAREST-1011 and commented
Hi, this is more a question for a better way than what we did with a small 'hack'.
Spring-Data-Rest does a lot of things out-of-the-box. And binding parameters to do a queryDsl-search is also cool and we would like to use it.
Unfortunately our company has some 'Rest-Guidelines' mentioning that parameters have to be 'snake_case' instead of 'camelCase'. The problem is, that the search has no mapping for that and will not work.
So I 'fixed' it in an ugly way and want to know how can we solve this issue in a better way.
The small showcase project is on github..
The ugly part starts here.
It would already be helpful if :
- QuerydslAwareRootResourceInformationHandlerMethodArgumentResolver is extendable
- the configuration that creates the RootResourceInformationHandlerMethodArgumentResolver as a bean could check for an existing bean already provided, maybe with '
@ConditionalOnMissingBean'
Affects: 2.6 GA (Ingalls)
1 votes, 2 watchers
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.