tonyair/silverstripe-framework
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-framework synced 2024-07-19 09:53:52 +02:00
Code Issues Projects Releases Wiki Activity

Merge pull request #3993 from tractorcow/pulls/3.1/fix-translations

Browse Source 
BUG Fix yaml generation to conform to version 1.1, accepted by transifex
This commit is contained in:
Ingo Schommer 2015-03-10 14:07:48 +13:00
commit d17272084a
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

@ -628,9 +628,13 @@ class i18nTextCollector_Writer_RailsYaml implements i18nTextCollector_Writer {
}
// Write YAML
$oldVersion = sfYaml::getSpecVersion();
sfYaml::setSpecVersion('1.1');
$yamlHandler = new sfYaml();
// 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(
'Level1.Level2.EntityName' => array('Text', 'Context'),
'Level1.OtherEntityName' => array('Other Text', 'Other Context'),
'Level1.BoolTest' => array('True'),
'Level1.FlagTest' => array('No'),
'Level1.TextTest' => array('Maybe')
);
$yaml = <<<YAML
de:
@ -424,6 +427,9 @@ de:
Level2:
EntityName: Text
OtherEntityName: 'Other Text'
BoolTest: 'True'
FlagTest: 'No'
TextTest: Maybe
YAML;
$this->assertEquals($yaml, Convert::nl2os($writer->getYaml($entities, 'de')));