Skip to content
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

Enhance entity creation with base field expansion and dynamic id key #271

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

chrisolof
Copy link

@chrisolof chrisolof commented Mar 6, 2024

Resolves #270.

Pairs really well with jhedstrom/drupalextension/pull/661.

…nsion

Per FieldHandlerInterface::expand(), field handlers expect an array of
field values.  The source entity object passed into expandEntityFields()
often has an inconsistent structure across its various field properties,
sometimes holding a single value, sometimes holding an array of values (as
expected), and sometimes holding an array of composite field parts for a
single field value.  Consistently providing field handlers the field values
in the structure they expect allows them to expand each field value
correctly.

jhedstromgh-270
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.

Entity creation does not expand base fields & assumes "id" for entity ID key (D8/9/10 driver)
1 participant