From bfda95210477891cc2c94a4af3631dbe05a51b3f Mon Sep 17 00:00:00 2001 From: Coleman Watts Date: Fri, 22 Aug 2014 09:45:15 +0100 Subject: [PATCH] CRM-14996 - Fix display bug in adv search --- CRM/Contact/BAO/Query.php | 3 +-- CRM/Contact/Form/Search/Criteria.php | 5 +++-- CRM/Core/BAO/Address.php | 2 +- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/CRM/Contact/BAO/Query.php b/CRM/Contact/BAO/Query.php index a636a96041..0781e4d1e2 100644 --- a/CRM/Contact/BAO/Query.php +++ b/CRM/Contact/BAO/Query.php @@ -3797,9 +3797,8 @@ WHERE id IN ( $groupIDs ) } $stateClause = "civicrm_address.state_province_id $op (" . implode(',', $value) . ')'; - $stateProvince = CRM_Core_PseudoConstant::stateProvince(); foreach ($value as $id) { - $names[] = CRM_Utils_Array::value($id, $stateProvince); + $names[] = CRM_Core_PseudoConstant::stateProvince($id, FALSE); } } else { diff --git a/CRM/Contact/Form/Search/Criteria.php b/CRM/Contact/Form/Search/Criteria.php index 598cd72b80..d550a53311 100644 --- a/CRM/Contact/Form/Search/Criteria.php +++ b/CRM/Contact/Form/Search/Criteria.php @@ -305,6 +305,7 @@ class CRM_Contact_Form_Search_Criteria { $parseStreetAddress = CRM_Utils_Array::value('street_address_parsing', $addressOptions, 0); $form->assign('parseStreetAddress', $parseStreetAddress); + $stateCountryMap = NULL; foreach ($elements as $name => $v) { list($title, $attributes, $select, $multiSelect) = $v; @@ -324,11 +325,11 @@ class CRM_Contact_Form_Search_Criteria { } if ($select) { - $stateCountryMap[] = array( + $stateCountryMap = array(array( 'state_province' => 'state_province', 'country' => 'country', 'county' => 'county', - ); + )); if ($select == 'stateProvince') { if (!empty($formValues['country'])) { $selectElements = array('' => ts('- select -')) + CRM_Core_PseudoConstant::stateProvinceForCountry($formValues['country']); diff --git a/CRM/Core/BAO/Address.php b/CRM/Core/BAO/Address.php index a7e18e3ac0..7d52be6219 100644 --- a/CRM/Core/BAO/Address.php +++ b/CRM/Core/BAO/Address.php @@ -659,7 +659,7 @@ ORDER BY civicrm_address.is_primary DESC, civicrm_address.location_type_id DESC, * @param $stateCountryMap * @param null $defaults */ - static function addStateCountryMap(&$stateCountryMap, $defaults = NULL) { + static function addStateCountryMap($stateCountryMap, $defaults = NULL) { // first fix the statecountry map if needed if (empty($stateCountryMap)) { return; -- 2.25.1