From 803e39be78a159140a5961625cc18c0cb70b5877 Mon Sep 17 00:00:00 2001 From: Edsel Date: Tue, 17 Mar 2015 17:22:48 +0530 Subject: [PATCH] CIVI-28 Added permission check for Contribution Summary Report --- CRM/Financial/BAO/FinancialType.php | 3 ++- CRM/Report/Form/Contribute/Summary.php | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/CRM/Financial/BAO/FinancialType.php b/CRM/Financial/BAO/FinancialType.php index 36a6902522..5a67acf19e 100644 --- a/CRM/Financial/BAO/FinancialType.php +++ b/CRM/Financial/BAO/FinancialType.php @@ -207,7 +207,7 @@ class CRM_Financial_BAO_FinancialType extends CRM_Financial_DAO_FinancialType { $permissions['administer CiviCRM Financial Types'] = $prefix . ts('administer CiviCRM Financial Types'); } - public static function getAvailableFinancialTypes(&$financialTypes, $action) { + public static function getAvailableFinancialTypes(&$financialTypes = NULL, $action = 'view') { $financialTypes = CRM_Contribute_PseudoConstant::financialType(); foreach ($financialTypes as $finTypeId => $type) { @@ -215,5 +215,6 @@ class CRM_Financial_BAO_FinancialType extends CRM_Financial_DAO_FinancialType { unset($financialTypes[$finTypeId]); } } + return $financialTypes; } } diff --git a/CRM/Report/Form/Contribute/Summary.php b/CRM/Report/Form/Contribute/Summary.php index 077134447e..1005fbc1a3 100644 --- a/CRM/Report/Form/Contribute/Summary.php +++ b/CRM/Report/Form/Contribute/Summary.php @@ -163,7 +163,7 @@ class CRM_Report_Form_Contribute_Summary extends CRM_Report_Form { 'financial_type_id' => array( 'title' => ts('Financial Type'), 'operatorType' => CRM_Report_Form::OP_MULTISELECT, - 'options' => CRM_Contribute_PseudoConstant::financialType(), + 'options' => CRM_Financial_BAO_FinancialType::getAvailableFinancialTypes(), 'type' => CRM_Utils_Type::T_INT, ), 'contribution_page_id' => array( -- 2.25.1