Merge pull request #490 from eileenmcnaughton/540

Fix filter grouping
eileenmcnaughton · Sep 16, 2021 · 883d9df · 883d9df
2 parents fb6790f + f0fc18d
commit 883d9df
Show file tree

Hide file tree

Showing 2 changed files with 31 additions and 11 deletions.
diff --git a/CRM/Extendedreport/Form/Report/ActivityExtended.php b/CRM/Extendedreport/Form/Report/ActivityExtended.php
@@ -1,5 +1,8 @@
 <?php
 
+
+use CRM_Extendedreport_ExtensionUtil as E;
+
 /**
  * Class CRM_Extendedreport_Form_Report_ActivityExtended
  */
@@ -26,43 +29,56 @@ public function __construct() {
           'prefix' => '',
           'prefix_label' => 'Source Contact ::',
           'filters' => TRUE,
+          'grouping' => 'source',
+          'group_title' => E::ts('Source Contact'),
         ]
       ) + $this->getColumns(
         'Email',
         [
           'prefix' => '',
           'prefix_label' => 'Source Contact Email ::',
           'filters' => TRUE,
+          'grouping' => 'source',
+          'group_title' => E::ts('Source Contact'),
         ]
-      ) + $this->getColumns(        
+      ) + $this->getColumns(
         'Contact',
         [
           'prefix' => 'target_',
           'group_by' => TRUE,
           'prefix_label' => 'Target Contact ::',
           'filters' => TRUE,
+          'grouping' => 'target',
+          'group_title' => E::ts('Target Contact'),
         ]
       ) + $this->getColumns(
         'Email',
         [
           'prefix' => 'target_',
           'prefix_label' => 'Target Contact Email ::',
           'filters' => TRUE,
+          'grouping' => 'target',
+          'group_title' => E::ts('Target Contact'),
         ]
       ) + $this->getColumns(
         'Contact', [
           'prefix' => 'assignee_',
           'prefix_label' => 'Assignee Contact ::',
           'filters' => TRUE,
+          'grouping' => 'assignee',
+          'group_title' => E::ts('Assignee Contact'),
         ]
       ) + $this->getColumns(
         'Email',
         [
           'prefix' => 'assignee_',
           'prefix_label' => 'Assignee Contact Email ::',
           'filters' => TRUE,
+          'grouping' => 'assignee',
+          'group_title' => E::ts('Assignee Contact'),
         ]
       ) + $this->getColumns('Activity', ['group_by' => TRUE]);
+
     parent::__construct();
   }
 
@@ -85,8 +101,7 @@ public function from() {
        LEFT JOIN civicrm_case
          ON case_activity_civireport.case_id = civicrm_case.id ";
     }
-
-
+
     $this->joinEmailFromContact();
     $this->joinEmailFromContact('target_');
     $this->joinEmailFromContact('assignee_');

diff --git a/CRM/Extendedreport/Form/Report/ExtendedReport.php b/CRM/Extendedreport/Form/Report/ExtendedReport.php
@@ -1310,13 +1310,18 @@ public function addFilters() {
       $count = 1;
       foreach ($this->getMetadataByType($filterString) as $fieldName => $field) {
         $table = $field['table_key'];
+        $groupingKey = $field['group_title'] ?? '';
         if ($filterString === 'filters') {
-          $filterGroups[$table] = [
-            'group_title' => $this->_columns[$field['table_key']]['group_title'],
-            'use_accordian_for_field_selection' => TRUE,
-          ];
+          $filterGroups[$groupingKey]['group_title'] = $field['group_title'] ?? '';
+          if (isset($field['use_accordion_for_field_selection'])) {
+            $filterGroups[$groupingKey]['use_accordion_for_field_selection'] = $field['table_key']['use_accordion_for_field_selection'];
+          }
+          else {
+            $filterGroups[$groupingKey]['use_accordion_for_field_selection'] = TRUE;
+          }
         }
         $prefix = ($filterString === 'join_filters') ? 'join_filter_' : '';
+        $filterGroups[$groupingKey]['tables'][$table][$prefix . $fieldName] = $field;
         $filters[$table][$prefix . $fieldName] = $field;
         $this->addFilterFieldsToReport($field, $fieldName, $table, $count, $prefix);
       }
@@ -3206,14 +3211,14 @@ protected function buildColumns($specs, $tableName, $daoName = NULL, $tableAlias
     $columns[$tableName]['prefix_label'] = isset($options['prefix_label']) ? $options['prefix_label'] : '';
     $columns[$tableName]['is_required_for_acls'] = $options['is_required_for_acls'] ?? FALSE;
     if (isset($options['group_title'])) {
-      $groupTitle = $options['group_title'];
+      $columns[$tableName]['group_title'] = $options['group_title'];
+      $columns[$tableName]['grouping'] = $options['grouping'];
     }
     else {
-
       // We can make one up but it won't be translated....
-      $groupTitle = ucfirst(str_replace('_', ' ', str_replace('civicrm_', '', $tableName)));
+      $columns[$tableName]['group_title'] = ucfirst(str_replace('_', ' ', str_replace('civicrm_', '', $tableName)));
     }
-    $columns[$tableName]['group_title'] = $groupTitle;
+
     return $columns;
   }