rules: Reduce code duplication

This commit is contained in:
Tomas Babej 2020-12-14 21:46:52 -05:00
parent 2336c4039d
commit e37ab8a9ed
No known key found for this signature in database
GPG key ID: B0747C6578F7D2F5

View file

@ -224,43 +224,23 @@ static void colorizeUDA (Task& task, const std::string& rule, const Color& base,
//////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////
static void colorizeDue (Task& task, const Color& base, Color& c, bool merge) static void colorizeDue (Task& task, const Color& base, Color& c, bool merge)
{ {
if (task.has ("due")) if (task.is_due ())
{
auto status = task.getStatus ();
if (status != Task::completed &&
status != Task::deleted &&
task.getDateState ("due") == Task::dateAfterToday)
applyColor (base, c, merge); applyColor (base, c, merge);
} }
}
//////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////
static void colorizeDueToday (Task& task, const Color& base, Color& c, bool merge) static void colorizeDueToday (Task& task, const Color& base, Color& c, bool merge)
{ {
if (task.has ("due")) if (task.is_duetoday ())
{
auto status = task.getStatus ();
auto dateState = task.getDateState ("due");
if (status != Task::completed &&
status != Task::deleted &&
(dateState == Task::dateLaterToday || dateState == Task::dateEarlierToday))
applyColor (base, c, merge); applyColor (base, c, merge);
} }
}
//////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////
static void colorizeOverdue (Task& task, const Color& base, Color& c, bool merge) static void colorizeOverdue (Task& task, const Color& base, Color& c, bool merge)
{ {
if (task.has ("due")) if (task.is_overdue ())
{
auto status = task.getStatus ();
auto dateState = task.getDateState ("due");
if (status != Task::completed &&
status != Task::deleted &&
(dateState == Task::dateBeforeToday || dateState == Task::dateEarlierToday))
applyColor (base, c, merge); applyColor (base, c, merge);
} }
}
//////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////
static void colorizeRecurring (Task& task, const Color& base, Color& c, bool merge) static void colorizeRecurring (Task& task, const Color& base, Color& c, bool merge)