X-Git-Url: https://vcs.fsf.org/?a=blobdiff_plain;f=CRM%2FContact%2FBAO%2FQuery.php;h=5247434ab824ea3793a0b7e5ac44c2168b814b4d;hb=2027a5085d14bf284bda113e4b340c48ff59a981;hp=0a1ba6f5910bceeefe73cad78239b222d5d31f7b;hpb=2995cd1ca2e16cfaf5a82ef987cb84d40e21cf93;p=civicrm-core.git diff --git a/CRM/Contact/BAO/Query.php b/CRM/Contact/BAO/Query.php index 0a1ba6f591..5247434ab8 100644 --- a/CRM/Contact/BAO/Query.php +++ b/CRM/Contact/BAO/Query.php @@ -5109,16 +5109,6 @@ civicrm_relationship.start_date > {$today} $summary['total']['currencyCount'] = count($summary['total']['median']); - if (!empty($summary['total']['amount'])) { - $summary['total']['amount'] = implode(', ', $summary['total']['amount']); - $summary['total']['avg'] = implode(', ', $summary['total']['avg']); - $summary['total']['mode'] = implode(', ', $summary['total']['mode']); - $summary['total']['median'] = implode(', ', $summary['total']['median']); - } - else { - $summary['total']['amount'] = $summary['total']['avg'] = $summary['total']['median'] = 0; - } - $this->addBasicCancelStatsToSummary($summary, $where, $from); return $summary; @@ -6544,7 +6534,8 @@ AND displayRelType.is_active = 1 * @return array */ protected function addBasicStatsToSummary(&$summary, $where, $from) { - $summary['total']['count'] = $summary['total']['amount'] = $summary['total']['avg'] = "n/a"; + $summary['total']['count'] = 0; + $summary['total']['amount'] = $summary['total']['avg'] = []; $query = " SELECT COUNT( conts.total_amount ) as total_count, @@ -6562,13 +6553,18 @@ AND displayRelType.is_active = 1 $dao = CRM_Core_DAO::executeQuery($query); - $summary['total']['count'] = 0; - $summary['total']['amount'] = $summary['total']['avg'] = []; while ($dao->fetch()) { $summary['total']['count'] += $dao->total_count; $summary['total']['amount'][] = CRM_Utils_Money::format($dao->total_amount, $dao->currency); $summary['total']['avg'][] = CRM_Utils_Money::format($dao->total_avg, $dao->currency); } + if (!empty($summary['total']['amount'])) { + $summary['total']['amount'] = implode(', ', $summary['total']['amount']); + $summary['total']['avg'] = implode(', ', $summary['total']['avg']); + } + else { + $summary['total']['amount'] = $summary['total']['avg'] = $summary['total']['median'] = 0; + } return $summary; } @@ -6603,7 +6599,8 @@ AND displayRelType.is_active = 1 ) as conts GROUP BY currency"; - $summary['total']['mode'] = CRM_Contribute_BAO_Contribution::computeStats('mode', $modeSQL); + $mode = CRM_Contribute_BAO_Contribution::computeStats('mode', $modeSQL); + $summary['total']['mode'] = implode(', ', (array) $mode); } /** @@ -6621,7 +6618,8 @@ AND displayRelType.is_active = 1 */ protected function addMedianToStats(&$summary, $where, $from) { $medianSQL = "{$from} {$where} AND civicrm_contribution.contribution_status_id = 1 "; - $summary['total']['median'] = CRM_Contribute_BAO_Contribution::computeStats('median', $medianSQL, 'civicrm_contribution'); + $median = CRM_Contribute_BAO_Contribution::computeStats('median', $medianSQL, 'civicrm_contribution'); + $summary['total']['median'] = implode(', ', (array) $median); } /**