cayenne-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [cayenne] johnthuss opened a new pull request #382: Cayenne 4.2.M1 testing
Date Fri, 03 May 2019 15:57:04 GMT
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

Mime
View raw message