<?php
/*
+--------------------------------------------------------------------+
- | CiviCRM version 4.5 |
+ | CiviCRM version 4.6 |
+--------------------------------------------------------------------+
| Copyright CiviCRM LLC (c) 2004-2014 |
+--------------------------------------------------------------------+
private static $_configureActionLinks;
private static $_onlineContributionLinks;
- private static $_links = NULL;
-
+ /**
+ * @var CRM_Utils_Pager
+ */
protected $_pager = NULL;
+ /**
+ * @var string
+ */
protected $_sortByCharacter;
/**
* Get the action links for this page.
*
- * @return array $_actionLinks
- *
- */ function &actionLinks() {
+ * @return array
+ */
+ function &actionLinks() {
// check if variable _actionsLinks is populated
if (!isset(self::$_actionLinks)) {
// helper variable for nicer formatting
/**
* Get the configure action links for this page.
*
- * @return array $_configureActionLinks
- *
+ * @return array
*/
- function &configureActionLinks() {
+ public function &configureActionLinks() {
// check if variable _actionsLinks is populated
if (!isset(self::$_configureActionLinks)) {
$urlString = 'civicrm/admin/contribute/';
/**
* Get the online contribution links.
*
- * @return array $_onlineContributionLinks.
- *
+ * @return array
*/
- function &onlineContributionLinks() {
+ public function &onlineContributionLinks() {
if (!isset(self::$_onlineContributionLinks)) {
$urlString = 'civicrm/contribute/transact';
$urlParams = 'reset=1&id=%%id%%';
/**
* Get the contributions links.
*
- * @return array $_contributionLinks
- *
+ * @return array
*/
- function &contributionLinks() {
+ public function &contributionLinks() {
if (!isset(self::$_contributionLinks)) {
//get contribution dates.
$dates = CRM_Contribute_BAO_Contribution::getContributionDates();
* type of action and executes that action.
* Finally it calls the parent's run method.
*
- * @return void
- * @access public
- *
+ * @return mixed
*/
- function run() {
+ public function run() {
// get the requested action
$action = CRM_Utils_Request::retrieve('action', 'String',
// default to 'browse'
);
// set breadcrumb to append to 2nd layer pages
- $breadCrumb = array(array('title' => ts('Manage Contribution Pages'),
+ $breadCrumb = array(array(
+ 'title' => ts('Manage Contribution Pages'),
'url' => CRM_Utils_System::url(CRM_Utils_System::currentPath(),
'reset=1'
),
}
}
elseif ($action & CRM_Core_Action::COPY) {
+ // @todo Unused local variable can be safely removed.
+ // But are there any side effects of CRM_Core_Session::singleton() that we
+ // need to preserve?
$session = CRM_Core_Session::singleton();
CRM_Core_Session::setStatus(ts('A copy of the contribution page has been created'), ts('Successfully Copied'), 'success');
$this->copy();
* all the fields in the page
*
* @return void
- * @access public
*/
- function copy() {
+ public function copy() {
$gid = CRM_Utils_Request::retrieve('gid', 'Positive',
$this, TRUE, 0, 'GET'
);
/**
* Browse all contribution pages
*
- * @param null $action
- *
- * @return void
- * @access public
+ * @param mixed $action
+ * Unused parameter.
* @static
*/
- function browse($action = NULL) {
+ public function browse($action = NULL) {
$this->_sortByCharacter = CRM_Utils_Request::retrieve('sortByCharacter',
'String',
$this
);
+ // @todo Unused local variable can be safely removed.
+ // But are there any side effects of CRM_Utils_Request::retrieve() that we
+ // need to preserve?
$createdId = CRM_Utils_Request::retrieve('cid', 'Positive',
$this, FALSE, 0
);
$action += array_sum(array_keys(self::contributionLinks()));
if ($dao->is_active) {
- $action -= CRM_Core_Action::ENABLE;
+ $action -= (int) CRM_Core_Action::ENABLE;
}
else {
- $action -= CRM_Core_Action::DISABLE;
+ $action -= (int) CRM_Core_Action::DISABLE;
}
//CRM-4418
if (!$allowToDelete) {
- $action -= CRM_Core_Action::DELETE;
+ $action -= (int) CRM_Core_Action::DELETE;
}
//build the configure links.
}
}
- function search() {
- if (isset($this->_action) &
- (CRM_Core_Action::ADD |
+ public function search() {
+ if (isset($this->_action) & (CRM_Core_Action::ADD |
CRM_Core_Action::UPDATE |
CRM_Core_Action::DELETE
)
*
* @return int|string
*/
- function whereClause(&$params, $sortBy = TRUE) {
+ public function whereClause(&$params, $sortBy = TRUE) {
+ // @todo Unused local variable can be safely removed.
$values = $clauses = array();
$title = $this->get('title');
$createdId = $this->get('cid');
}
}
- $value = $this->get( 'financial_type_id' );
+ $value = $this->get( 'financial_type_id' );
$val = array();
if ($value) {
if (is_array($value)) {
}
$type = implode(',', $val);
}
-
- $clauses[] = "financial_type_id IN ({$type})";
+ // @todo Variable 'type' might not have been defined.
+ $clauses[] = "financial_type_id IN ({$type})";
}
- if ($sortBy &&
- $this->_sortByCharacter !== NULL
- ) {
+ if ($sortBy && $this->_sortByCharacter !== NULL) {
$clauses[] = "title LIKE '" . strtolower(CRM_Core_DAO::escapeWildCardString($this->_sortByCharacter)) . "%'";
}
- $campainIds = $this->get('campaign_id');
- if (!CRM_Utils_System::isNull($campainIds)) {
- if (!is_array($campainIds)) {
- $campaignIds = array($campaignIds);
- }
- $clauses[] = '( campaign_id IN ( ' . implode(' , ', array_values($campainIds)) . ' ) )';
+ $campaignIds = $this->getCampaignIds();
+ if (count($campaignIds) >= 1) {
+ $clauses[] = '( campaign_id IN ( ' . implode(' , ', $campaignIds) . ' ) )';
}
if (empty($clauses)) {
return implode(' AND ', $clauses);
}
+ /**
+ * Gets the campaign ids from the session.
+ *
+ * @return int[]
+ */
+ function getCampaignIds() {
+ // The unfiltered value from the session cannot be trusted, it needs to be
+ // processed to get a clean array of positive integers.
+ $ids = array();
+ foreach ((array) $this->get('campaign_id') as $id) {
+ if ((string) (int) $id === (string) $id && $id > 0) {
+ $ids[] = $id;
+ }
+ }
+ return $ids;
+ }
+
/**
* @param $whereClause
* @param array $whereParams
$query = "
SELECT count(id)
- FROM civicrm_contribution_page
- WHERE $whereClause";
+FROM civicrm_contribution_page
+WHERE $whereClause";
$params['total'] = CRM_Core_DAO::singleValueQuery($query, $whereParams);
* @param $whereClause
* @param array $whereParams
*/
- function pagerAtoZ($whereClause, $whereParams) {
+ public function pagerAtoZ($whereClause, $whereParams) {
$query = "
- SELECT DISTINCT UPPER(LEFT(title, 1)) as sort_name
- FROM civicrm_contribution_page
- WHERE $whereClause
- ORDER BY LEFT(title, 1)
+SELECT DISTINCT UPPER(LEFT(title, 1)) as sort_name
+FROM civicrm_contribution_page
+WHERE $whereClause
+ORDER BY LEFT(title, 1)
";
$dao = CRM_Core_DAO::executeQuery($query, $whereParams);
}
/**
- * @param $sectionsInfo
+ * @param array $sectionsInfo
*
* @return array
*/
- function formatConfigureLinks($sectionsInfo) {
- //build the formatted configure links.
+ public function formatConfigureLinks($sectionsInfo) {
+ // build the formatted configure links.
$formattedConfLinks = self::configureActionLinks();
foreach ($formattedConfLinks as $act => & $link) {
$sectionName = CRM_Utils_Array::value('uniqueName', $link);
continue;
}
- $classes = array();
- if (isset($link['class'])) {
- $classes = $link['class'];
- }
-
if (empty($sectionsInfo[$sectionName])) {
$classes = array();
if (isset($link['class'])) {
return $formattedConfLinks;
}
}
-
projects / civicrm-core.git / blobdiff