Commit6eb836b

committed

[OptionsResolver] Fix force prepend normalizer

1 parentecb5f7c commit6eb836bCopy full SHA for 6eb836b

File tree

+12

-0

lines changed

src/Symfony/Component/OptionsResolver
- OptionsResolver.php
- Tests
  - OptionsResolverTest.php

+12

-0

lines changed

Lines changed: 1 addition & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -532,6 +532,7 @@ public function addNormalizer(string $option, \Closure $normalizer, bool $forceP`
`532`	`532`	`}`
`533`	`533`
`534`	`534`	`if ($forcePrepend) {`
	`535`	`+$this->normalizers[$option] =$this->normalizers[$option] ?? [];`
`535`	`536`	`array_unshift($this->normalizers[$option],$normalizer);`
`536`	`537`	`}else {`
`537`	`538`	`$this->normalizers[$option][] =$normalizer;`

Lines changed: 11 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -1506,6 +1506,17 @@ public function testForcePrependNormalizerClosure()`
`1506`	`1506`	`$this->assertEquals(['foo' =>'2nd-normalized-1st-normalized-bar'],$this->resolver->resolve());`
`1507`	`1507`	`}`
`1508`	`1508`
	`1509`	`+publicfunctiontestForcePrependNormalizerForResolverWithoutPreviousNormalizers()`
	`1510`	`+ {`
	`1511`	`+// defined by superclass`
	`1512`	`+$this->resolver->setDefault('foo','bar');`
	`1513`	`+$this->resolver->addNormalizer('foo',function (Options$options,$value) {`
	`1514`	`+return'1st-normalized-'.$value;`
	`1515`	`+ },true);`
	`1516`	`+`
	`1517`	`+$this->assertEquals(['foo' =>'1st-normalized-bar'],$this->resolver->resolve());`
	`1518`	`+ }`
	`1519`	`+`
`1509`	`1520`	`publicfunctiontestAddNormalizerFailsIfUnknownOption()`
`1510`	`1521`	`{`
`1511`	`1522`	`$this->expectException('Symfony\Component\OptionsResolver\Exception\UndefinedOptionsException');`

Comments

(0)