[Deprecated] DMZ 1.6.2 (DataMapper OverZealous Edition)

[eluser]Michael Ditto[/eluser]
[quote author="TheJim" date="1260939576"]

Code:

(Line 1400)
    function _delete($related_field, $arguments)
    {
        $this->delete($arguments[0], $related_field);
    }

This should have a return so we pass through the success/failure when deleting relations.[/quote]

I was just about to make the same observation. This manifested itself in the case of deleting a required relation and then saving the object with a new relation. The result of the save was an error that the relation was required, even though the operation was saving the relation. Simple fix that vexed me for more time than it should have had I noticed your post first. :-)

To clarify for the next hapless sap like me, that function should be modified thusly:

Code:

(Line 1400)
    function _delete($related_field, $arguments)
    {
        return ($this->delete($arguments[0], $related_field));
    }