Skip to content

Propagated the SqlType to the parameter source during Spel ex… #2079

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

mipo256
Copy link
Contributor

@mipo256 mipo256 commented Jul 5, 2025

Closes #2007

… during Spel expression evaluation

Signed-off-by: mipo256 <[email protected]>
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jul 5, 2025
@mp911de mp911de added type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged labels Jul 7, 2025
@schauder schauder changed the title GH-2007 Propagated the SqlType to the parameter source during Spel ex… Propagated the SqlType to the parameter source during Spel ex… Jul 9, 2025
schauder pushed a commit that referenced this pull request Jul 10, 2025
Signed-off-by: mipo256 <[email protected]>

Commit message edited by Jens Schauder

Original pull request #2079
Closes #2007
schauder added a commit that referenced this pull request Jul 10, 2025
Fix Oracle test setup.
Formatting.
Restructure some code
Author tags.

Original pull request #2079
See #2007
@schauder
Copy link
Contributor

Thanks for the PR. That's reviewed and merged.

We noted that SpEL expressions should also run through the conversion machine. Maybe you are interested in tackling that as well? (#2082)

@schauder schauder closed this Jul 10, 2025
@schauder schauder added this to the 4.0 M4 (2025.1.0) milestone Jul 10, 2025
@mipo256
Copy link
Contributor Author

mipo256 commented Jul 10, 2025

Yeah, sure @schauder. I'll take a look at it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incorrect MapParameterSource construction in @Query methods with SPEL
4 participants