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

修复 jpa 模块 获取 Entity 类型错误的问题 #41

Merged
merged 1 commit into from
Jun 13, 2022
Merged

修复 jpa 模块 获取 Entity 类型错误的问题 #41

merged 1 commit into from
Jun 13, 2022

Conversation

ydq
Copy link
Contributor

@ydq ydq commented Jun 12, 2022

修复 SimpleTypeUtil#isSimpleType 判断基础类型时缺少原始类型判断导致 jpa 模块 update / insert 系列方法获取 Entity 类型错误的问题

#40

@abel533
Copy link
Contributor

abel533 commented Jun 13, 2022

由于基本类型存在默认值,所以一般都不推荐使用基本类型,原来提供的 registerPrimitiveTypes(),如果你想支持基本类型,需要提前调用这个方法。

你觉得直接支持基本类型合适吗?

@ydq ydq changed the title 修复 jpa 模块 获取 Entity 类型错误的问题 wip:修复 jpa 模块 获取 Entity 类型错误的问题 Jun 13, 2022
@ydq ydq changed the title wip:修复 jpa 模块 获取 Entity 类型错误的问题 修复 jpa 模块 获取 Entity 类型错误的问题 Jun 13, 2022
@ydq
Copy link
Contributor Author

ydq commented Jun 13, 2022

由于基本类型存在默认值,所以一般都不推荐使用基本类型,原来提供的 registerPrimitiveTypes(),如果你想支持基本类型,需要提前调用这个方法。

你觉得直接支持基本类型合适吗?

就我个人而言哈,在Model字段定义的时候肯定不会去使用基本来行,但是如果是ORM查询的返回值的话,除非是类似于 count 这种 100% 有值的情况下可能会考虑

这个PR 之前只想着去快速修复 jpa 遇到的问题,而且在查看源码的时候 SimpleTypeUtil 这个类只在 Finder#isEntityClass 中有使用,所以没考虑那么深远。已经重新提了

@abel533 abel533 merged commit 113e9a8 into mybatis-mapper:develop Jun 13, 2022
@afk6
Copy link

afk6 commented Jun 19, 2022

看了下代码, 发现我遇到的也是这个问题. 那就坐等新版了.

@abel533
Copy link
Contributor

abel533 commented Jun 20, 2022

本周会发布新版,有个底层改动比较大,一直没发。

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

Successfully merging this pull request may close these issues.

3 participants