minor #19953 [Messenger] Update messenger.rst (svdv22)

javiereguiluz · javiereguiluz · commit 312603c86307 · 2024-06-12T17:34:06.000+02:00
This PR was merged into the 6.4 branch. Discussion ---------- [Messenger] Update messenger.rst Was confused for a second because getBatchSize was introduced in 6.4 but according to the docs it wasn't used. It is according to the actual code so here's the fix. See https://github.com/symfony/messenger/blob/6.4/Handler/BatchHandlerTrait.php#L58C9-L58C61 Commits ------- 4e2c66d Update messenger.rst
diff --git a/messenger.rst b/messenger.rst
@@ -2644,7 +2644,7 @@ provided in order to ease the declaration of these special handlers::
         // of the trait to define your own batch size...
         private function shouldFlush(): bool
         {
-            return 100 <= \count($this->jobs);
+            return $this->getBatchSize() <= \count($this->jobs);
         }
 
         // ... or redefine the `getBatchSize()` method if the default

Original file line number	Diff line number	Diff line change
`@@ -2644,7 +2644,7 @@ provided in order to ease the declaration of these special handlers::`
`2644`	`2644`	`// of the trait to define your own batch size...`
`2645`	`2645`	`private function shouldFlush(): bool`
`2646`	`2646`	`{`
`2647`		`- return 100 <= \count($this->jobs);`
	`2647`	`+ return $this->getBatchSize() <= \count($this->jobs);`
`2648`	`2648`	`}`
`2649`	`2649`
`2650`	`2650`	// ... or redefine the `getBatchSize()` method if the default