keep trailing newlines when dumping multi-line strings
This commit is contained in:
parent
04c67e61e2
commit
4c513c24c7
@ -72,10 +72,23 @@ class Dumper
|
||||
// If the first line starts with a space character, the spec requires a blockIndicationIndicator
|
||||
// http://www.yaml.org/spec/1.2/spec.html#id2793979
|
||||
$blockIndentationIndicator = (' ' === substr($value, 0, 1)) ? (string) $this->indentation : '';
|
||||
$output .= sprintf('%s%s%s |%s-', $prefix, $dumpAsMap ? Inline::dump($key, $flags).':' : '-', '', $blockIndentationIndicator);
|
||||
|
||||
if (isset($value[-2]) && "\n" === $value[-2] && "\n" === $value[-1]) {
|
||||
$blockChompingIndicator = '+';
|
||||
} elseif ("\n" === $value[-1]) {
|
||||
$blockChompingIndicator = '';
|
||||
} else {
|
||||
$blockChompingIndicator = '-';
|
||||
}
|
||||
|
||||
$output .= sprintf('%s%s%s |%s%s', $prefix, $dumpAsMap ? Inline::dump($key, $flags).':' : '-', '', $blockIndentationIndicator, $blockChompingIndicator);
|
||||
|
||||
foreach (explode("\n", $value) as $row) {
|
||||
$output .= sprintf("\n%s%s%s", $prefix, str_repeat(' ', $this->indentation), $row);
|
||||
if ('' === $row) {
|
||||
$output .= "\n";
|
||||
} else {
|
||||
$output .= sprintf("\n%s%s%s", $prefix, str_repeat(' ', $this->indentation), $row);
|
||||
}
|
||||
}
|
||||
|
||||
continue;
|
||||
|
@ -567,7 +567,7 @@ YAML;
|
||||
], 4, 0, Yaml::DUMP_MULTI_LINE_LITERAL_BLOCK));
|
||||
}
|
||||
|
||||
public function testNoTrailingNewlineWhenDumpingAsMultiLineLiteralBlock()
|
||||
public function testNoExtraTrailingNewlineWhenDumpingAsMultiLineLiteralBlock()
|
||||
{
|
||||
$data = [
|
||||
"a\nb",
|
||||
@ -579,6 +579,44 @@ YAML;
|
||||
$this->assertSame($data, Yaml::parse($yaml));
|
||||
}
|
||||
|
||||
public function testDumpTrailingNewlineInMultiLineLiteralBlocks()
|
||||
{
|
||||
$data = [
|
||||
'clip 1' => "one\ntwo\n",
|
||||
'clip 2' => "one\ntwo\n",
|
||||
'keep 1' => "one\ntwo\n",
|
||||
'keep 2' => "one\ntwo\n\n",
|
||||
'strip 1' => "one\ntwo",
|
||||
'strip 2' => "one\ntwo",
|
||||
];
|
||||
$yaml = $this->dumper->dump($data, 2, 0, Yaml::DUMP_MULTI_LINE_LITERAL_BLOCK);
|
||||
|
||||
$expected = <<<YAML
|
||||
'clip 1': |
|
||||
one
|
||||
two
|
||||
'clip 2': |
|
||||
one
|
||||
two
|
||||
'keep 1': |
|
||||
one
|
||||
two
|
||||
'keep 2': |+
|
||||
one
|
||||
two
|
||||
|
||||
'strip 1': |-
|
||||
one
|
||||
two
|
||||
'strip 2': |-
|
||||
one
|
||||
two
|
||||
YAML;
|
||||
|
||||
$this->assertSame($expected, $yaml);
|
||||
$this->assertSame($data, Yaml::parse($yaml));
|
||||
}
|
||||
|
||||
public function testZeroIndentationThrowsException()
|
||||
{
|
||||
$this->expectException('InvalidArgumentException');
|
||||
|
Reference in New Issue
Block a user