Skip to content
This repository has been archived by the owner on Sep 14, 2023. It is now read-only.

Recursive dao call ability was added #38

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

buzden
Copy link
Contributor

@buzden buzden commented Jun 25, 2014

Dao was changed to make able to insert the recursive structure of objects automatically.

Only a stub was added for the update and reload operations.

This change is backward-compartible. It it going after the dao-hooks pull request for easing of merging (is case of acceptance).

@agentgt
Copy link
Owner

agentgt commented Aug 14, 2014

The problem with the recursive dao idea is how do you know when to insert child objects or just associate?

In my opinion and this is after dealing with hibernates recursive support (ie graph serialization), Its not that much work to manually insert children and is more explicit about what is actually happening.

@buzden
Copy link
Contributor Author

buzden commented Aug 14, 2014

I agree that what should be done with a child is a not a thing that should be hardcoded in a library.

But what I suggested in this pull request is to give a library user a chance to do something with a child (by setting a kind of a callback) and do nothing by default. No predefined decisions were made and flexibility was improved, to my point of view.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants