Merge lp:~bac/launchpad/bug-515171 into lp:launchpad

Proposed by Brad Crittenden on 2010-02-01
Status: Merged
Merged at revision: not available
Proposed branch: lp:~bac/launchpad/bug-515171
Merge into: lp:launchpad
Diff against target: 17 lines (+0/-7)
1 file modified
lib/lp/registry/templates/sourcepackage-portlet-associations.pt (+0/-7)
To merge this branch: bzr merge lp:~bac/launchpad/bug-515171
Reviewer Review Type Date Requested Status
Henning Eggers (community) code 2010-02-01 Approve on 2010-02-01
Review via email: mp+18385@code.launchpad.net
To post a comment you must log in.
Brad Crittenden (bac) wrote :

= Summary =

In review for a previous branch it was suggested I make a macro to display a messaged
based on the plurality of a suggestion count. I inadvertently left the old method in
place too, so now the message appears twice.

== Proposed fix ==

Remove the redundant parts from the page template.

== Pre-implementation notes ==

None

== Implementation details ==

As above

== Tests ==

bin/test -vvt sourcepackage-views.txt

== Demo and Q/A ==

Look at https://edge.launchpad.net/ubuntu/lucid/+source/gimp-help and verify "Is one
of these projects the upstream for this source package?" appears only once.

= Launchpad lint =

Checking for conflicts. and issues in doctests and templates.
Running jslint, xmllint, pyflakes, and pylint.
Using normal rules.

Linting changed files:
  lib/lp/registry/templates/sourcepackage-portlet-associations.pt

Henning Eggers (henninge) :
review: Approve (code)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'lib/lp/registry/templates/sourcepackage-portlet-associations.pt'
2--- lib/lp/registry/templates/sourcepackage-portlet-associations.pt 2010-01-29 15:03:49 +0000
3+++ lib/lp/registry/templates/sourcepackage-portlet-associations.pt 2010-02-01 13:56:17 +0000
4@@ -67,13 +67,6 @@
5 </b>
6 </tal:message>
7
8- <div tal:condition="python: len(view.product_suggestions) > 1">
9- <b>Is one of these projects the upstream for this source
10- package?</b>
11- </div>
12- <div tal:condition="python: len(view.product_suggestions) == 1">
13- <b>Is the following project the upstream for this source package?</b>
14- </div>
15 <div metal:use-macro="context/@@launchpad_form/form">
16 <div class="actions" metal:fill-slot="buttons">
17 <input tal:replace="structure view/link/render"/>