8000 [DoctrineBridge] Don't use object IDs in DoctrineChoiceLoader when pa… · symfony/symfony@f6e5298 · GitHub
[go: up one dir, main page]
Skip to content

Commit f6e5298

Browse files
webmozartwebmozart
committed
[DoctrineBridge] Don't use object IDs in DoctrineChoiceLoader when passing a value closure
1 parent eaca0bb commit f6e5298

File tree

3 files changed

+466
-2
lines changed

3 files changed

+466
-2
lines changed

‎src/Symfony/Bridge/Doctrine/Form/ChoiceList/DoctrineChoiceLoader.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -110,9 +110,10 @@ public function loadValuesForChoices(array $choices, $value = null)
110110

111111
// Optimize performance for single-field identifiers. We already
112112
// know that the IDs are used as values
113+
$optimize = null === $value || is_array($value) && $value[0] === $this->idReader;
113114

114115
// Attention: This optimization does not check choices for existence
115-
if (!$this->choiceList && $this->idReader->isSingleId()) {
116+
if ($optimize && !$this->choiceList && $this->idReader->isSingleId()) {
116117
$values = array();
117118

118119
// Maintain order and indices of the given objects

0 commit comments

Comments
 (0)
0