When you delete a field that still has content then the field tables are renamed.

field_cron() then purges that data, but \Drupal\h5p\Plugin\Field\FieldType\H5PItem::delete() results in a query exception because it doesn't pass the deleted flag along.

Patch coming in a minute.

CommentFileSizeAuthor
#2 h5p-deleted-purge-3085464-2.patch1.14 KBberdir

Comments

Berdir created an issue. See original summary.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Active » Needs review
StatusFileSize
new h5p-deleted-purge-3085464-2.patch1.14 KB

This fixes the cron errors for me.

PS: Not having a 8.x-1.x-dev release is pretty uncommon and a bit confusing as you can't set the version to the branch.

miro_dietiker’s picture

miro_dietiker
Location Switzerland
commented
Status: Needs review » Reviewed & tested by the community

Yeah thx, i can see that this works. :-)

igonzalez’s picture

igonzalez commented

works for me

  • paalj committed c011bb5 on 8.x-1.x authored by Berdir 
    Issue #3085464 by Berdir: Delete query fails when purging data for...
paalj’s picture

paalj commented
Assigned: Unassigned » paalj
Status: Reviewed & tested by the community » Fixed
paalj’s picture

paalj commented
Status: Fixed » Closed (fixed)