Edgewall Software
Modify

Opened 14 years ago

Closed 2 years ago

#7141 closed defect (wontfix)

bugzilla2trac.py default behavior of TRAC_CLEAN

Reported by: dekimsey@… Owned by: Christian Boos
Priority: normal Milestone:
Component: contrib Version:
Severity: normal Keywords: bugzilla2trac.py
Cc: Branch:
Release Notes:
API Changes:
Internal 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@… 14 years ago.
patch referred to in comment:ticket:7141:2
bugzilla2trac.py.2.patch (5.4 KB ) - added by dekimsey@… 14 years ago.
working patch of the implementation

Download all attachments as: .zip

Change History (8)

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

Attachment: bugzilla2trac.py.patch added

patch referred to in comment:ticket:7141:2

comment:2 by Christian Boos, 14 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@…, 14 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@…, 14 years ago

Attachment: bugzilla2trac.py.2.patch added

working patch of the implementation

comment:4 by Christian Boos, 13 years ago

Milestone: 0.12.1not applicable

comment:5 by Christian Boos, 12 years ago

Component: ticket systemcontrib

comment:6 by Ryan J Ollos, 2 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 owner will remain Christian Boos.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Christian Boos 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.