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.

Sign up for GitHub

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

Jump to bottom

Save with cascade=True fix for unsaved child documents #2605

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

Save with cascade=True fix for unsaved child documents #2605

wants to merge 12 commits into from

Commits on Dec 14, 2021

  1. Issue MongoEngine#1236 

    Save the children documents first to avoid the issue where a parent cannot save due to having new children documents.
    @nickfrev
    nickfrev committed Dec 14, 2021
    Configuration menu
    Copy the full SHA
    efca40d View commit details
    Browse the repository at this point in the history

  2. Added Testing for saving new ReferenceField with cascade

    @nickfrev
    nickfrev committed Dec 14, 2021
    Configuration menu
    Copy the full SHA
    bb6f7d8 View commit details
    Browse the repository at this point in the history

  3. Moved cascade save handling higher up to avoid validation errors.

    @nickfrev
    nickfrev committed Dec 14, 2021
    Configuration menu
    Copy the full SHA
    e9a9dba View commit details
    Browse the repository at this point in the history

  4. Updated Authors

    @nickfrev
    nickfrev committed Dec 14, 2021
    Configuration menu
    Copy the full SHA
    e0cc779 View commit details
    Browse the repository at this point in the history

  5. Removed unnecessary newlines at end of test.

    @nickfrev
    nickfrev committed Dec 14, 2021
    Configuration menu
    Copy the full SHA
    979e491 View commit details
    Browse the repository at this point in the history

Commits on Dec 21, 2021

  1. Modularize the cascade save event 

    - Created a new base field (SaveableBaseField) which allows a field to marked as savable during a cascade save.
- Each SaveableBaseField defines a save method which describes how it will deal with a cascade save call this allows lists, dicts, and maps to be effected during a cascade save.
- Added an _is_saving flag during Document save to avoid saving a document that is already in the process of being saved. (Caused if there is a circular reference.)
    @nickfrev
    nickfrev committed Dec 21, 2021
    Configuration menu
    Copy the full SHA
    7fbdf8b View commit details
    Browse the repository at this point in the history

Commits on Dec 22, 2021

  1. Allowed ReferenceField cycles of unsaved documents 

    - Allows for users to cascade save unsaved documents even if a cycle exists.
    @nickfrev
    nickfrev committed Dec 22, 2021
    Configuration menu
    Copy the full SHA
    7672f2c View commit details
    Browse the repository at this point in the history

  2. Removed TOFIX comment that I fixed.

    @nickfrev
    nickfrev committed Dec 22, 2021
    Configuration menu
    Copy the full SHA
    ba400b7 View commit details
    Browse the repository at this point in the history

Commits on May 7, 2022

  1. Merge branch 'master' into save-with-cascade-include-nested

    @nickfrev
    nickfrev committed May 7, 2022
    Configuration menu
    Copy the full SHA
    e6ef197 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'master' into save-with-cascade-fix-for-unsaved-child-do… 

    …cuments
    @nickfrev
    nickfrev committed May 7, 2022
    Configuration menu
    Copy the full SHA
    f6215a6 View commit details
    Browse the repository at this point in the history

Commits on May 8, 2022

  1. Merge pull request #1 from nickfrev/save-with-cascade-include-nested 

    Multiple fixes for cascade=True save issues
    @nickfrev
    nickfrev committed May 8, 2022
    Configuration menu
    Copy the full SHA
    e68df3e View commit details
    Browse the repository at this point in the history

Commits on Oct 4, 2022

  1. Merge branch 'master' into save-with-cascade-fix-for-unsaved-child-do… 

    …cuments
    @nickfrev
    nickfrev committed Oct 4, 2022
    Configuration menu
    Copy the full SHA
    a1db6e0 View commit details
    Browse the repository at this point in the history