Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

EventBase.part_of should be acyclic #292

Open
cogat opened this issue Aug 29, 2017 · 0 comments
Open

EventBase.part_of should be acyclic #292

cogat opened this issue Aug 29, 2017 · 0 comments

Comments

@cogat
Copy link
Contributor

cogat commented Aug 29, 2017

Two events A and B can be made to be part_of one another, ie event A can indirectly (possibly even directly) be its own ancestor. This can cause recursion-depth errors, when trying to traverse relationships.

However, in the ACMILabs fork, we've also made part_of a many-to-many, since an event can be part_of many parent events. So the event structure should be a directed acyclic tree, rather than an mptt-type tree where every Event only has one parent.

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

No branches or pull requests

1 participant