search

bootique / bootique-cayenne

Cayenne Integration Module for Bootique
https://bootique.io
Apache License 2.0
5 stars 8 forks source link

More informative errors on mixing different configurations #67

Open andrus opened 5 years ago

andrus commented 5 years ago

As reported in the comments of #54 , combining CayenneModule.addProject(..) with an explicit YAML config of DataMaps (correctly) results in an error. But the error doesn't help to identify the cause. Perhaps we can do better validation and at least list all of the sources of Cayenne configuration in the exception.

aarrsseni commented 5 years ago

I looked on this issue and have some notes. Exception is thrown by cayenne code and happened because of duplicated entities in different datamaps. In 4.1.B3 validation of this case was added. If there are entities with the same names, cayenne will show warnings. So maybe this issue can wait for upgrade to cayenne 4.1.B3 or this validation should be added to boutique-cayenne module?

andrus commented 5 years ago

@aarrsseni . Yeah , let's wait till B3 (which will hopefully be RC1)