Edgewall Software
Modify

Opened 13 years ago

Last modified 9 years ago

#2709 new defect

bugzilla2trac should convert descriptions LF->CRLF

Reported by: maxb1@… Owned by:
Priority: normal Milestone: not applicable
Component: contrib Version: 0.9.3
Severity: minor Keywords: bugzilla2trac contrib
Cc: Branch:
Release Notes:
API Changes:

Description

The first time any of my bugzilla2trac-ed tickets was changed in any way, the description showed as spuriously changed - this was because all the LFs were translated to CRLFs somewhere in doing the round-trip between the web-browser and server.

To prevent this, bugzilla2trac should convert descriptions (and probably comments too) from LF line endings to CRLF line endings before storing them in the trac database.

Attachments (2)

bugzilla2py-CRLF-patch.diff (435 bytes ) - added by manuel@… 9 years ago.
Patch for the CRLF issue
bugzilla2py-CRLF-patch-univ.diff (1.1 KB ) - added by manuel@… 9 years ago.
Patch for the CRLF issue, universal diff

Download all attachments as: .zip

Change History (10)

comment:1 by sid, 12 years ago

Keywords: contrib added

comment:2 by Christian Boos, 12 years ago

Keywords: bugzilla2trac added
Milestone: none

comment:3 by Jeroen Ruigrok van der Werven, 11 years ago

Owner: changed from Jonas Borgström to Jeroen Ruigrok van der Werven
Status: newassigned

comment:4 by Jeroen Ruigrok van der Werven, 9 years ago

Owner: Jeroen Ruigrok van der Werven removed
Status: assignednew

comment:5 by Christian Boos, 9 years ago

Component: generalcontrib

by manuel@…, 9 years ago

Attachment: bugzilla2py-CRLF-patch.diff added

Patch for the CRLF issue

comment:6 by manuel@…, 9 years ago

The attachment bugzilla2py-CRLF-patch.diff contains a patch which seems to address the issue described in this entry (i.e. converting LF to CRLF during import)

comment:7 by Christian Boos, 9 years ago

There's a typo in the patch (s/pub/sub/), and please use a unified diff format for the patch, thanks!

by manuel@…, 9 years ago

Patch for the CRLF issue, universal diff

in reply to:  7 comment:8 by manuel@…, 9 years ago

Replying to cboos:

There's a typo in the patch (s/pub/sub/), and please use a unified diff format for the patch, thanks!

Fixed the typo, used GNU diff. Feel free to remove the old attachment.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The ticket will remain with no owner.
The ticket will be disowned.
as The resolution will be set.
The owner will be changed from (none) to anonymous.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.