Richard Wallace wrote:
> Ok, good to know.  What about the ValidatableAbstractComposite?  I think 
> I just tried that out of desperation, but do I need that?

No, the exception will be thrown automatically now. The whole 
Validatable concept needs to be rethought I think. It is still useful to 
have, but must be done differently.

> Hmm... well I tried changing the constraint just to NotNull and all the 
> tests still pass.  I've stepped through the Qi4j code and the 
> EntityPropertyInstance doesn't do any Constraint checking.   It's 
> parent, the PropertyInstance class does, but the one for Entities 
> doesn't.  I would assume this is so it can be done when the unit of work 
> is complete, but I've stepped through all the unit of work completion 
> code and can't find anywhere the constraints are checked.

Sounds like a bug, plain and simple. Will fix it, and it's great that 
you found it!

/Rickard

_______________________________________________
qi4j-dev mailing list
[email protected]
http://lists.ops4j.org/mailman/listinfo/qi4j-dev

Reply via email to