#12955 closed defect (duplicate)
batch-modify IGNORES plugin classes implementing the ITicketManipulator (validate_ticket before save), possibly ITicketChangeListener as well
Reported by: | Owned by: | ||
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | ticket system | Version: | 1.2.2 |
Severity: | normal | Keywords: | batch-modify, ITicketManipulator, ITicketChangeListener |
Cc: | Branch: | ||
Release Notes: | |||
API Changes: | |||
Internal Changes: |
Description
I use the ITicketManipulator extension point to implement a plugin that block specific changes to tickets by using the "validate_ticket" function before they get changed. This works well in the "Ticket View", but …
I just noticed, that on batch-modify my class implementing ITicketManipulator is ignored and all actions that the class should avoid can be easily performed on batch editing.
I suggest that the "batch-modify" has to ensure that the general extensions points of the ticket systems are correctly called ("validate_ticket" of all classes implementing ITicketManipulator needs to be called). This also counts for the ITicketChangeListener ("ticket_changed" and "ticket_deleted" need to be called as well)
Attachments (0)
Change History (2)
comment:1 by , 7 years ago
Resolution: | → duplicate |
---|---|
Status: | new → closed |
comment:2 by , 7 years ago
Replying to Stefan Gerstmayr <kyb.stefan@…>:
This also counts for the ITicketChangeListener ("ticket_changed" and "ticket_deleted" need to be called as well)
There was never a defect with ITicketChangeListener
. The change listener methods are called by Ticket.save_changes
: tags/trac-1.2.2/trac/ticket/batch.py@:209#L186.
Duplicate of #11723.