Merge pull request #18719 from JMAConsulting/core_2105
authorEileen McNaughton <emcnaughton@wikimedia.org>
Fri, 9 Oct 2020 20:56:04 +0000 (09:56 +1300)
committerGitHub <noreply@github.com>
Fri, 9 Oct 2020 20:56:04 +0000 (09:56 +1300)
core#2105: Groups children now get shown with SPAN CSS error

CRM/Contact/Form/Edit/TagsAndGroups.php
CRM/Contact/Form/Search/Basic.php
CRM/Contact/Form/Search/Criteria.php

index 9299246d9beb57312e3c9bd8172f874e589c74e8..de2084c0a2a2432c40ffa80156d560cb6192cb72 100644 (file)
@@ -88,7 +88,7 @@ class CRM_Contact_Form_Edit_TagsAndGroups {
       }
 
       if ($groupID || !empty($group)) {
-        $groups = CRM_Contact_BAO_Group::getGroupsHierarchy($ids);
+        $groups = CRM_Contact_BAO_Group::getGroupsHierarchy($ids, NULL, '- ');
 
         $attributes['skiplabel'] = TRUE;
         $elements = [];
index 6deec9e2a44dd3f4f396de3e6a80af0bdca2132b..537e19e0bdef1db6b52fcb814470e9d06bbc5772 100644 (file)
@@ -49,7 +49,7 @@ class CRM_Contact_Form_Search_Basic extends CRM_Contact_Form_Search {
 
     // add select for groups
     // Get hierarchical listing of groups, respecting ACLs for CRM-16836.
-    $groupHierarchy = CRM_Contact_BAO_Group::getGroupsHierarchy($this->_group, NULL, '&nbsp;&nbsp;');
+    $groupHierarchy = CRM_Contact_BAO_Group::getGroupsHierarchy($this->_group, NULL, '');
     if (!empty($searchOptions['groups'])) {
       $this->addField('group', [
         'entity' => 'group_contact',
index bd2d5829da5c6efc18760951f28fadd76716e4f4..6bcad01d86d63464a215fcf15a5a97d84ee7b42c 100644 (file)
@@ -42,7 +42,7 @@ class CRM_Contact_Form_Search_Criteria {
       // multiselect for groups
       if ($form->_group) {
         // Arrange groups into hierarchical listing (child groups follow their parents and have indentation spacing in title)
-        $groupHierarchy = CRM_Contact_BAO_Group::getGroupsHierarchy($form->_group, NULL, '&nbsp;&nbsp;', TRUE);
+        $groupHierarchy = CRM_Contact_BAO_Group::getGroupsHierarchy($form->_group, NULL, '', TRUE);
 
         $form->add('select', 'group', ts('Groups'), $groupHierarchy, FALSE,
          ['id' => 'group', 'multiple' => 'multiple', 'class' => 'crm-select2']