Skip to content

LikeParameterBinding should unwrap value if necessary#2585

Closed
quaff wants to merge 1 commit into
spring-projects:2.7.xfrom
quaff:patch-9
Closed

LikeParameterBinding should unwrap value if necessary#2585
quaff wants to merge 1 commit into
spring-projects:2.7.xfrom
quaff:patch-9

Conversation

@quaff

@quaff quaff commented Jun 30, 2022

Copy link
Copy Markdown
Contributor

Fix GH-2548 which introduced by GH-2461

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jun 30, 2022
gregturn added a commit that referenced this pull request Jul 13, 2022
Null values are wrapped with a special handler when interacting with Hibernate. However, this becomes an issue for queries when LIKE or CONTAINS are applied. In this situation, the null needs to be condensed into an empty string and any wildcards can then be applied with expected results.

Closes #2548, #2570.
Supercedes: #2585.
Related: #2461, #2544#
@gregturn gregturn added status: superseded An issue that has been superseded by another and removed status: waiting-for-triage An issue we've not yet triaged labels Jul 13, 2022
@gregturn

Copy link
Copy Markdown
Contributor

Thanks @quaff. I used your solution as inspiration to 3d8b287.

@gregturn gregturn closed this Jul 13, 2022
gregturn added a commit that referenced this pull request Sep 23, 2022
Null values are wrapped with a special handler when interacting with Hibernate. However, this becomes an issue for queries when LIKE or CONTAINS are applied. In this situation, the null needs to be condensed into an empty string and any wildcards can then be applied with expected results.

Closes #2548, #2570.
Supercedes: #2585.
Related: #2461, #2544#
@gregturn

Copy link
Copy Markdown
Contributor

The related solution above has been backported to 2.7.x.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: superseded An issue that has been superseded by another

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants