DDC-3317: [GH-1142] func_get_args() call order fix for HHVM bug #4100

New Issue

admin · 2026-01-22T14:35:11+01:00

admin commented

2026-01-22 14:35:11 +01:00

Originally created by @doctrinebot on GitHub (Sep 22, 2014).

Originally assigned to: @Ocramius on GitHub.

Jira issue originally created by user @doctrinebot:

This issue is created automatically through a Github pull request on behalf of TwoWholeWorms:

Url: https://github.com/doctrine/doctrine2/pull/1142

Message:

This PR fixes bugs in Doctrine\ORM\QueryBuilder where the values provided to method arguments to ::andWhere(), ::orWhere(), ::andHaving(), and ::orHaving() are overwritten by code within the methods in HHVM and PHP7.

After much fun and headscratching in both #doctrine and #hhvm on Freenode, we discovered that in the following code, $where = $this->getDqlPart('where'); overwrites the values stored internally for the $where parameter of the function which is returned by func*get*args():

public function orWhere($where)
{
    $where = $this->getDqlPart('where');
    $args  = func*get*args();
}

This means that instead of $args being set as expected to an array containing the string or object provided to the function, it actually contains the object returned by QueryBuilder::getDqlPart(), which causes all the problems you'd expect it to.

This PR simply swaps the order of the calls so that the function arguments are retrieved via func*get*args() before the $where variable is modified, fixing the problem.

Originally created by @doctrinebot on GitHub (Sep 22, 2014). Originally assigned to: @Ocramius on GitHub. Jira issue originally created by user @doctrinebot: This issue is created automatically through a Github pull request on behalf of TwoWholeWorms: Url: https://github.com/doctrine/doctrine2/pull/1142 Message: This PR fixes bugs in `Doctrine\ORM\QueryBuilder` where the values provided to method arguments to `::andWhere()`, `::orWhere()`, `::andHaving()`, and `::orHaving()` are overwritten by code within the methods in HHVM and PHP7. After much fun and headscratching in both #doctrine and #hhvm on Freenode, we discovered that in the following code, `$where = $this->getDqlPart('where');` overwrites the values stored internally for the `$where` parameter of the function which is returned by `func*get*args()`: ``` public function orWhere($where) { $where = $this->getDqlPart('where'); $args = func*get*args(); } ``` This means that instead of `$args` being set as expected to an array containing the string or object provided to the function, it actually contains the object returned by `QueryBuilder::getDqlPart()`, which causes all the problems you'd expect it to. This PR simply swaps the order of the calls so that the function arguments are retrieved via `func*get*args()` before the `$where` variable is modified, fixing the problem.

admin added the Bug label 2026-01-22 14:35:11 +01:00

admin closed this issue

2026-01-22 14:35:12 +01:00

admin commented

2026-01-22 14:35:13 +01:00

@doctrinebot commented on GitHub (Sep 22, 2014):

relates to DBAL-608: [GH-372] HHVM compatibility: func_get_args
relates to DDC-2681: [GH-790] HHVM compatibility: func_get_args

@doctrinebot commented on GitHub (Sep 22, 2014): - relates to [DBAL-608: [GH-372] HHVM compatibility: func_get_args](http://www.doctrine-project.org/jira/browse/DBAL-608) - relates to [DDC-2681: [GH-790] HHVM compatibility: func_get_args](http://www.doctrine-project.org/jira/browse/DDC-2681)

admin commented

2026-01-22 14:35:13 +01:00

@doctrinebot commented on GitHub (Sep 22, 2014):

Comment created by @doctrinebot:

A related Github Pull-Request [GH-1142] was closed:
https://github.com/doctrine/doctrine2/pull/1142

@doctrinebot commented on GitHub (Sep 22, 2014): Comment created by @doctrinebot: A related Github Pull-Request [GH-1142] was closed: https://github.com/doctrine/doctrine2/pull/1142

admin commented

2026-01-22 14:35:13 +01:00

@doctrinebot commented on GitHub (Sep 22, 2014):

Issue was closed with resolution "Fixed"

@doctrinebot commented on GitHub (Sep 22, 2014): Issue was closed with resolution "Fixed"

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: doctrine/archived-orm#4100