The PHPDoc for _batch_populate_queue() refers to BatchStaticQueue but there is no such thing. I blame Crell. Patch changes it to what we have in core, which is BatchMemoryQueue. I consider this patch critical.

CommentFileSizeAuthor
nobatchstaticqueuejv.patch918 bytesjvandyk
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Needs review » Reviewed & tested by the community

Looks great. Confirmed that BatchMemoryQueue is the correct class.

/me glares at Crell...

bdragon’s picture

bdragon CreditAttribution: bdragon commented

ARGH, beat me to the rtbc.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented

Muahahaha

marcingy’s picture

marcingy CreditAttribution: marcingy commented

nobatchstaticqueuejv.patch queued for re-testing.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)

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