Fix:fix bug serializable in config

mehdi-fathi · Jan 5, 2025 · 5d79f2d · 5d79f2d
1 parent 2704213
commit 5d79f2d
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 6 deletions.
diff --git a/src/QueryFilter/Core/FilterBuilder/IO/RequestFilter.php b/src/QueryFilter/Core/FilterBuilder/IO/RequestFilter.php
@@ -315,9 +315,9 @@ public function handleRequest($builder, ?array $ignore_request, ?array $accept_r
     private function handleRequestEncoded($request): void
     {
         if (isset($request['hashed_filters'])) {
-            $this->request = json_decode($this->decodeWithSalt($request['hashed_filters'], config('eloquentFilter.request_salt')()), true);
+            $this->request = json_decode($this->decodeWithSalt($request['hashed_filters'], config('eloquentFilter.request_salt')), true);
         } else {
-            $this->setRequestEncoded($request, config('eloquentFilter.request_salt')());
+            $this->setRequestEncoded($request, config('eloquentFilter.request_salt'));
         }
     }
 

diff --git a/src/config/config.php b/src/config/config.php
@@ -42,12 +42,11 @@
     ],
     'filtering_keys' => [
     ],
+
     /*
     * Set salt for encode request.
     */
-    'request_salt' => function () {
-        return 1234;
-    },
+    'request_salt' => 1234,
 
     /*
     * Cast sign method is prefix name method for change data before filtering.

diff --git a/tests/Tests/Eloquent/ModelFilterMockTest.php b/tests/Tests/Eloquent/ModelFilterMockTest.php
@@ -1769,7 +1769,7 @@ public function testWhereWithRequestHash2()
 
         EloquentFilter::setRequestEncoded([
             'title' => 'sport',
-        ], config('eloquentFilter.request_salt')());
+        ], 1234);
 
         $this->assertSame(EloquentFilter::getRequestEncoded(), 'MTIzNHsidGl0bGUiOiJzcG9ydCJ9');