Re: [Joda-interest] added hibernate persister for LocalTime and LocalDate

2007-01-31 Thread Ignacio Coloma
Hi, I was looking at the joda hibernate project, and already implemented my own persister for LocalTime and Duration (taking the great current implementation as starting point). A couple of things could be improved imho: * The persisters extend EnhancedUserType, but they could extend UserType

Re: [Joda-interest] added hibernate persister for LocalTime and LocalDate

2007-01-31 Thread Mario Ivankovits
Hi Ignacio! * The persisters extend EnhancedUserType, but they could extend UserType (the extra methods are not used) I am still not sure about it. The documentation of EnhancedUserType shows, that such types can be used as identifier or discriminator, so I assumed that this is not the case

Re: [Joda-interest] added hibernate persister for LocalTime and LocalDate

2007-01-31 Thread Ignacio Coloma
Hi Mario :) * The persisters extend EnhancedUserType, but they could extend UserType (the extra methods are not used) I am still not sure about it. The documentation of EnhancedUserType shows, that such types can be used as identifier or discriminator, so I assumed that this is not the

Re: [Joda-interest] added hibernate persister for LocalTime and LocalDate

2007-01-31 Thread Mario Ivankovits
Hi! Ok, ok, I know, I am lazy. Sorry for this. I'll have a look at it. Lazy? Your implementation is great! I only added some tuning. :-) But I AM lazy, yesterday I thought about cleaning up the object hierarchy but well ... you've seen what happened ;-) If you don't mind I