UDA: indicator column did not properly default to 'U'

- Thanks to JDufault.
This commit is contained in:
Paul Beckingham 2015-12-16 14:57:58 -05:00
parent c7b792bf47
commit 900ad3c26b
3 changed files with 16 additions and 2 deletions

View file

@ -278,3 +278,4 @@ suggestions:
Naga Kiran
David Badura
James Dietrich
JDufault

View file

@ -31,6 +31,7 @@
- Fixed broken build for Cygwin and older GCC (thanks to Richard Boß).
- The default configuration is now 256-color only.
- The 'columns' report now shows whether a column is modifiable or read only.
- UDA indicator column did not properly default to 'U' (thanks to JDufault).
- Numerous performance improvements. Taskwarrior 2.5.1 is between X% and
Y% faster than 2.5.0 when running various commands.

View file

@ -116,7 +116,13 @@ void ColumnUDA::measure (Task& task, unsigned int& minimum, unsigned int& maximu
else if (_style == "indicator")
{
if (task.has (_name))
minimum = maximum = utf8_width (context.config.get ("uda." + _name + ".indicator"));
{
auto indicator = context.config.get ("uda." + _name + ".indicator");
if (indicator == "")
indicator = "U";
minimum = maximum = utf8_width (indicator);
}
else
minimum = maximum = 0;
}
@ -171,7 +177,13 @@ void ColumnUDA::render (
else if (_style == "indicator")
{
if (task.has (_name))
renderStringRight (lines, width, color, context.config.get ("uda." + _name + ".indicator"));
{
auto indicator = context.config.get ("uda." + _name + ".indicator");
if (indicator == "")
indicator = "U";
renderStringRight (lines, width, color, indicator);
}
}
}
}