GothenburgBitFactory/timewarrior
1
0
Fork 0
mirror of https://github.com/GothenburgBitFactory/timewarrior.git synced 2025-07-07 20:06:39 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

TI-48: Timewarrior hook is not Python3 compatible

Browse source 
- Thanks to Josh Proehl, Armado Martinez.
This commit is contained in:
Paul Beckingham 2017-02-25 14:17:25 -05:00
parent c557174172
commit c3e2759028
3 changed files with 6 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

1
AUTHORS
View file

@ -21,6 +21,7 @@ The following submitted code, packages or analysis, and deserve special thanks:
Felix Wolfsteller Felix Wolfsteller
Jörg Krause Jörg Krause
Richard Brown Richard Brown
Armado Martinez
Thanks to the following, who submitted detailed bug reports and excellent Thanks to the following, who submitted detailed bug reports and excellent
suggestions: suggestions:

2
ChangeLog
View file

@ -14,6 +14,8 @@
(thanks to Michael Meier) (thanks to Michael Meier)
- TI-47 first call successfully creates new database but returns exit status 1 - TI-47 first call successfully creates new database but returns exit status 1
(thanks to Georg Sauthoff). (thanks to Georg Sauthoff).
- TI-48 Timewarrior hook is not Python3 compatible
(thanks to Josh Proehl, Armado Martinez)
- TI-49 new theme files aren't installed - TI-49 new theme files aren't installed
(thanks to Richard Brown). (thanks to Richard Brown).
- TI-51 in the taskwarrior hook, deleting a task doesn't stop the watch - TI-51 in the taskwarrior hook, deleting a task doesn't stop the watch

6
ext/on-modify.timewarrior
View file

@ -57,16 +57,16 @@ combined = ' '.join(['"%s"' % tag for tag in tags]).encode('utf-8').strip()
# Started task. # Started task.
if 'start' in new and not 'start' in old: if 'start' in new and not 'start' in old:
os.system('timew start ' + combined + ' :yes') os.system('timew start ' + combined.decode() + ' :yes')
# Stopped task. # Stopped task.
elif not 'start' in new and 'start' in old: elif not 'start' in new and 'start' in old:
os.system('timew stop ' + combined + ' :yes') os.system('timew stop ' + combined.decode() + ' :yes')
# TI-51 - in the taskwarrior hook, deleting a task doesn't stop the watch # TI-51 - in the taskwarrior hook, deleting a task doesn't stop the watch
# #
# Broadened to: Any task that is active, with a non-pending status should not # Broadened to: Any task that is active, with a non-pending status should not
# be tracked. # be tracked.
elif 'start' in new and new['status'] != 'pending': elif 'start' in new and new['status'] != 'pending':
os.system('timew stop ' + combined + ' :yes') os.system('timew stop ' + combined.decode() + ' :yes')