Skip to content

Provide DocumentReferenceSource to document reference lookup for use of properties in constructor creation #4486

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

sxhinzvc
Copy link
Contributor

This commit enables a source entity with a one-to-many @DocumentReference lookup property to be instantiated properly when the referenced entity - which contains the linking value - is used as a constructor argument.

@sxhinzvc sxhinzvc requested a review from mp911de August 28, 2023 20:39
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Aug 28, 2023
@mp911de mp911de added type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged labels Aug 29, 2023
@mp911de mp911de changed the title Fix one-to-many lookup resolution for properties used in constructor Provide DocumentReferenceSource to document reference lookup for use of properties in constructor creation Aug 29, 2023
@christophstrobl
Copy link
Member

LGTM

@mp911de
Copy link
Member

mp911de commented Aug 29, 2023

I updated the ticket title. One-to-many didn't fit necessarily as we can have single-valued references as well.

@sxhinzvc sxhinzvc marked this pull request as ready for review August 29, 2023 13:15
This commit enables document reference lookup to use `DocumentReferenceSource` to properly instantiate an entity containig a @DocumentReference `#self` property.

Closes #4484
Add myself to list of authors.

Original Pull Request: #4486
@christophstrobl christophstrobl self-assigned this Aug 30, 2023
christophstrobl pushed a commit that referenced this pull request Aug 31, 2023
This commit enables document reference lookup to use `DocumentReferenceSource` to properly instantiate an entity containig a @DocumentReference `#self` property.

Closes #4484
Original Pull Request: #4486
christophstrobl pushed a commit that referenced this pull request Aug 31, 2023
This commit enables document reference lookup to use `DocumentReferenceSource` to properly instantiate an entity containig a @DocumentReference `#self` property.

Closes #4484
Original Pull Request: #4486
christophstrobl pushed a commit that referenced this pull request Aug 31, 2023
This commit enables document reference lookup to use `DocumentReferenceSource` to properly instantiate an entity containig a @DocumentReference `#self` property.

Closes #4484
Original Pull Request: #4486
@christophstrobl christophstrobl deleted the issue/4484 branch August 31, 2023 07:26
@mp911de mp911de added this to the 4.0.10 (2022.0.10) milestone Aug 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A general bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Constructor creation using @DocumentReference SpEL referring to self in fails
4 participants