Merge lp:~mitya57/ubuntu-sponsoring/bold-fix-for-new-packages into lp:~ubuntu-dev/ubuntu-sponsoring/trunk

Proposed by Dmitry Shachnev
Status: Merged
Merged at revision: 140
Proposed branch: lp:~mitya57/ubuntu-sponsoring/bold-fix-for-new-packages
Merge into: lp:~ubuntu-dev/ubuntu-sponsoring/trunk
Diff against target: 23 lines (+6/-2)
1 file modified
sponsors-page.py (+6/-2)
To merge this branch: bzr merge lp:~mitya57/ubuntu-sponsoring/bold-fix-for-new-packages
Reviewer Review Type Date Requested Status
Daniel Holbach (community) Approve
Review via email: mp+208961@code.launchpad.net

Description of the change

Currently people who are in ~ubuntu-dev but not in ~motu are highlighted "bold" for needs-packaging bugs. One fix would be to explicitly check membership in ~motu, but I applied a hack that replaces "ubuntu" with "mathjax-docs" instead (a package that is not in any packageset or main).

To post a comment you must log in.
Revision history for this message
Daniel Holbach (dholbach) wrote :

Thanks. Merged.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'sponsors-page.py'
--- sponsors-page.py 2012-09-07 17:32:18 +0000
+++ sponsors-page.py 2014-03-02 09:56:02 +0000
@@ -30,14 +30,18 @@
30 "package-import",30 "package-import",
31)31)
3232
33# A package that is outside of main and package sets
34UNIVERSE_PACKAGE = "mathjax-docs"
35
33def htmlify_name(person, package, devs, distribution):36def htmlify_name(person, package, devs, distribution):
34 if not person:37 if not person:
35 return ""38 return ""
36 name = person.name39 name = person.name
37 if name not in devs:40 if name not in devs:
38 return name41 return name
39 if package == "ubuntu" or \42 if package == "ubuntu":
40 launchpad.person_can_upload(distribution, person, package):43 package = UNIVERSE_PACKAGE
44 if launchpad.person_can_upload(distribution, person, package):
41 return "<b>%s</b>" % name45 return "<b>%s</b>" % name
42 return "<i>%s</i>" % name46 return "<i>%s</i>" % name
4347

Subscribers

People subscribed via source and target branches