spring-projects / spring-projects/spring-data-rest

Configurable ResourceInformationHandlerMethodArgumentResolver [DATAREST-1011]

Open
#1,373 1 comment 0 reactions 1 assignee View on GitHub

@odrotbohm is already working on this.

Since Dec 31, 2020.

in: repository type: enhancement
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

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.