xorm
xorm copied to clipboard
go-xorm
time.Time默认值不再是 0001-01-01 00:00:00 了吗？
数据库：
create table `user`(
`name` varchar(512) NOT NULL,
`login_time` datetime NOT NULL DEFAULT '0000-00-00 00:00:00',
)
注意：login_time 是NOT NULL
model:
type User struct{
Name string `xorm:"'name'"
LoginTime time.Time `xorm:"'login_time'"`
}
Insert代码：
user:=&User{
Name:"string",
}
_,err:=session.InsertOne(user)
对于以前的版本(0.5以前?)，以上语句不会出错,因为生成的语句是：
INSERT INTO `user` (`name`,`login_time`) VALUES (?, ?) [args] [ string 0001-01-01 00:00:00 ] - took: 776.254µs
而升级到现在的版本后(dep ensure -update)：
name = "github.com/go-xorm/xorm"
packages = ["."]
revision = "d85bb4911c725a53b135fc12862748a41b6c07da"
version = "v0.7.0"
以上的插入代码会出错，因为生成的语句变成了：
INSERT INTO `user` (`name`,`login_time`) VALUES (?, ?) []interface {}{"string",interface{}(nil)} - took: 776.254µs
这样导致以前设置了NOT NULL字段的数据表，在插入的时候，会报错：
Error 1048: Column 'login_time' cannot be null
导致无法完美无缝的将xorm迁移到新版
同时,即使指定了默认值，生成的语句也不是按照默认值来的
payTime time.Time `xorm:"not null default '0001-01-01 00:00:00' pay_time"`
生成的语句依然是：
INSERT INTO `user` (`pay_time`) Values(?) []interface{}{""}
此时，指定了not null,所以 time.Time在这里又变成了 "" ，而不是之前的 interface{}(nil).
此时，如果mysql数据的sql_mode设置的是严格模式(一般情况下都是严格模式)，这这种日期会被认为是非法的日期，从而导致插入数据库不成功。
避免这么多不可控因素最好的方案就是，对于 time.Time 的默认值，首先按照 struct tag中定义的 default value来传递，如果struct tag 中没有定义default 值，那么应该采用golang的默认值： '0001-01-01 00:00:00' ，而不是一会儿是 "", 一会又是 interface{}(nil)
升级xorm后,sql中的time.Time默认值变为 interface{}(nil) 指定字段NOT NULL后,sql中的time.Time默认值为"", 均无法满足原有数据库字段NOT NULL DEFAULT '0000-00-00 00:00:00'的要求.（msyql5.7 严格模式下）
因此，目前一个兼容方案是 当 time.Time isZero，插入数据时省略该字段
if user.LoginTime.IsZero(){
session = session.Omit("login_time")
}
没有默认值时，建议最好不要用nil. 原因有三： 第一， golang中 time.Time 和 nil 是不能直接相等的，time.Time 的默认值不是 nil
t := new(time.Time)
if *t == nil { // panic . cannot convert nil to type time.Time
fmt.Println("err")
}
第二，如果采用nil为默认值，会导致更新数据库中NOT NULL字段的时候报错“不允许为null”
第三，虽然golang中 '0001-01-01 00:00:00' 和 '0000-00-00 00:00:00' 是一样的，但是在数据库为严格模式下，'0000-00-00' 这类时间会被认为是非法的时间，而'0001-01-01'则不会被认为为非法。
所以，最好的方案是最终还是采用和以前的版本一样的 '0001-01-01 00:00:00',既能保持向后兼容，又能不会受到 sql_mode 的限制。
当然最完美的方案就是，优先按照用户指定的 struct_tag 中的 default value来，用户没有指定，则采用golang的time.Time 的默认值
fmt.Println(new(time.Time)) // 0001-01-01 00:00:00 +0000 UTC
这样可以正常进行，
for _, v := range in {
_, err := dstSession.Omit("end_time").Insert(d)
panic(err)
}
但是，这样，任然会报 Error 1048: Column 'end_time' cannot be null
_, err := dstSession.Omit("end_time").Insert(in....)
这是为啥？？？
