Opened 14 years ago

Last modified 12 years ago

#2764 enhancement new

Replace deprecation warning in _ThreePhaseEvent.removeTrigger_BEFORE with an exception

Reported by: Jean-Paul Calderone Owned by:
Priority: low Milestone:
Component: core Keywords:
Cc: Thijs Triemstra Branch:


This behavior was initially allowed (by way of the reactor's system events system) but was implemented in a somewhat buggy manner (encountering it might result in a different trigger being skipped). That bug is fixed by #2509 by ignoring the remove request, except to issue a deprecation warning. The change was made after the release of Twisted 2.5, so it will probably appear first in Twisted 2.6. At the earliest, we can transform it into an exception in Twisted 2.7, so tentatively scheduling this ticket for that release.

Change History (3)

comment:1 Changed 12 years ago by Thijs Triemstra

Cc: Thijs Triemstra added

I guess 2.6 here means 8.x and 2.7's the upcoming 9.0

comment:2 Changed 12 years ago by Jonathan Lange

Milestone: Twisted-10.0

This ticket is low priority, old, doesn't have a suggested fix and 10.0.0 is already out. I'm removing it from any release milestone.

comment:3 Changed 11 years ago by <automation>

Owner: Glyph deleted
Note: See TracTickets for help on using tickets.