Merge lp:~brendan-donegan/checkbox/bug1314516_plainbox into lp:checkbox

[Brendan Donegan]

Really simple - use summary instead of id for displaying jobs during the run command.

[Zygmunt Krynicki (zyga)]

Perhaps tr_summary() ?

review: Needs Fixing

[Brendan Donegan (brendan-donegan)]

Why the question mark? Do you think I need to fix the other 'fixes' i did for GUI and CLI? I completely forgot about tr summary

[lp:~brendan-donegan/checkbox/bug1314516_plainbox]

2982. By Brendan Donegan on 2014-05-09

plainbox: Show summary instead of id for jobs during run command

Signed-off-by: Brendan Donegan <email address hidden>

[Brendan Donegan (brendan-donegan)]

updated to use tr_summary(). still wondering about the other merges

review: Needs Resubmitting

[Zygmunt Krynicki (zyga)]

+1

The question mark was just to subtly imply that I wanted that change. Thanks :-)

review: Approve

=== modified file 'plainbox/plainbox/impl/commands/run.py'
--- plainbox/plainbox/impl/commands/run.py	2014-05-04 14:31:36 +0000
+++ plainbox/plainbox/impl/commands/run.py	2014-05-09 12:49:09 +0000
@@ -447,7 +447,7 @@
                     self.run_single_job(job)
 
     def run_single_job(self, job):
-        print("[ {} ]".format(job.id).center(80, '-'))
+        print("[ {} ]".format(job.tr_summary()).center(80, '-'))
         description = job.tr_description()
         if description is not None:
             print(description)