Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
FieldStorageConfigStorage incorrectly attempts to load by properties if the conditions array contains the keys 'entity_type' and 'field_name'. This is due to the way that specific IDs are attempted to be loaded, but do not contain a period between the entity type and the field name, merging the two together (e.g. nodebody instead of node.body).
Comment | File | Size | Author |
---|---|---|---|
#6 | field-incorrect-id-loading-from-storage-2941161-6-D8.patch | 777 bytes | thomas.pischke.newscycle |
Comments
Comment #2
thomas.pischke.newscycle CreditAttribution: thomas.pischke.newscycle commentedHere is a patch to fix the issue.
Comment #3
thomas.pischke.newscycle CreditAttribution: thomas.pischke.newscycle commentedComment #5
amateescu CreditAttribution: amateescu for Pfizer, Inc. commentedOh wow, very nice catch! We need to add a test for this, probably somewhere inside
\Drupal\Tests\field\Kernel\FieldStorageCrudTest
:)Comment #6
thomas.pischke.newscycle CreditAttribution: thomas.pischke.newscycle commentedAh, whoops, I created the patch from the wrong directory. Moved it out to the parent level.
Comment #7
thomas.pischke.newscycle CreditAttribution: thomas.pischke.newscycle commentedComment #8
thomas.pischke.newscycle CreditAttribution: thomas.pischke.newscycle commentedComment #9
borisson_Setting to needs work, so we remember to write tests for this, based on #5.
Comment #11
init90closed as a duplicate of #2917606: FieldStorageConfigStorage::loadByProperties() with passed 'entity_type' and 'field_name' as conditions doesn't return any result