From: Deepak Srivastava Date: Fri, 31 Jan 2014 07:37:59 +0000 (+0530) Subject: CRM-14128 - Participant Count Report offers 'is not role' criteria - but treats it... X-Git-Url: https://vcs.fsf.org/?a=commitdiff_plain;h=e8b2628d3afb54f081b3678d5ab8b2113b82ea19;p=civicrm-core.git CRM-14128 - Participant Count Report offers 'is not role' criteria - but treats it the same as 'is role' http://issues.civicrm.org/jira/browse/CRM-14128 --- diff --git a/CRM/Report/Form/Event/ParticipantListCount.php b/CRM/Report/Form/Event/ParticipantListCount.php index e0ceb65370..bf8479231d 100644 --- a/CRM/Report/Form/Event/ParticipantListCount.php +++ b/CRM/Report/Form/Event/ParticipantListCount.php @@ -390,58 +390,9 @@ class CRM_Report_Form_Event_ParticipantListCount extends CRM_Report_Form_Event { ON {$this->_aliases['civicrm_participant']}.id ={$this->_aliases['civicrm_line_item']}.entity_id AND {$this->_aliases['civicrm_line_item']}.entity_table = 'civicrm_participant'"; } - function where() { - $clauses = array(); - foreach ($this->_columns as $tableName => $table) { - if (array_key_exists('filters', $table)) { - foreach ($table['filters'] as $fieldName => $field) { - $clause = NULL; - if (CRM_Utils_Array::value('type', $field) & CRM_Utils_Type::T_DATE) { - $relative = CRM_Utils_Array::value("{$fieldName}_relative", $this->_params); - $from = CRM_Utils_Array::value("{$fieldName}_from", $this->_params); - $to = CRM_Utils_Array::value("{$fieldName}_to", $this->_params); - - if ($relative || $from || $to) { - $clause = $this->dateClause($field['name'], $relative, $from, $to, $field['type']); - } - } - else { - $op = CRM_Utils_Array::value("{$fieldName}_op", $this->_params); - - if ($fieldName == 'rid') { - $value = CRM_Utils_Array::value("{$fieldName}_value", $this->_params); - if (!empty($value)) { - $clause = "( {$field['dbAlias']} REGEXP '[[:<:]]" . implode('[[:>:]]|[[:<:]]', $value) . "[[:>:]]' )"; - } - $op = NULL; - } - - if ($op) { - $clause = $this->whereClause($field, - $op, - CRM_Utils_Array::value("{$fieldName}_value", $this->_params), - CRM_Utils_Array::value("{$fieldName}_min", $this->_params), - CRM_Utils_Array::value("{$fieldName}_max", $this->_params) - ); - } - } - - if (!empty($clause)) { - $clauses[] = $clause; - } - } - } - } - - if (empty($clauses)) { - $this->_where = "WHERE {$this->_aliases['civicrm_participant']}.is_test = 0 "; - } - else { - $this->_where = "WHERE {$this->_aliases['civicrm_participant']}.is_test = 0 AND " . implode(' AND ', $clauses); - } - if ($this->_aclWhere) { - $this->_where .= " AND {$this->_aclWhere} "; - } + function storeWhereHavingClauseArray() { + parent::storeWhereHavingClauseArray(); + $this->_whereClauses[] = "{$this->_aliases['civicrm_participant']}.is_test = 0"; } function groupBy() {