8000 [Config] Do not array_unique EnumNode values · symfony/symfony@7de44c2 · GitHub
[go: up one dir, main page]
Skip to content

Commit 7de44c2

Browse files
fancywebfancyweb
committed
[Config] Do not array_unique EnumNode values
1 parent 991c2ba commit 7de44c2

File tree

7 files changed

+21
-17
lines changed

7 files changed

+21
-17
lines changed

‎src/Symfony/Component/Config/Builder/ConfigBuilderGenerator.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -426,9 +426,9 @@ private function getComment(BaseNode $node): string
426426
}
427427

428428
if ($node instanceof EnumNode) {
429-
$comment .= sprintf(' * @param ParamConfigurator|%s $value', implode('|', array_map(function ($a) {
429+
$comment .= sprintf(' * @param ParamConfigurator|%s $value', implode('|', array_unique(array_map(function ($a) {
430430
return var_export($a, true);
431-
}, $node->getValues())))."\n";
431+
}, $node->getValues()))))."\n";
432432
} else {
433433
$parameterTypes = $this->getParameterTypes($node);
434434
$comment .= ' * @param ParamConfigurator|'.implode('|', $parameterTypes).' $value'."\n";

‎src/Symfony/Component/Config/Definition/Builder/EnumNodeDefinition.php

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,6 @@ class EnumNodeDefinition extends ScalarNodeDefinition
2727
*/
2828
public function values(array $values): static
2929
{
30-
$values = array_unique($values);
31-
3230
if (!$values) {
3331
throw new \InvalidArgumentException('->values() must be called with at least one value.');
3432
}

‎src/Symfony/Component/Config/Definition/Dumper/XmlReferenceDumper.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,7 @@ private function writeNode(NodeInterface $node, int $depth = 0, bool $root = fal
109109
FloatNode::class,
110110
IntegerNode::class => 'numeric value',
111111
BooleanNode::class => 'true|false',
112-
EnumNode::class => implode('|', array_map('json_encode', $prototype->getValues())),
112+
EnumNode::class => implode('|', array_unique(array_map('json_encode', $prototype->getValues()))),
113113
default => 'value',
114114
};
115115
}
@@ -151,7 +151,7 @@ private function writeNode(NodeInterface $node, int $depth = 0, bool $root = fal
151151
}
152152

153153
if ($child instanceof EnumNode) {
154-
$comments[] = 'One of '.implode('; ', array_map('json_encode', $child->getValues()));
154+
$comments[] = 'One of '.implode('; ', array_unique(array_map('json_encode', $child->getValues())));
155155
}
156156

157157
if (\count($comments)) {

‎src/Symfony/Component/Config/Definition/Dumper/YamlReferenceDumper.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,7 @@ private function writeNode(NodeInterface $node, NodeInterface $parentNode = null
9898
}
9999
}
100100
} elseif ($node instanceof EnumNode) {
101-
$comments[] = 'One of '.implode('; ', array_map('json_encode', $node->getValues()));
101+
$comments[] = 'One of '.implode('; ', array_unique(array_map('json_encode', $node->getValues())));
102102
$default = $node->hasDefaultValue() ? Inline::dump($node->getDefaultValue()) : '~';
103103
} elseif (VariableNode::class === $node::class && \is_array($example)) {
104104
// If there is an array example, we are sure we dont need to print a default value

‎src/Symfony/Component/Config/Definition/EnumNode.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@ class EnumNode extends ScalarNode
2424

2525
public function __construct(?string $name, NodeInterface $parent = null, array $values = [], string $pathSeparator = BaseNode::DEFAULT_PATH_SEPARATOR)
2626
{
27-
$values = array_unique($values);
2827
if (!$values) {
2928
throw new \InvalidArgumentException('$values must contain at least one element.');
3029
}

‎src/Symfony/Component/Config/Tests/Definition/Builder/EnumNodeDefinitionTest.php

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -25,15 +25,6 @@ public function testWithOneValue()
2525
$this->assertEquals(['foo'], $node->getValues());
2626
}
2727

28-
public function testWithOneDistinctValue()
29-
{
30-
$def = new EnumNodeDefinition('foo');
31-
$def->values(['foo', 'foo']);
32-
33-
$node = $def->getNode();
34-
$this->assertEquals(['foo'], $node->getValues());
35-
}
36-
3728
public function testNoValuesPassed()
3829
{
3930
$this->expectException(\RuntimeException::class);
@@ -73,4 +64,12 @@ pu F841 blic function testSetDeprecated()
7364
$this->assertSame('vendor/package', $deprecation['package']);
7465
$this->assertSame('1.1', $deprecation['version']);
7566
}
67+
68+
public function testSameStringCoercedValuesAreDifferent()
69+
{
70+
$def = new EnumNodeDefinition('ccc');
71+
$def->values(['', false, null]);
72+
73+
$this->assertSame(['', false, null], $def->getNode()->getValues());
74+
}
7675
}

‎src/Symfony/Component/Config/Tests/Definition/EnumNodeTest.php

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -71,4 +71,12 @@ public function testWithPlaceHolderWithInvalidValue()
7171
$this->expectExceptionMessage('The value "foo" is not allowed for path "cookie_samesite". Permissible values: "lax", "strict", "none"');
7272
$node->finalize('custom');
7373
}
74+
75+
public function testSameStringCoercedValuesAreDifferent()
76+
{
77+
$node = new EnumNode('ccc', null, ['', false, null]);
78+
$this->assertSame('', $node->finalize(''));
79+
$this->assertFalse($node->finalize(false));
80+
$this->assertNull($node->finalize(null));
81+
}
7482
}

0 commit comments

Comments
 (0)
0