mybatis-mapper / mapper

MyBatis Mapper
https://mapper.mybatis.io
Apache License 2.0
325 stars 47 forks source link

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

Closed ydq closed 2 years ago

ydq commented 2 years ago

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

https://github.com/mybatis-mapper/mapper/issues/40

abel533 commented 2 years ago

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

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

ydq commented 2 years ago

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

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

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

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

afk6 commented 2 years ago

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

abel533 commented 2 years ago

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