From 7ff385e8d99be4426621694965c00ae8dbb2b607 Mon Sep 17 00:00:00 2001 From: Jamie McClelland Date: Mon, 26 Aug 2013 13:30:07 -0400 Subject: [PATCH] ensure correct open, click, etc. statistics See http://issues.civicrm.org/jira/browse/CRM-10100 We're counting opens for the same message more than once. By counting the event_queue_id we ensure only one count per message. --- CRM/Report/Form/Mailing/Summary.php | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/CRM/Report/Form/Mailing/Summary.php b/CRM/Report/Form/Mailing/Summary.php index a6ff9f209f..67f955f0b8 100644 --- a/CRM/Report/Form/Mailing/Summary.php +++ b/CRM/Report/Form/Mailing/Summary.php @@ -146,7 +146,7 @@ class CRM_Report_Form_Mailing_Summary extends CRM_Report_Form { 'dao' => 'CRM_Mailing_DAO_Mailing', 'fields' => array( 'delivered_count' => array( - 'name' => 'id', + 'name' => 'event_queue_id', 'title' => ts('Delivered'), ), 'accepted_rate' => array( @@ -164,7 +164,7 @@ class CRM_Report_Form_Mailing_Summary extends CRM_Report_Form { 'dao' => 'CRM_Mailing_DAO_Mailing', 'fields' => array( 'bounce_count' => array( - 'name' => 'id', + 'name' => 'event_queue_id', 'title' => ts('Bounce'), ), 'bounce_rate' => array( @@ -182,7 +182,7 @@ class CRM_Report_Form_Mailing_Summary extends CRM_Report_Form { 'dao' => 'CRM_Mailing_DAO_Mailing', 'fields' => array( 'open_count' => array( - 'name' => 'id', + 'name' => 'event_queue_id', 'title' => ts('Opened'), ), 'open_rate' => array( @@ -200,7 +200,7 @@ class CRM_Report_Form_Mailing_Summary extends CRM_Report_Form { 'dao' => 'CRM_Mailing_DAO_Mailing', 'fields' => array( 'click_count' => array( - 'name' => 'id', + 'name' => 'event_queue_id', 'title' => ts('Clicks'), ), 'CTR' => array( -- 2.25.1