From 025e3d5361d0c8c80f41cd6dc687e0a80a4e1213 Mon Sep 17 00:00:00 2001
From: "Victor.Sergeyev" <victor17011998@yandex.ru>
Date: Mon, 2 Dec 2024 15:54:06 +0400
Subject: [PATCH] change: sort and collapse stats

---
 resources/views/bootstrap-5/logs.blade.php   |  3 +++
 resources/views/bootstrap-5/stats.blade.php  | 24 ++++++++++++++++++--
 src/Http/Controllers/LogViewerController.php |  9 ++++++++
 3 files changed, 34 insertions(+), 2 deletions(-)

diff --git a/resources/views/bootstrap-5/logs.blade.php b/resources/views/bootstrap-5/logs.blade.php
index 134cd7d..e5c5ce3 100644
--- a/resources/views/bootstrap-5/logs.blade.php
+++ b/resources/views/bootstrap-5/logs.blade.php
@@ -46,6 +46,9 @@
                                 <a href="{{ route('log-viewer::logs.show', [$prefix, $date]) }}" class="btn btn-sm btn-info">
                                     <i class="bi bi-eye"></i>
                                 </a>
+                                <a href="{{ route('log-viewer::logs.stats', [$prefix, $date]) }}" class="btn btn-sm btn-warning">
+                                    <i class="bi bi-calculator"></i>
+                                </a>
                                 <a href="{{ route('log-viewer::logs.download', [$prefix, $date]) }}" class="btn btn-sm btn-success">
                                     <i class="bi bi-download"></i>
                                 </a>
diff --git a/resources/views/bootstrap-5/stats.blade.php b/resources/views/bootstrap-5/stats.blade.php
index fa99b29..6bd6ac4 100644
--- a/resources/views/bootstrap-5/stats.blade.php
+++ b/resources/views/bootstrap-5/stats.blade.php
@@ -15,9 +15,29 @@
     </ul>
     <ul class="list-group list-group-flush">
         @foreach ($context as $dottedKey => $values)
+            @php
+                $valuesCount = count($values);
+                $collapsable = $valuesCount > 3;
+                $collapseId = "values-list-{$dottedKey}";
+            @endphp
             <li class="list-group-item d-flex justify-content-between flex-wrap">
-                <span>{{ $dottedKey }}</span>
-                <ul class="list-group list-group-flush">
+                <span>
+                    {{ $dottedKey }}
+                    @if ($collapsable)
+                        <a
+                            class="btn btn-sm btn-light"
+                            role="button"
+                            data-bs-toggle="collapse"
+                            href="#{{ $collapseId }}"
+                            aria-expanded="false"
+                            aria-controls="{{ $collapseId }}"
+                        >
+                            <span class="badge text-bg-primary rounded-pill">{{ $valuesCount }}</span>
+                            <i class="bi bi-chevron-down"></i>
+                        </a>
+                    @endif
+                </span>
+                <ul id="{{ $collapseId }}" class="list-group list-group-flush @if ($collapsable) collapse @endif">
                     @foreach ($values as $value => $info)
                         <li class="list-group-item">
                             <span class="badge text-bg-primary rounded-pill">{{ $info['count'] }}</span>
diff --git a/src/Http/Controllers/LogViewerController.php b/src/Http/Controllers/LogViewerController.php
index 2dd1019..0f77ae2 100644
--- a/src/Http/Controllers/LogViewerController.php
+++ b/src/Http/Controllers/LogViewerController.php
@@ -150,6 +150,11 @@ public function stats(Request $request, string $prefix, string $date): View
                         $context[$dottedKey] = [];
                     }
 
+                    if (is_array($value)) {
+                        // empty array could not be flattened
+                        $value = '';
+                    }
+
                     if (isset($context[$dottedKey][$value])) {
                         $context[$dottedKey][$value]['count']++;
                     } else {
@@ -169,6 +174,10 @@ public function stats(Request $request, string $prefix, string $date): View
             }
         }
 
+        foreach ($context as $dottedKey => &$values) {
+            $values = collect($values)->sortByDesc('count')->all();
+        }
+
         return $this->view($request, 'stats', compact('level', 'log', 'search', 'levels', 'stats', 'context'));
     }