The code that obtains the field values fails when the field does not return a FieldItemListInterface.

$value = ($entity->$field_name)->getValue();
CommentFileSizeAuthor
#4 panopoly_wysiwyg-update_hook-2925652-4.patch632 bytesdsnopek
#2 panopoly_wysiwyg-update_hook-2925652-2.patch1.24 KBkerasai
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kerasai created an issue. See original summary.

kerasai’s picture

kerasai CreditAttribution: kerasai at myDropWizard commented
Title: Update Hook 8204 Does Not Complete » WYSIWYG Update Hook 8204 Does Not Complete
Status: Active » Needs review
FileSize
panopoly_wysiwyg-update_hook-2925652-2.patch1.24 KB

Adjusted the code that obtains the field value and added a conditional to check that the value is set and is not empty.

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
CreditAttribution: dsnopek commented
Status: Needs review » Fixed

Looks good to me - thanks! Committed :-)

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
CreditAttribution: dsnopek commented
Status: Fixed » Needs review
FileSize
panopoly_wysiwyg-update_hook-2925652-4.patch632 bytes

Found another issue. Let's try this one!

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
CreditAttribution: dsnopek commented
Status: Needs review » Fixed

Seems to be working - committed #4 too

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.