Merge lp:~dobey/unity-scope-click/show-publisher into lp:unity-scope-click/devel

[dobey]

Display the publisher in the header, not the first line of description.

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:223
http://jenkins.qa.ubuntu.com/job/unity-team-unity-scope-click-devel-ci/14/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-team-unity-scope-click-devel-trusty-amd64-ci/14
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-team-unity-scope-click-devel-trusty-armhf-ci/14
        deb: http://jenkins.qa.ubuntu.com/job/unity-team-unity-scope-click-devel-trusty-armhf-ci/14/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-team-unity-scope-click-devel-trusty-i386-ci/14

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity-team-unity-scope-click-devel-ci/14/rebuild

review: Approve (continuous-integration)

[Alejandro J. Cura (alecu)]

Looks good.

review: Approve

=== modified file 'scope/click/preview.cpp'
--- scope/click/preview.cpp	2014-04-25 18:38:01 +0000
+++ scope/click/preview.cpp	2014-04-28 17:46:20 +0000
@@ -143,12 +143,9 @@
 
     scopes::PreviewWidget header("hdr", "header");
     header.add_attribute_value("title", scopes::Variant(details.package.title));
-    if (!details.description.empty())
+    if (!details.publisher.empty())
     {
-        std::stringstream ss(details.description);
-        std::string first_line;
-        if (std::getline(ss, first_line))
-            header.add_attribute_value("subtitle", scopes::Variant(first_line));
+        header.add_attribute_value("subtitle", scopes::Variant(details.publisher));
     }
     if (!details.package.icon_url.empty())
         header.add_attribute_value("mascot", scopes::Variant(details.package.icon_url));