Skip to content

Conversation

@jc
Copy link

@jc jc commented May 29, 2025

SOQL query results may contain objects/map if they are a lookup or compound type (address or location). Previously the code failed to populate the record with the object value if it was not a lookup. This change populates record fields of type address or location with the value.

SOQL query results may contain objects/map if they are a lookup or
compound type (address or location). Previously the code failed to
populate the record with the object value if it was not a lookup. This
change populates record fields of type address or location with the
value.
@jc jc requested a review from nyergler May 29, 2025 15:37
@jc
Copy link
Author

jc commented May 29, 2025

I'm not sure we care but I found this issue when looking into https://github.com/polytomic/app/issues/11001

Copy link
Member

@nyergler nyergler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This makes sense to me. I poked at it a little to see if there was anything simpler possible, but it seems like this is the most reasonable way to distinguish between a compound value and a sub-query/lookup result.

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.

3 participants