8000 Merge branch '3.1' · symfony/symfony@8794617 · GitHub
[go: up one dir, main page]
Skip to content

Commit 8794617

Browse files
fabpotfabpot
committed
Merge branch '3.1'
* 3.1: Use "more entropy" option for uniqid() reset constraint options
2 parents 98bd062 + 53d2bde commit 8794617

File tree

4 files changed

+36
-2
lines changed

4 files changed

+36
-2
lines changed

‎src/Symfony/Bridge/ProxyManager/Tests/LazyProxy/PhpDumper/ProxyDumperTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,7 @@ public function getProxyCandidates()
107107
$definitions = array(
108108
array(new Definition(__CLASS__), true),
109109
array(new Definition('stdClass'), true),
110-
array(new Definition('foo'.uniqid()), false),
110+
array(new Definition(uniqid('foo', true)), false),
111111
array(new Definition(), false),
112112
);
113113

‎src/Symfony/Bridge/Twig/Command/LintCommand.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
105105
$template .= fread(STDIN, 1024);
106106
}
107107

108-
return $this->display($input, $output, $io, array($this->validate($twig, $template, uniqid('sf_'))));
108+
return $this->display($input, $output, $io, array($this->validate($twig, $template, uniqid('sf_', true))));
109109
}
110110

111111
$filesInfo = $this->getFilesInfo($twig, $filenames);

‎src/Symfony/Component/Validator/Constraint.php

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -129,6 +129,9 @@ public function __construct($options = null)
129129
unset($options['value']);
130130
}
131131

132+
if (is_array($options)) {
133+
reset($options);
134+
}
132135
if (is_array($options) && count($options) > 0 && is_string(key($options))) {
133136
foreach ($options as $option => $value) {
134137
if (array_key_exists($option, $knownOptions)) {

‎src/Symfony/Component/Validator/Tests/ConstraintTest.php

Lines changed: 31 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -206,4 +206,35 @@ public function testGetErrorNameForUnknownCode()
206206
{
207207
Constraint::getErrorName(1);
208208
}
209+
210+
public function testOptionsAsDefaultOption()
211+
{
212+
$constraint = new ConstraintA($options = array('value1'));
213+
214+
$this->assertEquals($options, $constraint->property2);
215+
216+
$constraint = new ConstraintA($options = array('value1', 'property1' => 'value2'));
217+
218+
$this->assertEquals($options, $constraint->property2);
219+
}
220+
221+
/**
222+
* @expectedException \Symfony\Component\Validator\Exception\InvalidOptionsException
223+
* @expectedExceptionMessage The options "0", "5" do not exist
224+
*/
225+
public function testInvalidOptions()
226+
{
227+
new ConstraintA(array('property2' => 'foo', 'bar', 5 => 'baz'));
228+
}
229+
230+
public function testOptionsWithInvalidInternalPointer()
231+
{
232+
$options = array('property1' => 'foo');
233+
next($options);
234+
next($options);
235+
236+
$constraint = new ConstraintA($options);
237+
238+
$this->assertEquals('foo', $constraint->property1);
239+
}
209240
}

0 commit comments

Comments
 (0)
0