fix group filtering

COmanage · Mar 4, 2024 · ed63183 · ed63183
1 parent 3492f1f
commit ed63183
Show file tree

Hide file tree

Showing 5 changed files with 31 additions and 16 deletions.
diff --git a/app/src/Lib/Traits/SearchFilterTrait.php b/app/src/Lib/Traits/SearchFilterTrait.php
@@ -90,14 +90,29 @@ public function getSearchableAttributes(string $controller, string $vv_tz=null):
           $fieldIsActive = $filterConfig[$column]['active'];
         }
       }
-      
-      $this->searchFilters[$column] = [
+
+      $attribute = [
         'type' => $type,
         'label' => \App\Lib\Util\StringUtilities::columnKey($modelname, $column, $vv_tz, true),
         'active' => $fieldIsActive,
         'order' => 99 // this is the default
       ];
 
+      // The column name should always go first, then the description will follow.
+      if($column == 'name') {
+        $this->searchFilters = [ $column => $attribute, ...$this->searchFilters];
+      } else if ($column == 'description') {
+        if(isset($this->searchFilters['name'])) {
+          $this->searchFilters = array_slice($this->searchFilters, 0, 1)
+            + [ $column => $attribute ]
+            + array_slice($this->searchFilters, 1);
+        } else {
+          $this->searchFilters = [ $column => $attribute, ...$this->searchFilters];
+        }
+      } else {
+        $this->searchFilters[$column] = $attribute;
+      }
+
       // For the date fields we search ranges
       if($type === 'timestamp') {
         $this->searchFilters[$column]['alias'][] = $column . '_starts_at';

diff --git a/app/src/Lib/Traits/TableMetaTrait.php b/app/src/Lib/Traits/TableMetaTrait.php
@@ -84,7 +84,8 @@ protected function filterMetadataFields() {
       'source_name_id',
       'source_pronoun_id',
       'source_telephone_number_id',
-      'source_url_id'
+      'source_url_id',
+      'owners_group_id'
     ];
 
     $newa = array();

diff --git a/app/src/Model/Table/GroupsTable.php b/app/src/Model/Table/GroupsTable.php
@@ -142,6 +142,15 @@ public function initialize(array $config): void {
         'class' => 'GroupTypeEnum'
       ]
     ]);
+
+    $this->setFilterConfig([
+      'identifier' => [
+        'type' => 'relatedModel',
+        'model' => 'Identifier',
+        'active' => true,
+        'order' => 4
+      ]
+    ]);
 
     $this->setPermissions([
   // XXX update for couAdmins, etc

diff --git a/app/src/Model/Table/PeopleTable.php b/app/src/Model/Table/PeopleTable.php
@@ -564,7 +564,7 @@ public function recalculateStatus(int $id): ?string {
         // Locked status cannot be recalculated. This isn't an error, per se.
         if($person->status == StatusEnum::Locked) {
           $this->llog('trace', 'Not recalculating Person " . $person->id . " status since the record is locked');
-          return $curStatus;
+          return $person->status;
         }
 
         // Update the Person status

diff --git a/app/templates/element/filter.php b/app/templates/element/filter.php
@@ -113,7 +113,7 @@
 
             // The populated variables are in plural while the column names are singular
             // Convention: It is a prerequisite that the vvar should be the plural of the column name
-            $populated_vvar = Inflector::pluralize($key);
+            $populated_vvar = lcfirst(Inflector::pluralize(Inflector::camelize($key)));
             $button_label = isset($$populated_vvar) ?
               $$populated_vvar[ $search_params[$key] ] :
               (is_array($search_params[$key]) ? 'Range' : $search_params[$key]);
@@ -146,16 +146,6 @@
         $field_datetime_columns = [];
 
         $inactiveFiltersCount = 0; // for re-balancing the columns and submit buttons
-
-        if(!empty($columns)) {
-          // The searchable attributes will be sorted first alphabetically
-          asort($vv_searchable_attributes);
-          // Sort the order attribute
-          uasort($vv_searchable_attributes, function ($item1, $item2) {
-            if ($item1['order'] == $item2['order']) return 0;
-            return $item1['order'] < $item2['order'] ? -1 : 1;
-          });
-        }
 
         foreach($vv_searchable_attributes as $key => $options) {
           if($options['type'] == 'boolean') {
@@ -226,7 +216,7 @@
 
           // The populated variables are in plural while the column names are singular
           // Convention: It is a prerequisite that the vvar should be the plural of the column name
-          $populated_vvar = Inflector::pluralize($key);
+          $populated_vvar = lcfirst(Inflector::pluralize(Inflector::camelize($key)));
           if(isset($$populated_vvar)) {
             // If we have an AutoViewVar matching the name of this key,
             // convert to a select