Merge lp:~nimit-svnit/gtg/bug-1043457 into lp:~gtg/gtg/old-trunk

[Nimit Shah]

Fix for bug-1043457: gtcli search returns closed tasks

Added one check for open tasks before displaying the tasks

[Izidor Matušov (izidor)]

It looks good. I will merge it after release of GTG 0.3

review: Approve (code)

[Nimit Shah (nimit-svnit)]

Okay!

[Nimit Shah (nimit-svnit)]

Just a reminder :) Can we merge it now?

[Lionel Dricot (ploum-deactivatedaccount)]

review: Approve

=== 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):