QueryBuilder::getQuery does not clone parameters, is this intended or a bug? #6221

New Issue

admin · 2026-01-22T15:29:06+01:00

admin commented

2026-01-22 15:29:06 +01:00

Originally created by @datadestroyd on GitHub (Apr 15, 2019).

Originally assigned to: @Ocramius on GitHub.

Bug Report

Q	A
Version	2.6.3

Summary

QueryBuilder::getQuery does not deep clone the parameters. I am unsure whether this is intended behavior or a bug. The docs don't mention it. However, I find it very counterintuitive.

Current behavior

When calling getQuery on a QueryBuilder, the parameters are copied into the Query instance. However, subsequently modifying parameters on the builder, e.g. to create another similar query, also modify the parameters in the previous Query instance.

How to reproduce

$builder = $em->createQueryBuilder()
  ->from('SomeEntity', 'e')
  ->select('e')
  ->where('e.someProperty = :param')
  ->setParameter('param', 'foo');

$queryA = $builder->getQuery();

$builder->setParameter('param', 'bar');

$queryB = $builder->getQuery();

// $queryA->getParameter('param')->getValue() === $queryB->getParameter('param')->getValue() === 'bar'

Expected behavior

$queryA->getParameter('param')->getValue() should return 'foo', not 'bar'.

Originally created by @datadestroyd on GitHub (Apr 15, 2019). Originally assigned to: @Ocramius on GitHub. ### Bug Report | Q | A |------------ | ------ | Version | 2.6.3 #### Summary `QueryBuilder::getQuery` does not deep clone the parameters. I am unsure whether this is intended behavior or a bug. The docs don't mention it. However, I find it very counterintuitive. #### Current behavior When calling `getQuery` on a `QueryBuilder`, the parameters are copied into the `Query` instance. However, subsequently modifying parameters on the builder, e.g. to create another similar query, also modify the parameters in the previous `Query` instance. #### How to reproduce ```php $builder = $em->createQueryBuilder() ->from('SomeEntity', 'e') ->select('e') ->where('e.someProperty = :param') ->setParameter('param', 'foo'); $queryA = $builder->getQuery(); $builder->setParameter('param', 'bar'); $queryB = $builder->getQuery(); // $queryA->getParameter('param')->getValue() === $queryB->getParameter('param')->getValue() === 'bar' ``` #### Expected behavior `$queryA->getParameter('param')->getValue()` should return `'foo'`, not `'bar'`.

admin added the Can't Fix Question labels 2026-01-22 15:29:06 +01:00

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#6221