From e2ef9c65ac9449029393bb97559c242ba075b6e4 Mon Sep 17 00:00:00 2001
From: Pradeep Nayak <pradpnayak@gmail.com>
Date: Tue, 24 Jul 2018 14:02:35 +0100
Subject: [PATCH] dev/core/issues/277, Fixed count to include only live
 recurring contribution

---
 CRM/Contribute/Page/Tab.php           | 12 +++++++++---
 templates/CRM/Contribute/Page/Tab.tpl |  2 +-
 2 files changed, 10 insertions(+), 4 deletions(-)

diff --git a/CRM/Contribute/Page/Tab.php b/CRM/Contribute/Page/Tab.php
index b5c20a9e0c..5431bc8daf 100644
--- a/CRM/Contribute/Page/Tab.php
+++ b/CRM/Contribute/Page/Tab.php
@@ -166,13 +166,14 @@ class CRM_Contribute_Page_Tab extends CRM_Core_Page {
    * Get all the recurring contribution information and assign to the template
    */
   private function addRecurringContributionsBlock() {
-    $activeContributions = $this->getActiveRecurringContributions();
-    $inactiveRecurringContributions = $this->getInactiveRecurringContributions();
+    list($activeContributions, $activeContributionsCount) = $this->getActiveRecurringContributions();
+    list($inactiveRecurringContributions, $inactiveContributionsCount) = $this->getInactiveRecurringContributions();
 
     if (!empty($activeContributions) || !empty($inactiveRecurringContributions)) {
       // assign vars to templates
       $this->assign('action', $this->_action);
       $this->assign('activeRecurRows', $activeContributions);
+      $this->assign('activeContributionsCount', $activeContributionsCount);
       $this->assign('inactiveRecurRows', $inactiveRecurringContributions);
       $this->assign('recur', TRUE);
     }
@@ -228,11 +229,16 @@ class CRM_Contribute_Page_Tab extends CRM_Core_Page {
    * @return mixed
    */
   private function buildRecurringContributionsArray($recurContributions) {
+    $liveRecurringContributionCount = 0;
     foreach ($recurContributions as $recurId => $recurDetail) {
       $action = array_sum(array_keys($this->recurLinks($recurId)));
       // no action allowed if it's not active
       $recurContributions[$recurId]['is_active'] = (!CRM_Contribute_BAO_Contribution::isContributionStatusNegative($recurDetail['contribution_status_id']));
 
+      if (empty($recurDetail['is_test'])) {
+        $liveRecurringContributionCount++;
+      }
+
       // Get the name of the payment processor
       if (!empty($recurDetail['payment_processor_id'])) {
         $recurContributions[$recurId]['payment_processor'] = CRM_Financial_BAO_PaymentProcessor::getPaymentProcessorName($recurDetail['payment_processor_id']);
@@ -265,7 +271,7 @@ class CRM_Contribute_Page_Tab extends CRM_Core_Page {
       }
     }
 
-    return $recurContributions;
+    return [$recurContributions, $liveRecurringContributionCount];
   }
 
   /**
diff --git a/templates/CRM/Contribute/Page/Tab.tpl b/templates/CRM/Contribute/Page/Tab.tpl
index f38a200afd..f4617bfd89 100644
--- a/templates/CRM/Contribute/Page/Tab.tpl
+++ b/templates/CRM/Contribute/Page/Tab.tpl
@@ -51,7 +51,7 @@
           </li>
           <li id="tab_recurring" class="crm-tab-button ui-corner-all ui-tabs-tab ui-corner-top ui-state-default ui-tab">
             <a href="#recurring-subtab" title="{ts}Recurring Contributions{/ts}">
-              {ts}Recurring Contributions{/ts} <em>{$activeRecurRows|@count}</em>
+              {ts}Recurring Contributions{/ts} <em>{$activeContributionsCount}</em>
             </a>
           </li>
         </ul>
-- 
2.25.1