Bug #803
- Here's a patch to make Taskwarrior not display the prompt and not delete the parent task if a recurring task is deleted when rc.confirmation is false. Signed-off-by: Paul Beckingham <paul@beckingham.net>
This commit is contained in:
committed by
Paul Beckingham
parent
56af60ceaf
commit
0962b66928
1
AUTHORS
1
AUTHORS
@@ -54,6 +54,7 @@ The following submitted code, packages or analysis, and deserve special thanks:
|
|||||||
Ben Boeckel
|
Ben Boeckel
|
||||||
Bryce Harrington
|
Bryce Harrington
|
||||||
Brendan O'Flaherty
|
Brendan O'Flaherty
|
||||||
|
Matt Kraai
|
||||||
|
|
||||||
Thanks to the following, who submitted detailed bug reports and excellent
|
Thanks to the following, who submitted detailed bug reports and excellent
|
||||||
suggestions:
|
suggestions:
|
||||||
|
|||||||
@@ -154,6 +154,8 @@
|
|||||||
Ben Boeckel).
|
Ben Boeckel).
|
||||||
+ Applied documentation patch for #799, fixing a broken link in the man page
|
+ Applied documentation patch for #799, fixing a broken link in the man page
|
||||||
(thanks to Bryce Harrington).
|
(thanks to Bryce Harrington).
|
||||||
|
+ Applied patch for #803, allowing rc.confirmation to bypass confirmation of
|
||||||
|
the deletion of a recurring task (thanks to Matt Kraai).
|
||||||
|
|
||||||
# Untracked Bugs, biggest first.
|
# Untracked Bugs, biggest first.
|
||||||
+ Fixed bug that required the '%YAML' prologue in a YAML import.
|
+ Fixed bug that required the '%YAML' prologue in a YAML import.
|
||||||
|
|||||||
@@ -92,7 +92,8 @@ int CmdDelete::execute (std::string& output)
|
|||||||
std::string parent = task->get ("parent");
|
std::string parent = task->get ("parent");
|
||||||
if (parent != "")
|
if (parent != "")
|
||||||
{
|
{
|
||||||
if (confirm (STRING_CMD_DELETE_CONF_RECUR))
|
if (context.config.getBoolean ("confirmation") &&
|
||||||
|
confirm (STRING_CMD_DELETE_CONF_RECUR))
|
||||||
{
|
{
|
||||||
// Scan all pending tasks for siblings of this task, and the parent
|
// Scan all pending tasks for siblings of this task, and the parent
|
||||||
// itself, and delete them.
|
// itself, and delete them.
|
||||||
|
|||||||
Reference in New Issue
Block a user