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'];

Comments

damienmckenna’s picture

Issue summary: View changes
Parent issue: » #2507375: Plan for Notifications v7.x-1.0 release
StatusFileSize
new756 bytes

The patch was built against the site root rather than the module's directory. This patch fixes that.

phenaproxima’s picture

Status: Needs review » Reviewed & tested by the community

Looks like the correct fix, given what's in Notifications_Event::send_list().

  • Nafes committed e059336 on 7.x-1.x authored by DamienMcKenna
    Issue #1637456 by BarisW, DamienMcKenna: Wrong success count in logs.
    
Nafes’s picture

Status: Reviewed & tested by the community » Fixed

Committed. Thank you, DamienMcKenna and phenaproxima!

Status: Fixed » Closed (fixed)

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