Skip to content

Commit b7d8229

Browse files
Fix WhereInWalker description to better describe the behaviour of this class (#9268)
* Fix WhereInWalker description: - change the verb "replace" with "append" to better describe the behaviour of this class * Rephrase comment in WhereInWalker as suggested from reviewer Co-authored-by: Alexander M. Turek <[email protected]> * Rephrase comment in WhereInWalker as suggested from reviewer Co-authored-by: Alexander M. Turek <[email protected]> Co-authored-by: Alexander M. Turek <[email protected]>
1 parent 9a2f1f3 commit b7d8229

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

lib/Doctrine/ORM/Tools/Pagination/WhereInWalker.php

+2-4
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@
2828
use function reset;
2929

3030
/**
31-
* Replaces the whereClause of the AST with a WHERE id IN (:foo_1, :foo_2) equivalent.
31+
* Appends a condition "id IN (:foo_1, :foo_2)" to the whereClause of the AST.
3232
*/
3333
class WhereInWalker extends TreeWalkerAdapter
3434
{
@@ -43,9 +43,7 @@ class WhereInWalker extends TreeWalkerAdapter
4343
public const PAGINATOR_ID_ALIAS = 'dpid';
4444

4545
/**
46-
* Replaces the whereClause in the AST.
47-
*
48-
* Generates a clause equivalent to WHERE IN (:dpid_1, :dpid_2, ...)
46+
* Appends a condition equivalent to "WHERE IN (:dpid_1, :dpid_2, ...)" to the whereClause of the AST.
4947
*
5048
* The parameter namespace (dpid) is defined by
5149
* the PAGINATOR_ID_ALIAS

0 commit comments

Comments
 (0)