gtg team mailing list archive

[Nimit Shah <nimit.svnit@xxxxxxxxx>]

Nimit Shah has proposed merging lp:~nimit-svnit/gtg/bug-1043457 into lp:gtg.

Requested reviews:
  Gtg developers (gtg)

For more details, see:
https://code.launchpad.net/~nimit-svnit/gtg/bug-1043457/+merge/123404

Fix for bug-1043457: gtcli search returns closed tasks

Added one check for open tasks before displaying the tasks
-- 
https://code.launchpad.net/~nimit-svnit/gtg/bug-1043457/+merge/123404
Your team Gtg developers is requested to review the proposed merge of lp:~nimit-svnit/gtg/bug-1043457 into lp:gtg.

=== modified file 'CHANGELOG'
--- CHANGELOG	2012-08-27 11:39:18 +0000
+++ CHANGELOG	2012-09-08 07:59:19 +0000
@@ -52,6 +52,7 @@
     * Fix for bug #897425: Write manpage for gtcli and update other manpages as well
     * Fix for bug-1037051 (Due date is not set for a new subtask), by Nimit Shah
     * Fix for bug #1036955: Due date is not preselected when start date is filled, by Steve Scheel
+    * Fix for bug #1043457: gtcli search returns closed tasks, by Nimit Shah
 
 2012-02-13 Getting Things GNOME! 0.2.9
     * Big refractorization of code, now using liblarch

=== modified file 'gtcli'
--- gtcli	2012-07-23 12:04:01 +0000
+++ gtcli	2012-09-08 07:59:19 +0000
@@ -352,10 +352,11 @@
     gtg = connect_to_gtg()
     tasks = gtg.SearchTasks(expression)
     for task in tasks:
-        text = textwrap.fill(task['title'],
-                             initial_indent='',
-                             subsequent_indent=' ' * 40)
-        print "  %-36s  %s" % (task['id'], text)
+        if task['status'] == 'Active':
+            text = textwrap.fill(task['title'],
+                                 initial_indent='',
+                                 subsequent_indent=' ' * 40)
+            print "  %-36s  %s" % (task['id'], text)
 
 
 def run_command(args):