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

3.2.0 backports of the fixes for dict-based child entity update implementation #313

Merged
merged 3 commits into from
Jun 5, 2024

Conversation

Yogu
Copy link
Member

@Yogu Yogu commented Feb 7, 2024

No description provided.

… add new child entities

Previously, we did not check whether the child entities we wanted to update
actually existed.

The bug has been introduced with the new dict-based child entity update
implementation.
…rrently were no child entities

When the dict-based child entity impelemntation was used, and an non-empty
update... for child entity was given, and this child entity list for the
updated root entity was currently empty, and AQL error was generated.

The problem was that we used a range generator that should have generated
an empty range, but the AQL ranges are never empty, they just are inverted
if the end position is lower than the start position. This resulted in an
invalid ZIP call, which yielded NULL.
@Yogu Yogu merged commit 70a434c into release/3.2.0 Jun 5, 2024
20 of 22 checks passed
@Yogu Yogu deleted the 3.2.0-child-entity-update-fixes branch June 5, 2024 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant