From 4b0430e12d5b61865e7e9ccb693e6e3dac96cd45 Mon Sep 17 00:00:00 2001
From: PatrickePatate <lucienpuget1@gmail.com>
Date: Mon, 19 Jan 2026 13:48:16 +0100
Subject: [PATCH 1/3] Hide GlobalFilters if they are empty

---
 resources/js/filters/components/GlobalFilters.vue | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/resources/js/filters/components/GlobalFilters.vue b/resources/js/filters/components/GlobalFilters.vue
index 1fb3ee340..9b49459fe 100644
--- a/resources/js/filters/components/GlobalFilters.vue
+++ b/resources/js/filters/components/GlobalFilters.vue
@@ -21,7 +21,7 @@
         <input name="_token" :value="getCsrfToken()" type="hidden">
         <div class="grid gap-2">
             <template v-for="filter in filters.rootFilters" :key="filter.key">
-                <template v-if="filter.type === 'select'">
+                <template v-if="filter.type === 'select' && filter.values.length">
                     <Select
                         :name="`filterValues[${filter.key}]`"
                         v-model="filters.currentValues[filter.key]"

From 36ca290ee23ea07653cb1d1b61f6c25aab352d2c Mon Sep 17 00:00:00 2001
From: antoine <guingand.antoine@gmail.com>
Date: Mon, 19 Jan 2026 16:26:40 +0100
Subject: [PATCH 2/3] Do not send filters if no values

---
 .../js/filters/components/GlobalFilters.vue   |   2 +-
 src/Filters/GlobalFilters/GlobalFilters.php   |   2 +-
 src/Filters/SelectFilterTrait.php             |  10 +-
 src/Http/Middleware/HandleInertiaRequests.php |   7 +-
 tests/Http/GlobalFilterControllerTest.php     | 177 +++++++++++++-----
 5 files changed, 143 insertions(+), 55 deletions(-)

diff --git a/resources/js/filters/components/GlobalFilters.vue b/resources/js/filters/components/GlobalFilters.vue
index 9b49459fe..1fb3ee340 100644
--- a/resources/js/filters/components/GlobalFilters.vue
+++ b/resources/js/filters/components/GlobalFilters.vue
@@ -21,7 +21,7 @@
         <input name="_token" :value="getCsrfToken()" type="hidden">
         <div class="grid gap-2">
             <template v-for="filter in filters.rootFilters" :key="filter.key">
-                <template v-if="filter.type === 'select' && filter.values.length">
+                <template v-if="filter.type === 'select'">
                     <Select
                         :name="`filterValues[${filter.key}]`"
                         v-model="filters.currentValues[filter.key]"
diff --git a/src/Filters/GlobalFilters/GlobalFilters.php b/src/Filters/GlobalFilters/GlobalFilters.php
index 2b559415c..ca38f1ca6 100644
--- a/src/Filters/GlobalFilters/GlobalFilters.php
+++ b/src/Filters/GlobalFilters/GlobalFilters.php
@@ -20,7 +20,7 @@ public function getFilters(): array
     {
         if ($this->globalFilters === null) {
             $this->globalFilters = collect(sharp()->config()->get('global_filters'))
-                ->filter(fn (GlobalRequiredFilter $filter) => $filter->authorize())
+                ->filter(fn (GlobalRequiredFilter $filter) => $filter->authorize() && count($filter->cachedValues()))
                 ->values();
         }
 
diff --git a/src/Filters/SelectFilterTrait.php b/src/Filters/SelectFilterTrait.php
index ef65f1eb6..44af54c6c 100644
--- a/src/Filters/SelectFilterTrait.php
+++ b/src/Filters/SelectFilterTrait.php
@@ -12,6 +12,7 @@ trait SelectFilterTrait
     private bool $isMaster = false;
     private bool $isSearchable = false;
     private array $searchKeys = ['label'];
+    private ?array $cachedValues = null;
 
     final public function isMaster(): bool
     {
@@ -72,9 +73,9 @@ public function toArray(): array
         ]);
     }
 
-    protected function formattedValues(): array
+    public function formattedValues(): array
     {
-        $values = $this->values();
+        $values = $this->cachedValues();
 
         if (! is_array(collect($values)->first())) {
             return collect($values)
@@ -86,5 +87,10 @@ protected function formattedValues(): array
         return $values;
     }
 
+    public function cachedValues(): array
+    {
+        return $this->cachedValues ??= $this->values();
+    }
+
     abstract public function values(): array;
 }
diff --git a/src/Http/Middleware/HandleInertiaRequests.php b/src/Http/Middleware/HandleInertiaRequests.php
index 571a4e9e0..76e6dfd19 100644
--- a/src/Http/Middleware/HandleInertiaRequests.php
+++ b/src/Http/Middleware/HandleInertiaRequests.php
@@ -113,9 +113,10 @@ public function share(Request $request)
                             ],
                         ])
                         : null,
-                    'globalFilters' => app(GlobalFilters::class)->isEnabled()
-                        ? GlobalFiltersData::from(app(GlobalFilters::class)->toArray())
-                        : null,
+                    'globalFilters' => transform(app(GlobalFilters::class), fn (GlobalFilters $globalFilters) => $globalFilters->isEnabled()
+                            ? GlobalFiltersData::from($globalFilters->toArray())
+                            : null
+                    ),
                     'menu' => fn () => MenuData::from(app(SharpMenuManager::class)),
                     'auth' => fn () => [
                         'user' => UserData::from(auth()->user()),
diff --git a/tests/Http/GlobalFilterControllerTest.php b/tests/Http/GlobalFilterControllerTest.php
index 7bbbfbfe3..c01281e46 100644
--- a/tests/Http/GlobalFilterControllerTest.php
+++ b/tests/Http/GlobalFilterControllerTest.php
@@ -6,6 +6,134 @@
 
 beforeEach(function () {
     login();
+});
+
+describe('global filters', function () {
+    beforeEach(function () {
+        sharp()->config()->addGlobalFilter(
+            new class() extends GlobalRequiredFilter
+            {
+                public function buildFilterConfig(): void
+                {
+                    $this->configureKey('test');
+                }
+
+                public function values(): array
+                {
+                    return [
+                        1 => 'One',
+                        2 => 'Two',
+                        3 => 'Three',
+                    ];
+                }
+
+                public function defaultValue(): mixed
+                {
+                    return 2;
+                }
+            }
+        );
+    });
+
+    it('allows to user to update a global filter', function () {
+        $this->withoutExceptionHandling();
+
+        $this
+            ->followingRedirects()
+            ->post(route('code16.sharp.filters.update', ['2']), ['filterValues' => ['test' => '1']])
+            ->assertOk();
+
+        $this->assertEquals('1', sharp()->context()->globalFilterValue('test'));
+    });
+
+    it('sets to global filter to the default value if missing', function () {
+        $this
+            ->followingRedirects()
+            ->post(route('code16.sharp.filters.update', ['2']), ['filterValues' => []])
+            ->assertOk();
+
+        $this->assertEquals(2, sharp()->context()->globalFilterValue('test'));
+    });
+
+    it('does not allow to set a global filter to an unexpected value', function () {
+        $this
+            ->followingRedirects()
+            ->post(route('code16.sharp.filters.update', ['2']), ['filterValues' => ['test' => 4]])
+            ->assertOk();
+
+        $this->assertEquals(2, sharp()->context()->globalFilterValue('test'));
+    });
+
+    it('sends the current value of the global filter with every inertia request', function () {
+        sharp()->config()->declareEntity(PersonEntity::class);
+
+        $this
+            ->followingRedirects()
+            ->get('/sharp/s-list/person')
+            ->assertInertia(fn (Assert $page) => $page
+                ->has('globalFilters.config.filters._root.0', fn (Assert $filter) => $filter
+                    ->where('key', 'test')
+                    ->where('required', true)
+                    ->etc()
+                )
+                ->where('globalFilters.filterValues.current.test', 2)
+                ->where('globalFilters.filterValues.default.test', 2)
+            );
+
+        $this
+            ->followingRedirects()
+            ->get('/sharp/3/s-list/person')
+            ->assertInertia(fn (Assert $page) => $page
+                ->has('globalFilters.config.filters._root.0', fn (Assert $filter) => $filter
+                    ->where('key', 'test')
+                    ->etc()
+                )
+                ->where('globalFilters.filterValues.current.test', 3)
+                ->where('globalFilters.filterValues.default.test', 2)
+            );
+    });
+});
+
+it('does not sends filters of none configured', function () {
+    sharp()->config()->declareEntity(PersonEntity::class);
+
+    $this
+        ->followingRedirects()
+        ->get('/sharp/s-list/person')
+        ->assertInertia(fn (Assert $page) => $page
+            ->where('globalFilters', null)
+        );
+});
+
+it('does not sends filters of there have no values', function () {
+    sharp()->config()->declareEntity(PersonEntity::class);
+
+    sharp()->config()->addGlobalFilter(
+        new class() extends GlobalRequiredFilter
+        {
+            public function buildFilterConfig(): void
+            {
+                $this->configureKey('test-no-values');
+            }
+
+            public function values(): array
+            {
+                return [];
+            }
+
+            public function defaultValue(): mixed
+            {
+                return null;
+            }
+        }
+    );
+
+    $this
+        ->followingRedirects()
+        ->get('/sharp/s-list/person')
+        ->assertInertia(fn (Assert $page) => $page
+            ->where('globalFilters', null)
+        );
 
     sharp()->config()->addGlobalFilter(
         new class() extends GlobalRequiredFilter
@@ -30,62 +158,15 @@ public function defaultValue(): mixed
             }
         }
     );
-});
-
-it('allows to user to update a global filter', function () {
-    $this->withoutExceptionHandling();
-
-    $this
-        ->followingRedirects()
-        ->post(route('code16.sharp.filters.update', ['2']), ['filterValues' => ['test' => '1']])
-        ->assertOk();
-
-    $this->assertEquals('1', sharp()->context()->globalFilterValue('test'));
-});
-
-it('sets to global filter to the default value if missing', function () {
-    $this
-        ->followingRedirects()
-        ->post(route('code16.sharp.filters.update', ['2']), ['filterValues' => []])
-        ->assertOk();
-
-    $this->assertEquals(2, sharp()->context()->globalFilterValue('test'));
-});
-
-it('does not allow to set a global filter to an unexpected value', function () {
-    $this
-        ->followingRedirects()
-        ->post(route('code16.sharp.filters.update', ['2']), ['filterValues' => ['test' => 4]])
-        ->assertOk();
-
-    $this->assertEquals(2, sharp()->context()->globalFilterValue('test'));
-});
-
-it('sends the current value of the global filter with every inertia request', function () {
-    sharp()->config()->declareEntity(PersonEntity::class);
-
-    $this
-        ->followingRedirects()
-        ->get('/sharp/s-list/person')
-        ->assertInertia(fn (Assert $page) => $page
-            ->has('globalFilters.config.filters._root.0', fn (Assert $filter) => $filter
-                ->where('key', 'test')
-                ->where('required', true)
-                ->etc()
-            )
-            ->where('globalFilters.filterValues.current.test', 2)
-            ->where('globalFilters.filterValues.default.test', 2)
-        );
 
     $this
         ->followingRedirects()
         ->get('/sharp/3/s-list/person')
         ->assertInertia(fn (Assert $page) => $page
+            ->has('globalFilters.config.filters._root', 1)
             ->has('globalFilters.config.filters._root.0', fn (Assert $filter) => $filter
                 ->where('key', 'test')
                 ->etc()
             )
-            ->where('globalFilters.filterValues.current.test', 3)
-            ->where('globalFilters.filterValues.default.test', 2)
         );
 });

From 5b9a2d8257b98745fd165da5252e4e4bcb8388e7 Mon Sep 17 00:00:00 2001
From: antoine <guingand.antoine@gmail.com>
Date: Mon, 19 Jan 2026 16:28:49 +0100
Subject: [PATCH 3/3] cleanup

---
 src/Filters/SelectFilterTrait.php             | 2 +-
 src/Http/Middleware/HandleInertiaRequests.php | 7 ++++---
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/Filters/SelectFilterTrait.php b/src/Filters/SelectFilterTrait.php
index 44af54c6c..2ba0af81f 100644
--- a/src/Filters/SelectFilterTrait.php
+++ b/src/Filters/SelectFilterTrait.php
@@ -73,7 +73,7 @@ public function toArray(): array
         ]);
     }
 
-    public function formattedValues(): array
+    protected function formattedValues(): array
     {
         $values = $this->cachedValues();
 
diff --git a/src/Http/Middleware/HandleInertiaRequests.php b/src/Http/Middleware/HandleInertiaRequests.php
index 76e6dfd19..4ad29b6ca 100644
--- a/src/Http/Middleware/HandleInertiaRequests.php
+++ b/src/Http/Middleware/HandleInertiaRequests.php
@@ -113,10 +113,11 @@ public function share(Request $request)
                             ],
                         ])
                         : null,
-                    'globalFilters' => transform(app(GlobalFilters::class), fn (GlobalFilters $globalFilters) => $globalFilters->isEnabled()
+                    'globalFilters' => transform(app(GlobalFilters::class), function (GlobalFilters $globalFilters) {
+                        return $globalFilters->isEnabled()
                             ? GlobalFiltersData::from($globalFilters->toArray())
-                            : null
-                    ),
+                            : null;
+                    }),
                     'menu' => fn () => MenuData::from(app(SharpMenuManager::class)),
                     'auth' => fn () => [
                         'user' => UserData::from(auth()->user()),