Related to the great work done on #2054555, and badly needed as some queued tasks can be storing a lot of state within the data column causing this table to become very large very quickly. Unfortunately in between applying the patch in #17 and it getting merged, it seems that a bug was created. Specifically, within _advancedqueue_purge_history the conditional within which the db_delete of old rows is run is referencing an undefined variable. Looks like a simple mistake entered when the patch was applied and modified slightly. I'm attaching a patch which resolves this.

CommentFileSizeAuthor
#2 advancedqueue-cleanup-table-defined-var-fix-2854318-1.patch826 bytessambonner
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sambonner created an issue. See original summary.

sambonner’s picture

sambonner CreditAttribution: sambonner at Catalyst IT commented
FileSize
advancedqueue-cleanup-table-defined-var-fix-2854318-1.patch826 bytes

Attaching patch.

hgoto’s picture

hgoto CreditAttribution: hgoto as a volunteer commented
Status: Active » Reviewed & tested by the community

Thank you @sambonner. I confirmed that the bug exists and the patch fixes it.

deggertsen’s picture

deggertsen CreditAttribution: deggertsen commented

+1 to this patch. Thank you.

  • pjcdawkins committed ead37a4 on 7.x-1.x authored by sambonner 
    Issue #2854318 by sambonner: Advancedqueue table cleanup - table never...
pjcdawkins’s picture

pjcdawkins CreditAttribution: pjcdawkins at Platform.sh commented
Status: Reviewed & tested by the community » Fixed

Committed. Many thanks, @sambonner @hgoto and @deggertsen!

Status: Fixed » Closed (fixed)

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