Hello
i have a problem with Varnish Purger, the Purge queue is not filled anymore. In Drupal.log i find the following message
node/4/edit|243||Notice: Undefined offset: 0 in varnish_image_purge_entity_update() (line 98 of /var/XXX/modules/varnish_purge/modules/varnish_image_purge/varnish_image_purge.module)
And one line below
node/4/edit|243||Warning: array_flip(): Can only flip STRING and INTEGER values! in Drupal\Core\Entity\EntityStorageBase->loadMultiple()
It already worked in the past.
I hope you can help me.
With kind regards
Chris
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | 2933886-4.varnish_purge.Undefined-offset-0-in-varnishimagepurgeentityupdate.patch | 2.08 KB | mikelutz |
Comments
Comment #2
ChrisBee commentedComment #3
mikelutzWhy was this issue closed as works as designed with no comment? This is causing me headaches as well. varnish_image_purge_entity_update() collects all the image fields on an entity, and accesses element 0 without checking to see if the image field is empty or not, throwing a notice in the array index access attempt, and a warning when calling File::load(NULL).
In addition, if I'm reading this correctly, if the image field has a cardinality > 1, only the first image in the field is queued for cache removal, and all the rest are ignored. This should be looping through all the images in the image field, if available and queueing them all (or none if the field is empty)
Comment #4
mikelutzThis patch checks to make sure the image field is not empty, and purges all images in a field, not just the first one.
Comment #6
misc commentedThanks for the patch @mikelutz, I have added it to the latest dev, and it will be in the new release later today.