Maintain formatting on comment field in assests.

Description

Carriage returns and line feeds are dropped from the comment field when it's stuffed in the database. The field type probably needs to be changed to preserve simple formatting.

Environment

Operating System: All Platform: PC

Acceptance / Success Criteria

None

Lucidchart Diagrams

Activity

Show:

Former user September 8, 2008 at 7:12 PM

merged to 1.6 release in revision 10425

Benjamin Reed August 7, 2008 at 8:40 PM

heh, actually, r10075 and r10076, silly formatting bug

Benjamin Reed August 7, 2008 at 8:39 PM

alright, I changed everything to use opencsv, and now we can make real CSV files, not just fake them up by stripping carriage returns and commas out of the input fields.

Fixed in 1.6-testing, r10075

Benjamin Reed August 7, 2008 at 4:39 PM

it's not just a matter of the text field, the modify-asset code strips out \n, \r, \f, and "," because the importer/exporter code relies on things being a one-line CSV

The solution is to encode those things, I guess, maybe I should look into a "real" CSV tool, since it will encode things right.

Tarus Balog July 30, 2008 at 3:02 PM

I wonder if we can just make the comment field a text object.

Fixed

Details

Assignee

Reporter

Components

Fix versions

Affects versions

Priority

PagerDuty

Created March 25, 2008 at 1:08 PM
Updated January 27, 2017 at 4:25 PM
Resolved September 8, 2008 at 7:12 PM

Flag notifications