Skip to content
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

Documentation: Changing parameter to ordinal #18961

Merged
merged 2 commits into from
Aug 5, 2021
Merged

Documentation: Changing parameter to ordinal #18961

merged 2 commits into from
Aug 5, 2021

Conversation

georgleber
Copy link
Contributor

Using named parameter in find() leads to: java.lang.IllegalArgumentException: Could not locate ordinal parameter [1], expecting one of []

Using named parameter in find() leads to: java.lang.IllegalArgumentException: Could not locate ordinal parameter [1], expecting one of []
@gsmet
Copy link
Member

gsmet commented Jul 23, 2021

Hmmm, I would expect it to work and if not it would be a bug. Could you share a project reproducing the issue? Thanks!

@georgleber
Copy link
Contributor Author

Hi Guillaume, you can find an example here (the testcase is showing the error):
https://github.com/GeorgHenkel/quarkus-bug-namedquery-param

Copy link
Contributor

@gastaldi gastaldi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That makes sense.
@gsmet how would you expect it to work if there isn't a parameter named name in the find() method?

@gsmet gsmet merged commit 5ede9b3 into quarkusio:main Aug 5, 2021
@quarkus-bot quarkus-bot bot added this to the 2.2 - main milestone Aug 5, 2021
@gsmet
Copy link
Member

gsmet commented Aug 5, 2021

Good catch, thanks!

@gsmet gsmet modified the milestones: 2.2 - main, 2.1.2.Final Aug 9, 2021
@georgleber georgleber deleted the GeorgHenkel-patch-1 branch October 19, 2021 06:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants