Skip to content

DATAJPA-1497 Allow to add a special parameter type from a Spring Data JPA extension #305

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

Closed
wants to merge 1 commit into from

Conversation

reda-alaoui
Copy link
Contributor

@reda-alaoui reda-alaoui commented Jan 18, 2019

This change allows to add a special parameter type by overriding org.springframework.data.jpa.repository.query.JpaParameters.JpaParameter.isBindable()

@reda-alaoui
Copy link
Contributor Author

Hello?

@schauder
Copy link
Contributor

schauder commented Jun 4, 2019

At the very lest this needs some test, demonstrating how to use it.

@reda-alaoui reda-alaoui force-pushed the DATAJPA-1497 branch 3 times, most recently from eb367e5 to 6effa3b Compare August 16, 2019 16:53
@reda-alaoui
Copy link
Contributor Author

@schauder , the test you requested has been added.

@reda-alaoui
Copy link
Contributor Author

Ping :)

1 similar comment
@reda-alaoui
Copy link
Contributor Author

Ping :)

@schauder
Copy link
Contributor

See my comment on the jira ticket: https://jira.spring.io/browse/DATAJPA-1497

@schauder schauder closed this Oct 17, 2019
@schauder schauder reopened this Jan 13, 2020
schauder pushed a commit that referenced this pull request Jan 13, 2020
…ta JPA extension.

This change allows to add a special parameter type by overriding org.springframework.data.jpa.repository.query.JpaParameters.JpaParameter.isBindable().

A "special parameter type" is a type of parameter that does not get bound to the query via the normal mechanism but gets some special treatment like `Class` or `Pageable`).

Original pull request: #305.
schauder added a commit that referenced this pull request Jan 13, 2020
Improved naming and formatting.
Made the `QueryExtractor` an argument of the `JpaQueryMethodFactory` constructor instead of its method.

Original pull request: #305.
@schauder
Copy link
Contributor

Thanks for your work.
I polished this and merged it into master.

@schauder schauder closed this Jan 13, 2020
@reda-alaoui reda-alaoui deleted the DATAJPA-1497 branch January 13, 2020 16:56
schauder added a commit that referenced this pull request Jan 14, 2020
Moved JpaQueryMethodFactory creation into a separate class.
Removed http URL in copyright statement.
Simplified bean injection of JpaQueryMethodFactory.build

Originial pull request: #305.

Signed-off-by: Jens Schauder <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants