- Many files were missing an explicit cmake.h include. Some were not including
it at all. Now it's used almost everywhere.
(cherry picked from commit 82ae86979c497e6d1d0c6b2b5a55aa379ec82c98)
- Remove duplicate modify call to TDB2.
- Check if quitting in bulk change only if permission fails.
- Correct test logic (the unit for each operation is the second, not the day).
- Consistency in test for feature.1013.t
- Correctly color deleted tasks.
test
- Fixed bug #1038, which prints blank lines with bulk changes and when the
verbose attributes does not specify it.
- Lines do a better separation between each changes also.
- Remove the test that prevents 'until' attributes to be modified for
non-recurring tasks (thanks to Stéphane Pezennec).
- Unit test.
Signed-off-by: Paul Beckingham <paul@beckingham.net>
- When bulk tasks are modified, the "project changed" messages are
now retained until processing is completed, so that only one message
per project is generated.
- Fixed bug that was not properly evaluating cases where recurrence was
being removed from a task.
Signed-off-by: Paul Beckingham <paul@beckingham.net>
- Modified the 'modify' command so that it only applies changes if
the requested changes made a difference. For example if the command
task 1 mod pri:H
was followed by:
task 1 mod pri:H
then the second command will report 'task not changed'.
- While fixing bug.360.t, discovered a bigger problem, which is not yet
fixed. When a due date is removed from a recurring child task, the
imask, parent and recur attributes should also be removed. Similarly
when a recur frequency is removed from a recurring child task, the
imask and parent should also be removed.
- Net result: two new failing tests.
- Converted the 'modify' command to TDB2. This has a negative impact
on unit tests, but that represents bugs and ommissions in TDB2,
not flaws in the 'modify' command. I hope.
- Re-enabled the modify command.
- Re-implemented Arguments::inject_defaults to accomodate the idea
of the default command, the modify command without the modify
keyword, and the automatic info command.
- Added proper handling for sequences, in that they must be contiguous.
- Added placeholder code for default command, and automatic info report.
- Eliminated Context::parse.
- Eliminated Context::run Timer, because when it goes out of scope, it
adds timing messages to the deubg output, which at the end of Context::run
has already been displayed. In addition, the Context::dispatch timer
is about 0.2 milliseconds shorter, so the two are redundant.
- Eliminated stored arg_overrides and file_override in Context.
- Removed Filter, Subst, Task, Sequence from Context.
- Remove shadow file support. Hallelujah.
- Disabled/commented out most commands, ready for the big transition
to the new parsing style.
- Obsoleted Subst.{h,cpp}.
- Obsoleted Command::exectute 'commandLine' argument. It is worse
than unnecessary, it is an uncategorized raw argument string, which
is only really useful for the 'execute' command, which itself now
calls Arguments::combine to reconstruct the command line string.
