Fix wrong usage of xorm

efectn / go-orm-benchmarks

Advanced benchmarks for +15 Go ORMs.

MIT License

308 stars 33 forks source link

Closed lunny closed 2 years ago

lunny commented 2 years ago

For xorm, some usage is wrong.

Update should have a condition, otherwise it will update all the records
Read should use id as condition to get one record from database and disable automatically conditions from the struct.

efectn commented 2 years ago

Hi @lunny. XORM gives ID condition is error, expect 0 primarykeys, there are 1 after this changes. Can you check it? Thanks

lunny commented 2 years ago

Hi @lunny. XORM gives ID condition is error, expect 0 primarykeys, there are 1 after this changes. Can you check it? Thanks

I have sent #3 to fix that.