done

bezhanSalleh · Oct 24, 2024 · 1150bf0 · 1150bf0
1 parent 7b6c62f
commit 1150bf0
Show file tree

Hide file tree

Showing 4 changed files with 56 additions and 23 deletions.
diff --git a/src/FilamentGoogleAnalytics.php b/src/FilamentGoogleAnalytics.php
@@ -6,25 +6,25 @@
 
 class FilamentGoogleAnalytics
 {
-    public string $previous;
+    public int|float $previous = 0;
 
     public string $format;
 
-    public function __construct(public ?string $value = null) {}
+    public function __construct(public int|float $value = 0) {}
 
-    public static function for(?string $value = null)
+    public static function for(int|float $value = 0): static
     {
         return new static($value);
     }
 
-    public function previous(string $previous)
+    public function previous(int|float $previous): static
     {
         $this->previous = $previous;
 
         return $this;
     }
 
-    public function format(string $format)
+    public function format(string $format): static
     {
         $this->format = $format;
 
@@ -33,11 +33,12 @@ public function format(string $format)
 
     public function compute(): int
     {
-        if ($this->value == 0 || $this->previous == 0 || $this->previous == null) {
-            return 0;
-        }
-
-        return intval((($this->value - $this->previous) / $this->previous) * 100);
+        return match (true) {
+            $this->value == 0 && $this->previous == 0 => 0,
+            $this->value > 0 && $this->previous == 0 => $this->value,
+            $this->previous != 0 => intval((($this->value - $this->previous) / $this->previous) * 100),
+            default => 0,
+        };
     }
 
     public function trajectoryValue()

diff --git a/src/Traits/MetricDiff.php b/src/Traits/MetricDiff.php
@@ -3,9 +3,10 @@
 namespace BezhanSalleh\FilamentGoogleAnalytics\Traits;
 
 use Carbon\Carbon;
+use Spatie\Analytics\Period;
+use Spatie\Analytics\OrderBy;
 use Illuminate\Support\Collection;
 use Spatie\Analytics\Facades\Analytics;
-use Spatie\Analytics\Period;
 
 trait MetricDiff
 {
@@ -15,6 +16,9 @@ private function get(string $metric, string $dimensions, Period $period): Collec
             $period,
             [$metric],
             [$dimensions],
+            orderBy: [
+                OrderBy::dimension($dimensions, true),
+            ],
         );
 
         $results = $analyticsData;

diff --git a/src/Traits/Sessions.php b/src/Traits/Sessions.php
@@ -13,19 +13,33 @@ private function sessionsToday(): array
     {
         $results = $this->get('sessions', 'date', Period::days(1));
 
-        return [
-            'previous' => $results->first()['value'] ?? 0,
-            'result' => $results->last()['value'] ?? 0,
-        ];
+        return match(true) {
+            ($results->containsOneItem() && ($results->first()['date'])->isYesterday()) => [
+                'previous' => $results->first()['value'],
+                'result' => 0,
+            ],
+            ($results->containsOneItem() && ($results->first()['date'])->isToday()) => [
+                'previous' => 0,
+                'result' => $results->first()['value'],
+            ],
+            $results->isEmpty() => [
+                'previous' => 0,
+                'result' => 0,
+            ],
+            default => [
+                'previous' => $results->last()['value'] ?? 0,
+                'result' => $results->first()['value'] ?? 0,
+            ]
+        };
     }
 
     private function sessionsYesterday(): array
     {
         $results = $this->get('sessions', 'date', Period::create(Carbon::yesterday()->clone()->subDay(), Carbon::yesterday()));
 
         return [
-            'previous' => $results->first()['value'] ?? 0,
-            'result' => $results->last()['value'] ?? 0,
+            'previous' => $results->last()['value'] ?? 0,
+            'result' => $results->first()['value'] ?? 0,
         ];
     }
 

diff --git a/src/Traits/SessionsDuration.php b/src/Traits/SessionsDuration.php
@@ -13,19 +13,33 @@ private function sessionDurationToday(): array
     {
         $results = $this->get('averageSessionDuration', 'date', Period::days(1));
 
-        return [
-            'previous' => $results->last()['value'] ?? 0,
-            'result' => $results->first()['value'] ?? 0,
-        ];
+        return match(true) {
+            ($results->containsOneItem() && ($results->first()['date'])->isYesterday()) => [
+                'previous' => $results->first()['value'],
+                'result' => 0,
+            ],
+            ($results->containsOneItem() && ($results->first()['date'])->isToday()) => [
+                'previous' => 0,
+                'result' => $results->first()['value'],
+            ],
+            $results->isEmpty() => [
+                'previous' => 0,
+                'result' => 0,
+            ],
+            default => [
+                'previous' => $results->last()['value'] ?? 0,
+                'result' => $results->first()['value'] ?? 0,
+            ]
+        };
     }
 
     private function sessionDurationYesterday(): array
     {
         $results = $this->get('averageSessionDuration', 'date', Period::create(Carbon::yesterday()->clone()->subDay(), Carbon::yesterday()));
 
         return [
-            'previous' => $results->first()['value'] ?? 0,
-            'result' => $results->last()['value'] ?? 0,
+            'previous' => $results->last()['value'] ?? 0,
+            'result' => $results->first()['value'] ?? 0,
         ];
     }