Development

#4740 ([PATCH] YAML dumper should remove carriage returns (\r))

You must first sign up to be able to contribute.

Ticket #4740 (closed defect: fixed)

Opened 8 months ago

Last modified 7 months ago

[PATCH] YAML dumper should remove carriage returns (\r)

Reported by: bschussek Assigned to: fabien
Priority: minor Milestone: 1.1.6
Component: yaml Version:
Keywords: Cc:
Qualification: Unreviewed

Description

The class sfYamlInline currently only replaces newlines by the string "\n" when it reads multiline strings. Carriage returns should be returned as well though, which they are not currently.

I attached a patch which just removes carriage returns. An alternative would be to replace them by "\r".

Attachments

yaml_parser_patch.txt (1.0 kB) - added by bschussek on 10/24/08 15:12:49.

Change History

10/24/08 15:12:49 changed by bschussek

  • attachment yaml_parser_patch.txt added.

11/27/08 09:56:23 changed by fabien

  • status changed from new to closed.
  • resolution set to fixed.

(In [13391]) [1.1, 1.2] YAML dumper should remove carriage returns (\r) (closes #4740)

11/27/08 09:56:35 changed by fabien

  • milestone set to 1.1.6.

The Sensio Labs Network

Since 1998, Sensio Labs has been promoting the Open-Source software movement by providing quality web application development, training, consulting, and supporting several large Open-Source projects.

Open-Source Software

Services

> More