tonyair/silverstripe-framework
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-framework synced 2024-10-22 14:05:37 +02:00
Code Issues Projects Releases Wiki Activity

BUG Fix yaml generation to conform to version 1.1, accepted by transifex

Browse Source
This commit is contained in:
Damian Mooyman 2015-03-10 13:42:22 +13:00
parent cf2089bc54
commit 622ad54c5b
2 changed files with 11 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
i18n/i18nTextCollector.php
View File

@ -632,9 +632,13 @@ class i18nTextCollector_Writer_RailsYaml implements i18nTextCollector_Writer {
} }
// Write YAML // Write YAML
$oldVersion = sfYaml::getSpecVersion();
sfYaml::setSpecVersion('1.1');
$yamlHandler = new sfYaml(); $yamlHandler = new sfYaml();
// TODO Dumper can't handle YAML comments, so the context information is currently discarded // TODO Dumper can't handle YAML comments, so the context information is currently discarded
return $yamlHandler->dump(array($locale => $entitiesNested), 99); $result = $yamlHandler->dump(array($locale => $entitiesNested), 99);
sfYaml::setSpecVersion($oldVersion);
return $result;
} }
} }

6
tests/i18n/i18nTextCollectorTest.php
View File

@ -417,6 +417,9 @@ PHP;
$entities = array( $entities = array(
'Level1.Level2.EntityName' => array('Text', 'Context'), 'Level1.Level2.EntityName' => array('Text', 'Context'),
'Level1.OtherEntityName' => array('Other Text', 'Other Context'), 'Level1.OtherEntityName' => array('Other Text', 'Other Context'),
'Level1.BoolTest' => array('True'),
'Level1.FlagTest' => array('No'),
'Level1.TextTest' => array('Maybe')
); );
$yaml = <<<YAML $yaml = <<<YAML
de: de:
@ -424,6 +427,9 @@ de:
Level2: Level2:
EntityName: Text EntityName: Text
OtherEntityName: 'Other Text' OtherEntityName: 'Other Text'
BoolTest: 'True'
FlagTest: 'No'
TextTest: Maybe
YAML; YAML;
$this->assertEquals($yaml, Convert::nl2os($writer->getYaml($entities, 'de'))); $this->assertEquals($yaml, Convert::nl2os($writer->getYaml($entities, 'de')));