Modify

Opened 10 years ago

Closed 7 years ago

#2018 closed defect (fixed)

batchmodifyplugin creates Ticket object with id as str, not int, breaking convention

Reported by: thomasvs Owned by: CuriousCurmudgeon
Priority: normal Component: BatchModifyPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.11

Description

This caused my ITicketChangeListener to break. I grepped through trac source code and everywhere Ticket objects are created with an int.

I guess this should also be explicitly documented as such, there's no point in allowing both IMO.

Attaching patch.

Attachments (1)

batchmod.patch (489 bytes) - added by thomasvs 10 years ago.

Download all attachments as: .zip

Change History (5)

Changed 10 years ago by thomasvs

Attachment: batchmod.patch added

comment:1 Changed 9 years ago by bill.coffman@…

Pls note, the patch of thomasvs fixed my error. Thanks Thomas!

I have timingandestimationplugin, and just upgraded from 0.10 to 0.11.1. Batch modifications would work correctly on the first change only, and then stop with an internal error, TypeError: int argument required, and a stack trace.

Although this patch was created for 0.10, it fixes my problem with 0.11.1, and BatchModifyPlugin-0.2.

I strongly recommend the author incorporates this patch (or equivalent) into his code.

comment:2 Changed 7 years ago by CuriousCurmudgeon

Owner: changed from ashwin_phatak to CuriousCurmudgeon
Status: newassigned

comment:3 Changed 7 years ago by CuriousCurmudgeon

(In [7416]) refs #2018

  • Cast id to int when creating Ticket object. Thanks to thomasvs.

comment:4 Changed 7 years ago by CuriousCurmudgeon

Resolution: fixed
Status: assignedclosed
Trac Release: 0.100.11

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain CuriousCurmudgeon.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.