Checkbox

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

  1. bug1297500
  2. Merge into trunk
Proposed by Brendan Donegan
Status: Merged
Approved by: Zygmunt Krynicki
Approved revision: 2895
Merged at revision: 2895
Proposed branch: lp:~brendan-donegan/checkbox/bug1297500
Merge into: lp:checkbox
Diff against target: 12 lines (+1/-1)
1 file modified
checkbox-gui/checkbox-gui/testitemmodel.cpp (+1/-1)
To merge this branch: bzr merge lp:~brendan-donegan/checkbox/bug1297500
Reviewer Review Type Date Requested Status
Zygmunt Krynicki (community) Approve
Review via email: mp+214489@code.launchpad.net

Description of the change

Simple patch to make it so that the partial_id of the job is used to display in the UI, so that namespaces don't take up most of the space.

To post a comment you must log in.
Revision history for this message
Zygmunt Krynicki (zyga) wrote :

+1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'checkbox-gui/checkbox-gui/testitemmodel.cpp'
2--- checkbox-gui/checkbox-gui/testitemmodel.cpp 2013-12-10 17:41:24 +0000
3+++ checkbox-gui/checkbox-gui/testitemmodel.cpp 2014-04-07 09:14:21 +0000
4@@ -178,7 +178,7 @@
5 description = variant.toString();
6 }
7
8- variant = *iface->properties.find("name");
9+ variant = *iface->properties.find("partial_id");
10 if (variant.isValid() && variant.canConvert(QMetaType::QString) ) {
11 testname = variant.toString();
12 }

Subscribers

People subscribed via source and target branches