Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
A setup is a field collection entity, one of which fields is a custom field with an embedded view. This results in a 'bad operand' critical error in entity.controller.inc, which comes from the fact that += is used to merge arrays rather than array_merge(). using array merge solves the problem. Please use array_merge() to merge the arrays in buildContent().
Comment | File | Size | Author |
---|---|---|---|
#6 | entity_array_merge_2344689-6.patch | 861 bytes | alexrayu |
#4 | entity_array_merge_2344689-4.patch | 941 bytes | alexrayu |
#1 | entity_2344689_1_array_merge.patch | 1021 bytes | alexrayu |
Comments
Comment #1
alexrayu CreditAttribution: alexrayu commentedHere is a proposed patch.
Comment #2
alexrayu CreditAttribution: alexrayu commentedComment #4
alexrayu CreditAttribution: alexrayu commentedCorrected file paths.
Comment #6
alexrayu CreditAttribution: alexrayu commentedMore corrections to patch.
Comment #7
alexrayu CreditAttribution: alexrayu commentedPlease review and include in the release! It seems like a little thing, but it makes the difference between a normal work of the site and a critical error in our case.
Comment #8
Chris Matthews CreditAttribution: Chris Matthews as a volunteer commentedThe 4 year old patch in #6 to entity.controller.inc applied cleanly to the latest entity 7.x-1.x-dev and (if still relevant) needs review.