Edgewall Software
Modify

Opened 18 years ago

Closed 4 years ago

#2709 closed defect (wontfix)

bugzilla2trac should convert descriptions LF->CRLF

Reported by: maxb1@… Owned by:
Priority: normal Milestone:
Component: contrib Version: 0.9.3
Severity: minor Keywords: bugzilla2trac contrib
Cc: Branch:
Release Notes:
API Changes:
Internal 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@… 14 years ago.
Patch for the CRLF issue
bugzilla2py-CRLF-patch-univ.diff (1.1 KB ) - added by manuel@… 14 years ago.
Patch for the CRLF issue, universal diff

Download all attachments as: .zip

Change History (11)

comment:1 by sid, 17 years ago

Keywords: contrib added

comment:2 by Christian Boos, 17 years ago

Keywords: bugzilla2trac added
Milestone: none

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

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

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

Owner: Jeroen Ruigrok van der Werven removed
Status: assignednew

comment:5 by Christian Boos, 14 years ago

Component: generalcontrib

by manuel@…, 14 years ago

Attachment: bugzilla2py-CRLF-patch.diff added

Patch for the CRLF issue

comment:6 by manuel@…, 14 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, 14 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@…, 14 years ago

Patch for the CRLF issue, universal diff

in reply to:  7 comment:8 by manuel@…, 14 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.

comment:9 by Ryan J Ollos, 4 years ago

Milestone: not applicable
Resolution: wontfix
Status: newclosed

bugzilla2trac.py is no longer maintainer by Trac developers and is being deleted from the trunk. If you need the plugin, please consider copying it from 1.4-stable and creating a plugin on trac-hacks.org.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The ticket will remain with no owner.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from (none) to the specified user.

Add Comment


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