GothenburgBitFactory/taskwarrior
1
0
Fork 0
mirror of https://github.com/GothenburgBitFactory/taskwarrior.git synced 2025-08-07 13:48:33 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

Hooks

Browse source 
- Implemented new interface.  Testing time.
This commit is contained in:
Paul Beckingham 2014-09-14 15:55:22 -04:00
parent ab72cdad4d
commit 8be6c589da
1 changed files with 14 additions and 36 deletions
Download patch file Download diff file Expand all files Collapse all files

50
src/Hooks.cpp
View file

@ -94,10 +94,11 @@ bool Hooks::enable (bool value)
// //
void Hooks::onLaunch () void Hooks::onLaunch ()
{ {
context.timer_hooks.start ();
if (! _enabled) if (! _enabled)
return; return;
context.timer_hooks.start ();
std::vector <std::string> matchingScripts = scripts ("on-launch"); std::vector <std::string> matchingScripts = scripts ("on-launch");
std::vector <std::string>::iterator i; std::vector <std::string>::iterator i;
for (i = matchingScripts.begin (); i != matchingScripts.end (); ++i) for (i = matchingScripts.begin (); i != matchingScripts.end (); ++i)
@ -151,10 +152,11 @@ void Hooks::onLaunch ()
// //
void Hooks::onExit () void Hooks::onExit ()
{ {
context.timer_hooks.start ();
if (! _enabled) if (! _enabled)
return; return;
context.timer_hooks.start ();
std::vector <std::string> matchingScripts = scripts ("on-exit"); std::vector <std::string> matchingScripts = scripts ("on-exit");
std::vector <std::string>::iterator i; std::vector <std::string>::iterator i;
for (i = matchingScripts.begin (); i != matchingScripts.end (); ++i) for (i = matchingScripts.begin (); i != matchingScripts.end (); ++i)
@ -199,16 +201,16 @@ void Hooks::onExit ()
// //
void Hooks::onAdd (std::vector <Task>& changes) void Hooks::onAdd (std::vector <Task>& changes)
{ {
/*
context.timer_hooks.start ();
if (! _enabled) if (! _enabled)
return; return;
context.timer_hooks.start ();
std::vector <std::string> matchingScripts = scripts ("on-add"); std::vector <std::string> matchingScripts = scripts ("on-add");
std::vector <std::string>::iterator i; std::vector <std::string>::iterator i;
for (i = matchingScripts.begin (); i != matchingScripts.end (); ++i) for (i = matchingScripts.begin (); i != matchingScripts.end (); ++i)
{ {
std::string input = after.composeJSON () + "\n"; std::string input = changes[0].composeJSON () + "\n";
std::string output; std::string output;
std::vector <std::string> args; std::vector <std::string> args;
int status = execute (*i, args, input, output); int status = execute (*i, args, input, output);
@ -219,22 +221,11 @@ void Hooks::onAdd (std::vector <Task>& changes)
if (status == 0) if (status == 0)
{ {
bool first = true; changes.clear ();
for (line = lines.begin (); line != lines.end (); ++line) for (line = lines.begin (); line != lines.end (); ++line)
{ {
if (line->length () && (*line)[0] == '{') if (line->length () && (*line)[0] == '{')
{ changes.push_back (Task (*line));
Task newTask (*line);
// TODO Not sure if this first/!first thing is good.
if (first)
{
after = newTask;
first = false;
}
else
context.tdb2.add (newTask);
}
else else
context.footnote (*line); context.footnote (*line);
} }
@ -250,7 +241,6 @@ void Hooks::onAdd (std::vector <Task>& changes)
} }
context.timer_hooks.stop (); context.timer_hooks.stop ();
*/
} }
//////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////
@ -270,16 +260,16 @@ void Hooks::onAdd (std::vector <Task>& changes)
// //
void Hooks::onModify (const Task& before, std::vector <Task>& changes) void Hooks::onModify (const Task& before, std::vector <Task>& changes)
{ {
/*
context.timer_hooks.start ();
if (! _enabled) if (! _enabled)
return; return;
context.timer_hooks.start ();
std::vector <std::string> matchingScripts = scripts ("on-modify"); std::vector <std::string> matchingScripts = scripts ("on-modify");
std::vector <std::string>::iterator i; std::vector <std::string>::iterator i;
for (i = matchingScripts.begin (); i != matchingScripts.end (); ++i) for (i = matchingScripts.begin (); i != matchingScripts.end (); ++i)
{ {
std::string afterJSON = after.composeJSON (); std::string afterJSON = changes[0].composeJSON ();
std::string input = before.composeJSON () std::string input = before.composeJSON ()
+ "\n" + "\n"
+ afterJSON + afterJSON
@ -294,22 +284,11 @@ void Hooks::onModify (const Task& before, std::vector <Task>& changes)
if (status == 0) if (status == 0)
{ {
bool first = true; changes.clear ();
for (line = lines.begin (); line != lines.end (); ++line) for (line = lines.begin (); line != lines.end (); ++line)
{ {
if (line->length () && (*line)[0] == '{') if (line->length () && (*line)[0] == '{')
{ changes.push_back (Task (*line));
Task newTask (*line);
// TODO Not sure if this first/!first thing is good.
if (first)
{
after = newTask;
first = false;
}
else
context.tdb2.modify (newTask);
}
else else
context.footnote (*line); context.footnote (*line);
} }
@ -325,7 +304,6 @@ void Hooks::onModify (const Task& before, std::vector <Task>& changes)
} }
context.timer_hooks.stop (); context.timer_hooks.stop ();
*/
} }
//////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////