Launchpad itself

Merge lp:~edwin-grubbs/launchpad/bug-588252-default-radio-button into lp:launchpad

  1. bug-588252-default-radio-button
  2. Merge into devel
Proposed by Edwin Grubbs
Status: Merged
Approved by: Curtis Hovey
Approved revision: no longer in the source branch.
Merged at revision: 11425
Proposed branch: lp:~edwin-grubbs/launchpad/bug-588252-default-radio-button
Merge into: lp:launchpad
Diff against target: 11 lines (+1/-1)
1 file modified
lib/lp/registry/browser/sourcepackage.py (+1/-1)
To merge this branch: bzr merge lp:~edwin-grubbs/launchpad/bug-588252-default-radio-button
Reviewer Review Type Date Requested Status
Curtis Hovey (community) code + ui Approve
Review via email: mp+33466@code.launchpad.net

Description of the change

This branch makes the "Choose another upstream project" the default radio button on the source package +index page, when the source package does not have an upstream link already.

You can see the change on this page:
https://launchpad.dev/ubuntu/warty/+source/foobar

To post a comment you must log in.
Revision history for this message
Curtis Hovey (sinzui) wrote :

Thanks looks good to land

review: Approve (code + ui)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'lib/lp/registry/browser/sourcepackage.py'
2--- lib/lp/registry/browser/sourcepackage.py 2010-08-20 20:31:18 +0000
3+++ lib/lp/registry/browser/sourcepackage.py 2010-08-23 22:35:55 +0000
4@@ -542,7 +542,7 @@
5 self.form_fields = Fields(
6 Choice(__name__='upstream',
7 title=_('Registered upstream project'),
8- default=None,
9+ default=self.other_upstream,
10 vocabulary=upstream_vocabulary,
11 required=True))
12