Fix missing parameters entry in console stats command

bilfeldt · Jun 21, 2024 · 4886958 · 4886958
1 parent b636591
commit 4886958
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/src/Commands/LaravelRouteStatisticsCommand.php b/src/Commands/LaravelRouteStatisticsCommand.php
@@ -29,11 +29,14 @@ class LaravelRouteStatisticsCommand extends Command
     public function handle()
     {
         $query = $this->getQuery();
+        $fields = $this->getFields();
 
         if ($this->option('group')) {
             $query->select($this->option('group'))
                 ->addSelect(DB::raw('MAX(date) as last_used'))
                 ->addSelect(DB::raw('SUM(counter) as counter'));
+        } else {
+            $query->select($fields);
         }
 
         $this->applyFilters($query);
@@ -43,7 +46,7 @@ public function handle()
         $results = $query->limit($this->option('limit'))->get();
 
         $this->table(
-            $this->getFields(),
+            $fields,
             $results->toArray()
         );
 
@@ -111,16 +114,17 @@ protected function getFields(): array
             return array_merge($this->option('group'), ['last_used', 'counter']);
         }
 
-        return [
+        return array_filter([
             'id',
             'user_id',
             'team_id',
             'method',
             'route',
             'status',
+            config('route-statistics.store_parameters') === true ? 'parameters' : null,
             'ip',
             'date',
             'counter',
-        ];
+        ]);
     }
 }