[GitHub] [cayenne] johnthuss opened a new pull request #382: Cayenne 4.2.M1 testing

[GitBox <gi...@apache.org>]

johnthuss opened a new pull request #382: Cayenne 4.2.M1 testing
URL: https://github.com/apache/cayenne/pull/382
 
 
   I'm trying out 4.2.M1 snapshot in order to provide testing feedback. 
   
   These changes aren't really meant to be applied directly, but to provide feedback into the issues I'm having.
   
   1) Joda time support is lacking in the class generation templates. I understand this is deprecated now, but since it hasn't been removed yet it should still work. Thinking ahead to the future there will need to a way for third parties to integrate Joda support into cayenne without having to create a hard fork. This doesn't appear to currently be possible - at least I don't know how to provide an additional Module to cgen via ant that could provide PropertyDescriptorCreator changes.
   
   2) I need to have a method like PropertyUtils.propertyTypeDefinition exposed so I can utilize it in my templates. The refactoring to move logic into PropertyUtils makes the templates clean and simple, but users need access to smaller pieces as well. 
   
   3) Using "between" expressions on Joda time DateProperty instances fails. This is a bug that still needs to be fixed. I don't know where to start on this one.
   
   4) Ant's cgen task was completely broken. I doubt this is the correct fix, but it does get it running again.
   
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services