See #2310093: Config install and import should map from storage record not set properties directly

Include the better upstream fix - https://github.com/symfony/symfony/pull/11976

The issue we have to best illustrated by the following code:

$yaml = new \Symfony\Component\Yaml\Yaml();

$expected = array('float' => (float) 1);
$test = $yaml->parse($yaml->dump($expected));

if ($expected === $test) {
  print "match!\n";
}
else {
  var_dump($test);
}

which outputs

array(1) {
  'float' =>
  int(1)
}

Our float has become an integer!

Files: 
CommentFileSizeAuthor
#1 2350917.1.patch41.58 KBalexpott
PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 78,987 pass(es). View

Comments

alexpott’s picture

FileSize
41.58 KB
PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 78,987 pass(es). View
Berdir’s picture

Status: Needs review » Reviewed & tested by the community

Confirmed that this works now, patch is green, no reason to wait, let's unblock the storage issue.

  • catch committed ee39402 on 8.0.x
    Issue #2350917 by alexpott: Update Symfony YAML library to support whole...
catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.0.x, thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.