search

skyscreamer / yoga

Yoga is RESTful but flexible.
http://yoga.skyscreamer.org/
Apache License 2.0
156 stars 66 forks source link

Consider Date primitive #235

Closed kentongray closed 10 years ago

kentongray commented 10 years ago

I was having issues with Yoga breaking date handling in comparison to how plain Jersey did it. Jersey would serialize an epoch int where as Yoga would just give me { }. I tracked it down into the PojoProperty and it was calling ObjectUtil.isPrimitive(). By adding Date to the list of primitives this solves this issue.

I can't imagine any time you would ever want to get the date object or use a selector on it (seeing as how it only has one non-deprecated method). I checked and do not see any reason this should cause issues and all tests passed.

Let me know what you think. I've been able to work around this via extending PojoProperty but I feel like this is more the desired behavior.

jkovacs commented 10 years ago

+1, see also issue #162

carterpage commented 10 years ago

Jackson still seems to have some problems with dates, but this is a step in the right direction.