From 7c006637c5e5d6dea8481dff55e849ab20c90928 Mon Sep 17 00:00:00 2001
From: kurund <kurund@civicrm.org>
Date: Wed, 31 Jul 2013 03:48:32 +0530
Subject: [PATCH] CRM-12814, fixed the total issue, code cleanup and notice
 fixes

----------------------------------------
* CRM-12814: mailings tab slow performance
  http://issues.civicrm.org/jira/browse/CRM-12814
---
 CRM/Contact/BAO/Contact.php |  5 +----
 CRM/Mailing/BAO/Mailing.php | 23 +++++++++++------------
 CRM/Mailing/Page/AJAX.php   |  5 ++---
 CRM/Mailing/Page/Tab.php    |  1 -
 4 files changed, 14 insertions(+), 20 deletions(-)

diff --git a/CRM/Contact/BAO/Contact.php b/CRM/Contact/BAO/Contact.php
index 2edbefdf59..b2f1eefe00 100644
--- a/CRM/Contact/BAO/Contact.php
+++ b/CRM/Contact/BAO/Contact.php
@@ -2448,10 +2448,7 @@ AND       civicrm_openid.is_primary = 1";
         return CRM_Activity_BAO_Activity::getActivitiesCount($input);
 
       case 'mailing':
-        $params = array(
-          'contact_id' => $contactId,
-          'admin' => FALSE,
-        );
+        $params = array('contact_id' => $contactId);
         return CRM_Mailing_BAO_Mailing::getContactMailingsCount($params);
 
       default:
diff --git a/CRM/Mailing/BAO/Mailing.php b/CRM/Mailing/BAO/Mailing.php
index a38ddbf2f4..ee75e2aff5 100644
--- a/CRM/Mailing/BAO/Mailing.php
+++ b/CRM/Mailing/BAO/Mailing.php
@@ -2762,7 +2762,7 @@ AND        m.id = %1
     $params['total'] = CRM_Mailing_BAO_Mailing::getContactMailingsCount($params);
 
     //CRM-12814
-    if ( !empty($mailings) ) {
+    if (!empty($mailings)) {
       $openCounts = CRM_Mailing_Event_BAO_Opened::getMailingTotalCount(array_keys($mailings));
       $clickCounts = CRM_Mailing_Event_BAO_TrackableURLOpen::getMailingTotalCount(array_keys($mailings));
     }
@@ -2774,20 +2774,21 @@ AND        m.id = %1
       $contactMailings[$mailingId]['start_date'] = CRM_Utils_Date::customFormat($values['start_date']);
       $contactMailings[$mailingId]['recipients'] = CRM_Utils_System::href(ts('(recipients)'), 'civicrm/mailing/report/event',
         "mid={$values['mailing_id']}&reset=1&cid={$params['contact_id']}&event=queue&context=mailing");
+
       $contactMailings[$mailingId]['mailing_creator'] = CRM_Utils_System::href(
-          $values['creator_name'],
-          'civicrm/contact/view',
-          "reset=1&cid={$values['creator_id']}");
+        $values['creator_name'],
+        'civicrm/contact/view',
+        "reset=1&cid={$values['creator_id']}");
 
       //CRM-12814
-      $contactMailings[$mailingId]['openstats'] = "Opens: ".$openCounts[$values['mailing_id']].
-        "<br />Clicks: ".$clickCounts[$values['mailing_id']];
+      $contactMailings[$mailingId]['openstats'] = ts('Opens') . ': ' . $openCounts[$values['mailing_id']] .
+        '<br />' . ts('Clicks') . ': ' . $clickCounts[$values['mailing_id']];
 
       $actionLinks = array(
         CRM_Core_Action::VIEW => array(
-          'name'  => ts('View'),
-          'url'   => 'civicrm/mailing/view',
-          'qs'    => "reset=1&id={$values['mailing_id']}",
+          'name' => ts('View'),
+          'url' => 'civicrm/mailing/view',
+          'qs' => "reset=1&id={$values['mailing_id']}",
           'title' => ts('View Mailing'),
           'class' => 'crm-mailing-view',
         ),
@@ -2837,9 +2838,7 @@ AND        m.id = %1
    */
   static public function getContactMailingsCount(&$params) {
     $params['version'] = 3;
-
-    $result = civicrm_api('MailingContact', 'getcount', $params);
-    return $result['values']['count'];
+    return civicrm_api('MailingContact', 'getcount', $params);
   }
 }
 
diff --git a/CRM/Mailing/Page/AJAX.php b/CRM/Mailing/Page/AJAX.php
index 5810cb890f..73c2ae8d21 100644
--- a/CRM/Mailing/Page/AJAX.php
+++ b/CRM/Mailing/Page/AJAX.php
@@ -41,7 +41,7 @@ class CRM_Mailing_Page_AJAX {
   /**
    * Function to fetch the template text/html messages
    */
-  static function template() {
+  public static function template() {
     $templateId = CRM_Utils_Type::escape($_POST['tid'], 'Integer');
 
     $messageTemplate = new CRM_Core_DAO_MessageTemplates();
@@ -63,9 +63,8 @@ class CRM_Mailing_Page_AJAX {
   /**
    * Function to retrieve contact mailings
    */
-  function getContactMailings() {
+  public static function getContactMailings() {
     $contactID = CRM_Utils_Type::escape($_GET['contact_id'], 'Integer');
-    //$context = CRM_Utils_Type::escape(CRM_Utils_Array::value('context', $_GET), 'String');
 
     $sortMapper = array(
       0 => 'subject', 1 => 'creator_name', 2 => '', 3 => 'start_date', 4 => '', 5 => 'links',
diff --git a/CRM/Mailing/Page/Tab.php b/CRM/Mailing/Page/Tab.php
index 68d135aca9..8abb75ab4d 100644
--- a/CRM/Mailing/Page/Tab.php
+++ b/CRM/Mailing/Page/Tab.php
@@ -64,7 +64,6 @@ class CRM_Mailing_Page_Tab extends CRM_Contact_Page_View {
    * @access public
    */
   function preProcess() {
-    $context       = CRM_Utils_Request::retrieve('context', 'String', $this);
     $this->_action = CRM_Utils_Request::retrieve('action', 'String', $this, FALSE, 'browse');
     $this->_id     = CRM_Utils_Request::retrieve('id', 'Positive', $this);
 
-- 
2.25.1