search

kaleidos / grails-postgresql-extensions

Grails plugin to use postgresql native elements such as arrays, hstores,...
Apache License 2.0
78 stars 63 forks source link

Equality JsonMapType #86

Closed igilfanov closed 8 years ago

igilfanov commented 8 years ago

Checking Map values for equality should be revised.

http://stackoverflow.com/questions/4082416/equality-between-2-hashmap

ilopmar commented 8 years ago

Thanks for submitting the issue.

Could you be please more specific about the problem? Does it means that this https://github.com/kaleidos/grails-postgresql-extensions/blob/master/src/main/groovy/net/kaleidos/hibernate/usertype/JsonMapType.java#L40 should be done in a different way?

Could you send me an example of what's happending?

Regards, Iván.

igilfanov commented 8 years ago

It does not update value of the field. The fact that it can be done differently or - guess. Just try to change the value, under such conditions, when the values of other fields unchanged.

ilopmar commented 8 years ago

Could you create an sample project that reproduces the issue and push it to github?

igilfanov commented 8 years ago

Not today, but I'll try. Thank you

ilopmar commented 8 years ago

Perfect. That way I'll be able to reproduce it and fix it.

igilfanov commented 8 years ago

https://github.com/igilfanov/jsonbupdate

igilfanov commented 8 years ago

I should have used beforeValidate