In signup_status_admin_signup_count_change_batch(), processing the query results with the following code fails to process all rows.

$query = db_query("SELECT nid FROM {signup_log} WHERE status IN ($placeholders) GROUP BY nid", $status_cids);
while ($nid = db_result($query)) {
  $context['sandbox']['nodes'][] = $nid;
}

I believe this is due to db_result() being best suited for pulling one result from a query.

CommentFileSizeAuthor
#1 signup_status-622488.patch896 bytesmlsamuelson

Comments

mlsamuelson’s picture

Status: Active » Needs review
StatusFileSize
new896 bytes

Changing the code above to

$query = db_query("SELECT nid FROM {signup_log} WHERE status IN ($placeholders) GROUP BY nid", $status_cids);
while ($nid = db_fetch_object($query)) {
  $context['sandbox']['nodes'][] = $nid->nid;
}

resolves the issue and processes all result rows.

Patch attached.