Details
-
Type:
Bug
-
Status:
Closed
-
Priority:
Critical
-
Resolution: Fixed
-
Affects Version/s: 2.0-ALPHA3
-
Fix Version/s: 2.0-ALPHA4
-
Component/s: Mapping Drivers
-
Security Level: All
-
Labels:None
Description
Hi there,
I've attached a small patch which addresses the following issues:
-> YAML did not support cascading options "all" (as described in documentation) and "detach" (as used in AssociationMapping class). The attached patch fixes that.
-> XML does not support these things either, but the attached patch DOES NOT FIX THAT (see below).
Also addressed:
YAML had syntax like cascadeRemove, cascadePersist etc., but the documentation for the annotation driver uses only "persist, remove" etc.
This patch changes the syntax to the following:
cascade: [ persist, remove ]
OR
cascade:
- persist
- remove
(Both work!)
It is now also possible to use cascade: [all]
The YAML Driver was changed in that way that he must not know about the currently supported cascate actions. This has been pushed towards the AssociationMapping and I strongly believe that this is the right way.
On the other hand, I did not change the XML Driver functionality. It does still know about the cascade actions and does most probably not support the detach and "all" actions. That should be fixed, but won't take too long.