Modify ↓
Opened 17 years ago
Closed 15 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)
Change History (5)
Changed 17 years ago by
Attachment: | batchmod.patch added |
---|
comment:1 Changed 16 years ago by
comment:2 Changed 15 years ago by
Owner: | changed from ashwin_phatak to CuriousCurmudgeon |
---|---|
Status: | new → assigned |
comment:3 Changed 15 years ago by
comment:4 Changed 15 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Trac Release: | 0.10 → 0.11 |
Note: See
TracTickets for help on using
tickets.
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.