Ubuntu Software Center

Merge lp:~mvo/software-center/sca-debtags into lp:software-center

  1. sca-debtags
  2. Merge into trunk
Proposed by Michael Vogt
Status: Merged
Merged at revision: 2749
Proposed branch: lp:~mvo/software-center/sca-debtags
Merge into: lp:software-center
Diff against target: 22 lines (+4/-1)
1 file modified
softwarecenter/db/update.py (+4/-1)
To merge this branch: bzr merge lp:~mvo/software-center/sca-debtags
Reviewer Review Type Date Requested Status
Gary Lasker (community) Approve
Review via email: mp+93009@code.launchpad.net

Description of the change

This branch fixes the debtags parsing from the software-center-agent

To post a comment you must log in.
Revision history for this message
Gary Lasker (gary-lasker) wrote :

Looks just fine, thank you mvo!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'softwarecenter/db/update.py'
2--- softwarecenter/db/update.py 2012-02-13 10:52:22 +0000
3+++ softwarecenter/db/update.py 2012-02-14 15:28:17 +0000
4@@ -159,7 +159,7 @@
5 'Comment' : 'Comment',
6 'Version' : 'version',
7 'Supported-Distros': 'series',
8- 'Tags' : 'tags',
9+ # tags are special, see _apply_exception
10 }
11
12 # map from requested key to a static data element
13@@ -184,6 +184,9 @@
14 self.sca_application.Description = "\n".join(
15 self.sca_application.description.split("\n")[1:]).strip()
16
17+ # debtags is send as a list, but we need it as a comma seperated string
18+ self.sca_application.Tags = ",".join(getattr(self.sca_application, "debtags", []))
19+
20 # we only support a single video currently :/
21 if hasattr(self.sca_application, "video_embedded_html_urls"):
22 self.sca_application.video_embedded_html_url = self.sca_application.video_embedded_html_urls[0]

Subscribers

People subscribed via source and target branches