From 5d53ceffbcc0cdf397100b362c648582be78ec88 Mon Sep 17 00:00:00 2001
From: Lukas Frey <mail@lukasfrey.cz>
Date: Thu, 2 May 2024 16:01:47 +0200
Subject: [PATCH] refactor: moved platform detection to helper function

Signed-off-by: Lukas Frey <mail@lukasfrey.cz>
---
 .../docs/03-resources/08-global-search.md      |  5 +++--
 .../src/Panel/Concerns/HasGlobalSearch.php     | 13 ++-----------
 .../{panels => support}/src/Enums/Platform.php |  2 +-
 packages/support/src/helpers.php               | 18 ++++++++++++++++++
 4 files changed, 24 insertions(+), 14 deletions(-)
 rename packages/{panels => support}/src/Enums/Platform.php (80%)

diff --git a/packages/panels/docs/03-resources/08-global-search.md b/packages/panels/docs/03-resources/08-global-search.md
index 13baac37b1..ae5e2f2fd8 100644
--- a/packages/panels/docs/03-resources/08-global-search.md
+++ b/packages/panels/docs/03-resources/08-global-search.md
@@ -193,13 +193,14 @@ For example, to provide a custom suffix for each platform on your own, you could
 
 ```php
 use Filament\Panel;
-use Filament\Enums\Platform;
+use Filament\Support\Enums\Platform;
+use function Filament\Support\detect_platform;
 
 public function panel(Panel $panel): Panel
 {
     return $panel
         // ...
-        ->globalSearchSuffix(fn(Platform $platform) => match($platform) {
+        ->globalSearchFieldSuffix(fn() => match(detect_platform()) {
             Platform::Windows, Platform::Linux => 'CTRL+K',
             Platform::Mac => '⌘K'
             default => null,
diff --git a/packages/panels/src/Panel/Concerns/HasGlobalSearch.php b/packages/panels/src/Panel/Concerns/HasGlobalSearch.php
index 014f0fa49c..9f59aff753 100644
--- a/packages/panels/src/Panel/Concerns/HasGlobalSearch.php
+++ b/packages/panels/src/Panel/Concerns/HasGlobalSearch.php
@@ -4,9 +4,9 @@ namespace Filament\Panel\Concerns;
 
 use Closure;
 use Exception;
-use Filament\Enums\Platform;
 use Filament\GlobalSearch\Contracts\GlobalSearchProvider;
 use Filament\GlobalSearch\DefaultGlobalSearchProvider;
+use Filament\Support\Enums\Platform;
 use Illuminate\Support\Arr;
 use Illuminate\Support\Stringable;
 
@@ -105,16 +105,7 @@ trait HasGlobalSearch
 
     public function getGlobalSearchFieldSuffix(): ?string
     {
-        $userAgent = request()->userAgent();
-
-        return $this->evaluate($this->globalSearchFieldSuffix, [
-            'platform' => match (true) {
-                str_contains($userAgent, 'Windows') => Platform::Windows,
-                str_contains($userAgent, 'Mac') => Platform::Mac,
-                str_contains($userAgent, 'Linux') => Platform::Linux,
-                default => Platform::Other,
-            },
-        ]);
+        return $this->evaluate($this->globalSearchFieldSuffix);
     }
 
     public function getGlobalSearchProvider(): ?GlobalSearchProvider
diff --git a/packages/panels/src/Enums/Platform.php b/packages/support/src/Enums/Platform.php
similarity index 80%
rename from packages/panels/src/Enums/Platform.php
rename to packages/support/src/Enums/Platform.php
index 1df86abc02..b81dcca556 100644
--- a/packages/panels/src/Enums/Platform.php
+++ b/packages/support/src/Enums/Platform.php
@@ -1,6 +1,6 @@
 <?php
 
-namespace Filament\Enums;
+namespace Filament\Support\Enums;
 
 enum Platform: string
 {
diff --git a/packages/support/src/helpers.php b/packages/support/src/helpers.php
index 21c58d9bb8..dbfb645a97 100644
--- a/packages/support/src/helpers.php
+++ b/packages/support/src/helpers.php
@@ -2,6 +2,7 @@
 
 namespace Filament\Support;
 
+use Filament\Support\Enums\Platform;
 use Filament\Support\Facades\FilamentColor;
 use Filament\Support\Facades\FilamentView;
 use Illuminate\Contracts\Support\Htmlable;
@@ -219,3 +220,20 @@ if (! function_exists('Filament\Support\generate_search_term_expression')) {
         return Str::lower($search);
     }
 }
+
+if (! function_exists('Filament\Support\detect_platform')) {
+    /**
+     * Attempts to detect the platform from the user-agent.
+     */
+    function detect_platform(): Platform
+    {
+        $userAgent = request()->userAgent();
+
+        return match (true) {
+            str_contains($userAgent, 'Windows') => Platform::Windows,
+            str_contains($userAgent, 'Mac') => Platform::Mac,
+            str_contains($userAgent, 'Linux') => Platform::Linux,
+            default => Platform::Other,
+        };
+    }
+}