- Fixed DOM bug that was performing the lookup '1.due' by locating task
  1, then attempting a get of the '1.due' attribute, instead of just 'due'.
This commit is contained in:
Paul Beckingham 2011-10-16 23:33:06 -04:00
parent 330cc0218b
commit bc12ac1ee6
2 changed files with 8 additions and 25 deletions

View file

@ -107,28 +107,11 @@ const std::string DOM::get (const std::string& name)
else if (len > 8 && else if (len > 8 &&
name.substr (0, 8) == "context.") name.substr (0, 8) == "context.")
{ {
if (name == "context.program") if (name == "context.program") return /*_cache[name] =*/ context.a3[0]._raw;
return /*_cache[name] =*/ context.a3[0]._raw; else if (name == "context.args") return /*_cache[name] =*/ context.a3.combine ();
else if (name == "context.width") return /*_cache[name] =*/ format (context.terminal_width);
else if (name == "context.args") else if (name == "context.height") return /*_cache[name] =*/ format (context.terminal_height);
{ else throw format (STRING_DOM_UNREC, name);
return /*_cache[name] =*/ context.a3.combine ();
}
else if (name == "context.width")
{
std::stringstream s;
s << context.terminal_width;
return /*_cache[name] =*/ s.str ();
}
else if (name == "context.height")
{
std::stringstream s;
s << context.terminal_height;
return /*_cache[name] =*/ s.str ();
}
else
throw format (STRING_DOM_UNREC, name);
} }
// TODO stats.<name> // TODO stats.<name>
@ -246,7 +229,7 @@ const std::string DOM::get (const std::string& name, const Task& task)
if (attr == "id") return format (ref.id); if (attr == "id") return format (ref.id);
else if (attr == "urgency") return format (ref.urgency_c (), 4, 3); else if (attr == "urgency") return format (ref.urgency_c (), 4, 3);
else if (A3::is_attribute (name, canonical)) return ref.get (canonical); else if (A3::is_attribute (attr, canonical)) return ref.get (canonical);
} }
} }
@ -266,7 +249,7 @@ const std::string DOM::get (const std::string& name, const Task& task)
if (name == "id") return format (ref.id); if (name == "id") return format (ref.id);
else if (name == "urgency") return format (ref.urgency_c (), 4, 3); else if (name == "urgency") return format (ref.urgency_c (), 4, 3);
else if (A3::is_attribute (name, canonical)) return ref.get (canonical); else if (A3::is_attribute (attr, canonical)) return ref.get (canonical);
} }
} }

View file

@ -159,7 +159,7 @@ void E9::eval (const Task& task, std::vector <Arg>& value_stack)
value_stack.push_back (result); value_stack.push_back (result);
} }
// Operand. // Operand (non-op).
else else
{ {
// Derive _value from _raw, and push on the stack. // Derive _value from _raw, and push on the stack.