Skip to content

DATAJDBC-417 - Fixed saving an entity containing a null embeddable with a reference to a further entity. #169

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

schauder
Copy link
Contributor

@schauder schauder commented Sep 18, 2019

@schauder schauder changed the title DATAJDBC-417 Reprocucer DATAJDBC-417 Reproducer Oct 16, 2019
Removed superfluous method.
…th a reference to a further entity.

Constructing the DbActions assumed that the parent of a path exists (i.e. is not null) and created parent nodes.
@schauder schauder changed the title DATAJDBC-417 Reproducer DATAJDBC-417 - Fixed saving an entity containing a null embeddable wi… …th a reference to a further entity. Oct 30, 2019
@schauder schauder changed the title DATAJDBC-417 - Fixed saving an entity containing a null embeddable wi… …th a reference to a further entity. DATAJDBC-417 - Fixed saving an entity containing a null embeddable with a reference to a further entity. Nov 13, 2019
@schauder schauder marked this pull request as ready for review November 13, 2019 10:50
mp911de pushed a commit that referenced this pull request Nov 13, 2019
Removed superfluous method.

Original pull request: #169.
mp911de pushed a commit that referenced this pull request Nov 13, 2019
…th a reference to a further entity.

Constructing the DbActions assumed that the parent of a path exists (i.e. is not null) and created parent nodes.

Original pull request: #169.
mp911de added a commit that referenced this pull request Nov 13, 2019
Reformat code. Fix collection-like node creation to cast to Iterable and consider arrays.

Original pull request: #169.
mp911de pushed a commit that referenced this pull request Nov 13, 2019
Removed superfluous method.

Original pull request: #169.
mp911de pushed a commit that referenced this pull request Nov 13, 2019
…th a reference to a further entity.

Constructing the DbActions assumed that the parent of a path exists (i.e. is not null) and created parent nodes.

Original pull request: #169.
mp911de added a commit that referenced this pull request Nov 13, 2019
Reformat code. Fix collection-like node creation to cast to Iterable and consider arrays.

Original pull request: #169.
@mp911de
Copy link
Member

mp911de commented Nov 13, 2019

That's merged, polished, and backported now.

@mp911de mp911de closed this Nov 13, 2019
@mp911de mp911de deleted the issue/DATAJDBC-417 branch November 13, 2019 12:37
mp911de added a commit that referenced this pull request Feb 21, 2022
We now support Converters on Entity-level if object materialization/dematerialization is handled by application code. We're using a custom R2DBC MappingContext to create mapping metadata for types that have custom converters registered.
mp911de added a commit that referenced this pull request Feb 21, 2022
Move R2dbcMappingContextUnitTests to correct source folder.
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