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

Regression: 3.5.17 DateOnlyTypeHandler returns java.sql.Date #3332

Closed
JoshKloster opened this issue Dec 27, 2024 · 3 comments
Closed

Regression: 3.5.17 DateOnlyTypeHandler returns java.sql.Date #3332

JoshKloster opened this issue Dec 27, 2024 · 3 comments
Assignees
Labels
Milestone

Comments

@JoshKloster
Copy link
Contributor

Bug report:

In 3.5.17 a change was made to DateOnlyTypeHandler.java that effected the return type of multiple functions.

https://github.com/mybatis/mybatis-3/pull/3247/files#diff-3100284b93f6e47f7e09229bac897dc14edbb2e162002f24e804113f49d8b6a6L47

Previously a java.util.Date object was returned from the "getNullableResult" functions. After the change now java.sql.Date objects are returned.

MyBatis version

3.5.17

Database vendor and version

Test case or example project

Steps to reproduce

Expected result

Actual result

@harawata
Copy link
Member

Thank you for the report, @JoshKloster ,

If you are keen, please send us a pull request.
If you are busy, that's fine. I'll fix it when I have time.

@JoshKloster
Copy link
Contributor Author

JoshKloster commented Dec 27, 2024

#3334 - thank you!

@harawata harawata self-assigned this Dec 27, 2024
@harawata harawata added this to the 3.5.18 milestone Dec 27, 2024
@harawata harawata changed the title MyBatis 3.5.17 DateOnlyTypeHandler changed return type Regression: 3.5.17 DateOnlyTypeHandler returns java.sql.Date Dec 27, 2024
@harawata
Copy link
Member

#3334 is merged.
Thank you, @JoshKloster !

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

No branches or pull requests

2 participants