Add a constant for default max per page

loic425 · loic425 · commit 2263e37ff0ae · 2024-09-03T14:10:13.000+02:00
diff --git a/src/Component/src/Grid/State/RequestGridProvider.php b/src/Component/src/Grid/State/RequestGridProvider.php
@@ -25,6 +25,8 @@
 
 final class RequestGridProvider implements ProviderInterface
 {
+    private const DEFAULT_MAX_PER_PAGE = 10;
+
     public function __construct(
         private ?GridViewFactoryInterface $gridViewFactory = null,
         private ?GridProviderInterface $gridProvider = null,
@@ -80,7 +82,7 @@ private function resolveMaxPerPage(?int $requestLimit, array $gridLimits = []):
         if (null === $requestLimit) {
             $firstGridLimit = reset($gridLimits);
 
-            return false === $firstGridLimit ? 10 : $firstGridLimit;
+            return false === $firstGridLimit ? self::DEFAULT_MAX_PER_PAGE : $firstGridLimit;
         }
 
         if (!empty($gridLimits)) {

Original file line number	Diff line number	Diff line change
`@@ -25,6 +25,8 @@`
`25`	`25`
`26`	`26`	`final class RequestGridProvider implements ProviderInterface`
`27`	`27`	`{`
	`28`	`+ private const DEFAULT_MAX_PER_PAGE = 10;`
	`29`	`+`
`28`	`30`	`public function __construct(`
`29`	`31`	`private ?GridViewFactoryInterface $gridViewFactory = null,`
`30`	`32`	`private ?GridProviderInterface $gridProvider = null,`
`@@ -80,7 +82,7 @@ private function resolveMaxPerPage(?int $requestLimit, array $gridLimits = []):`
`80`	`82`	`if (null === $requestLimit) {`
`81`	`83`	`$firstGridLimit = reset($gridLimits);`
`82`	`84`
`83`		`- return false === $firstGridLimit ? 10 : $firstGridLimit;`
	`85`	`+ return false === $firstGridLimit ? self::DEFAULT_MAX_PER_PAGE : $firstGridLimit;`
`84`	`86`	`}`
`85`	`87`
`86`	`88`	`if (!empty($gridLimits)) {`