Deluge
Opened 9 years ago
Last modified 22 months ago
#2706 new bug
[Execute] Potential issue when removing torrent with data
| Reported by: | Cas | Owned by: | |
|---|---|---|---|
| Priority: | minor | Milestone: | 2.1.1 |
| Component: | Plugin | Version: | develop |
| Keywords: | Cc: |
Description
A user pointed out that the behaviour of execute with delete event and data removed is unknown. It likely is an issue as the old path will be passed to scripts so probably the best action is to send an empty path. The only issue I can foresee is determining whether data was also removed just from the emitted event so may only be fixable in develop.
Change History (5)
comment:1 Changed 6 years ago by HellFire
comment:3 Changed 5 years ago by Cas
- Milestone changed from 2.0.0 to 2.1.x
Ticket retargeted after milestone closed
comment:5 Changed 22 months ago by Cas
- Milestone changed from 2.1.0 to 2.1.1
Ticket retargeted after milestone closed
Note: See
TracTickets for help on using
tickets.
This is definitely an issue. I'm using the plugin to move data to a separate folder (for ease of organization) when removing a torrent and choosing remove & delete results in some data being moved, rather than none.
I don't think sending an empty path for remove & delete is the best option, depending on what users are using the plugin for, they may still want the path sent. It could specify the action (remove / delete) as an additional argument or environment variable.