Edgewall Software
Modify

Opened 11 years ago

Last modified 9 years ago

#7141 new defect

bugzilla2trac.py default behavior of TRAC_CLEAN

Reported by: dekimsey@… Owned by: Christian Boos
Priority: normal Milestone: not applicable
Component: contrib Version:
Severity: normal Keywords: bugzilla2trac.py
Cc: Branch:
Release Notes:
API Changes:

Description

Default behavior of TRAC_CLEAN should really really not be True, this is cruel. Thank god for sys admins who actually back stuff up…..

Oh and kudos on the really effective cleaning mechanism, it was very thorough.

Attachments (2)

bugzilla2trac.py.patch (4.8 KB ) - added by dekimsey@… 11 years ago.
patch referred to in comment:ticket:7141:2
bugzilla2trac.py.2.patch (5.4 KB ) - added by dekimsey@… 11 years ago.
working patch of the implementation

Download all attachments as: .zip

Change History (7)

comment:1 by dekimsey@…, 11 years ago

Component: generalticket system
Owner: changed from Jonas Borgström to Christian Boos

In my setup, I had to import bugzilla tickets into an existing Trac database. The following patch is the changes that I had to make in order to get this to function correctly. Please note the attachment also contains the changes from the patch posted ticket #5585, specifically attachment:ticket:5585:bugzilla_3.0_compatibility.patch.

  • Changed the default of TRAC_CLEAN to false.
  • Changed the return value of hasTickets to be an integer value ( the current index )
  • Changed assertNoTickets to not really assert anything.
  • Modified inserting ticket print out to indicate the bugid ⇒ ticketid conversion
  • Changed addAttachment to accept the ticket id from the arguments
  • Support for bugzilla 3.0, from patch in attachment:ticket:5585:bugzilla_3.0_compatibility.patch
  • Added a printout to state what the starting index for all the bugs to be imported.
  • Modified various references to bugid to use the ticket['id'] instead.

by dekimsey@…, 11 years ago

Attachment: bugzilla2trac.py.patch added

patch referred to in comment:ticket:7141:2

comment:2 by Christian Boos, 11 years ago

Milestone: 0.11.1

About bugzilla2trac.py.patch: it looks empty but isn't, because the internal Trac patch viewer fails, then Pygments tries to render it but produces no output instead of failing - that's why Trac doesn't revert to plain text.

WARNING: HTML preview using <trac.mimeview.patch.PatchRenderer object at 0x02F8D390> failed (Invalid unified diff content)
DEBUG: Trying to render HTML preview using PygmentsRenderer [lineno]

It seems there's a problem with the patch itself:

patching file bugzilla2trac.py
patch: **** malformed patch at line 12: @@ -234,12 +234,13 @@

Please have a look at TracDev/SubmittingPatches. Best is to use svn diff to produce the patches.

comment:3 by dekimsey@…, 11 years ago

I had edited the patch post-creation, probably screwed the syntax a bit in the process. Nonetheless, here is the patch again, this time without any edits on my part :).

by dekimsey@…, 11 years ago

Attachment: bugzilla2trac.py.2.patch added

working patch of the implementation

comment:4 by Christian Boos, 10 years ago

Milestone: 0.12.1not applicable

comment:5 by Christian Boos, 9 years ago

Component: ticket systemcontrib

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The owner will remain Christian Boos.
The ticket will be disowned.
as The resolution will be set.
The owner will be changed from Christian Boos to anonymous.

Add Comment


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