-
Notifications
You must be signed in to change notification settings - Fork 38.3k
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
NamedParameterUtils has broken square brackets handling #27925
Labels
in: data
Issues in data modules (jdbc, orm, oxm, tx)
type: regression
A bug that is also a regression
Milestone
Comments
spring-projects-issues
added
the
status: waiting-for-triage
An issue we've not yet triaged or decided on
label
Jan 12, 2022
sbrannen
added
in: data
Issues in data modules (jdbc, orm, oxm, tx)
type: regression
A bug that is also a regression
labels
Jan 12, 2022
parameterName is changed from |
jhoeller
removed
the
status: waiting-for-triage
An issue we've not yet triaged or decided on
label
Jan 18, 2022
Maybe consider #17773 as well for potential improvement to the parsing algorithm. |
With some custom treatment, we can restore the original intent of #27716 as well as preserve square brackets for index/key expressions. Finally making it into RC2... |
jhoeller
added a commit
that referenced
this issue
Oct 18, 2022
jhoeller
added a commit
that referenced
this issue
Oct 19, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
in: data
Issues in data modules (jdbc, orm, oxm, tx)
type: regression
A bug that is also a regression
This is a simple unit test which passes with the
5.3.x
but fails with the current6.0
milestones:The point is that we may have a reference for the value from nested
Map
property.The typical use case is messaging, for example with Spring Integration, where a
Message<?>
in the request is used as a source for parameter values.We even advertise this in the docs: https://docs.spring.io/spring-integration/docs/current/reference/html/jdbc.html#jdbc-outbound-channel-adapter.
I suspect that the cause for breaking change is this fix: 64b6bee. Or the previous one: 86eda27.
Thanks
The text was updated successfully, but these errors were encountered: