a bad query is preventing batch processing from working correctly. token is a string, so it should not be escaped as a number.

attached patch corrects.

CommentFileSizeAuthor
batch_select.patch611 byteshunmonk
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Needs review » Fixed

Committed to CVS HEAD. Thanks.

yched’s picture

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

Er, quite true - that's because token used to be something else (an int)
The bug was hidden because the value was also inserted using %d until the 'remove sequences' patch got in.

What don't quite get is that I'm not getting any error, even without this patch.

Anyway. Needed fix, simple patch, RTBC.

yched’s picture

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

oops, cross posted with dries

Anonymous’s picture

(not verified) CreditAttribution: commented
Status: Fixed » Closed (fixed)