Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Check for cyclic dependencies on update #42

Open
2 tasks
Nostradamos opened this issue Oct 29, 2017 · 2 comments
Open
2 tasks

Check for cyclic dependencies on update #42

Nostradamos opened this issue Oct 29, 2017 · 2 comments

Comments

@Nostradamos
Copy link
Owner

Nostradamos commented Oct 29, 2017

Currently it's possible to have a plant where it's generationParents include it's own plantId
If possible try to supress this. Maybe not possible, or not that easy.
Attributes where this kind of error is possible:

  • generationParents
  • plantClonedFrom
PlantId -> GenerationParents -\
  /|\                         |
   |------------------------
@Nostradamos
Copy link
Owner Author

Should be possible to fix with an extra query in front of the update process.

@Nostradamos
Copy link
Owner Author

Nostradamos commented Dec 3, 2017

Could possibly get solved by a CHECK constraint:
https://sqlite.org/lang_createtable.html

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

No branches or pull requests

1 participant