Ticket #5114 enhancement closed fixed

Opened 2 years ago

Last modified 2 years ago

LoopingCall should be resetable

Reported by: chris- Owned by: therve
Priority: normal Milestone:
Component: core Keywords: patch, review
Cc: thijs Branch: branches/reset-loop-5114
Author: therve Launchpad Bug:

Description

It would be a good thing if the LoopingCall would be "resetable", meaning it should be easy to stop the currently scheduled call and reschedule it with the currently set interval, providing a nicer alternative to stopping the call and starting a new LoopingCall with the exact same settings. The following patch implements this.

Attachments

reset_loop.patch Download (1.5 KB) - added by chris- 2 years ago.

Change History

Changed 2 years ago by chris-

1

Changed 2 years ago by thijs

  • keywords patch, review added; patch removed
  • cc thijs added

Adding missing 'review' keyword (see ReviewProcess).

2

Changed 2 years ago by therve

  • owner set to therve

3

Changed 2 years ago by therve

  • branch set to branches/reset-loop-5114
  • branch_author set to therve

(In [31998]) Branching to 'reset-loop-5114'

4

Changed 2 years ago by therve

  • status changed from new to closed
  • resolution set to fixed

(In [32000]) Merge reset-loop-5114

Author: chris Reviewer: therve Fixes: #5114

Add a reset method to LoopingCall to be able to stop it and reschedule at once.

Note: See TracTickets for help on using tickets.