ensure correct open, click, etc. statistics
authorJamie McClelland <jm@mayfirst.org>
Mon, 26 Aug 2013 17:30:07 +0000 (13:30 -0400)
committerJamie McClelland <jm@mayfirst.org>
Tue, 4 Aug 2015 14:59:51 +0000 (10:59 -0400)
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

index a6ff9f209fa25621a71a2489c6d5502758b420ac..67f955f0b892300d7bc78286ed81078dd01d4dfb 100644 (file)
@@ -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(