Skip to content

DATAMONGO-1509 - Write type hint as last element of an Document #411

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

Closed
wants to merge 3 commits into from

Conversation

christophstrobl
Copy link
Member

Always add type hint as last property of document.
This necessary to assure document equality within MongoDB in cases where the query contains full Document comparisons. Unfortunately this also might break existing stuff since the order of properties within a Document is changed with this commit.

Always add type hint as last property of document.
This necessary to assure document equality within mongodb in cases where the query contains full document comparisons. Unfortunately this also might break existing stuff since the order of properties within a Document is changed with this commit.
@mp911de mp911de changed the title Issue/2.0.x/datamongo 1509 DATAMONGO-1509 - Write type hint as last element of an Document Nov 17, 2016
mp911de pushed a commit that referenced this pull request Nov 17, 2016
Always add type hint as last property of a Document.
This is necessary to assure document equality within MongoDB in cases where the query contains full document comparisons. Unfortunately this also might break existing stuff since the order of properties within a Document is changed with this commit.

Original pull request: #411.
mp911de added a commit that referenced this pull request Nov 17, 2016
Adopt type hint assertion for existing _class field checks. Simplify test code to use Collections.singletonList instead of Arrays.asList. Replace BasicDBList with List in JavaDoc. Use type inference for DocumentTestUtils.getAsDBList to avoid casts in test code. Extend documentation.

Original pull request: #411.
@mp911de mp911de closed this Nov 17, 2016
@mp911de mp911de self-assigned this Nov 17, 2016
@odrotbohm odrotbohm deleted the issue/2.0.x/DATAMONGO-1509 branch November 18, 2016 13:15
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.

2 participants