Skip to content

DATAMONGO-978 - Derived delete query should pass on type information. #199

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 2 commits into from

Conversation

christophstrobl
Copy link
Member

We now pass on type information for derived delete queries to the according delete operation. This propagates the information correctly to the Before and After events.

Before this change the type would have been set to null in case of non collection like method return type.


Should be back ported to 1.5.x.

We now pass on type information for derived delete queries to the according delete operation. This propagates the information correctly to the according Before and After events.

Before this change the type would have been set to null in case of non collection like method return type.
christophstrobl added a commit that referenced this pull request Jul 3, 2014
We now pass on type information for derived delete queries to the according delete operation. This propagates the information correctly to the according Before and After events.

Before this change the type would have been set to null in case of non collection like method return type.

Original pull request: #199.
@thomasdarimont
Copy link

LGTM, cherry-picked with a typo fixed (Abstrac[t]...) via 998bb09.

christophstrobl added a commit that referenced this pull request Jul 3, 2014
We now pass on type information for derived delete queries to the according delete operation. This propagates the information correctly to the according Before and After events.

Before this change the type would have been set to null in case of non collection like method return type.

Original pull request: #199.
@mp911de mp911de deleted the issue/DATAMONGO-978 branch February 10, 2016 11:16
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