in the send_all function in notifications.event.inc the number of sent mails are calculated like this:
$sent = count($results['sent']);
$skip = count($results['skip']);
$success = count($results['success']);
However, while $sent and $skip are arrays with the subscription ID's, $success is not:
$results['success'] = count(array_filter($results['results']));
So $success = count($results['success']); should actually be $success = $results['success'];
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | notifications-n1637456-1.patch | 756 bytes | damienmckenna |
Comments
Comment #1
damienmckennaThe patch was built against the site root rather than the module's directory. This patch fixes that.
Comment #2
phenaproximaLooks like the correct fix, given what's in Notifications_Event::send_list().
Comment #4
Nafes commentedCommitted. Thank you, DamienMcKenna and phenaproxima!