on several places in the module there is the following code:
implode(',', array_fill(0, count($gids), '%d'))

the correct way to implode the $gids is the following one:
implode(',', $gids)

Comments

MGN’s picture

MGN commented
Category: bug » support
Priority: Critical » Normal

It looks like this code was written before the db_placeholders api function was developed.

I think the correct approach would be to use the api function : db_placeholders($gids, 'int').

IMHO This certainly is not a critical bug report - more of a coding preference.