Merge lp:~ivo-kracht/launchpad/bug-353097 into lp:launchpad

Proposed by Ivo Kracht
Status: Merged
Approved by: Abel Deuring
Approved revision: no longer in the source branch.
Merged at revision: 15406
Proposed branch: lp:~ivo-kracht/launchpad/bug-353097
Merge into: lp:launchpad
Diff against target: 97 lines (+17/-14)
4 files modified
lib/lp/bugs/browser/bugalsoaffects.py (+2/-2)
lib/lp/bugs/stories/bug-also-affects/xx-bug-also-affects.txt (+10/-7)
lib/lp/bugs/stories/bug-also-affects/xx-bugtracker-information.txt (+4/-4)
lib/lp/bugs/stories/bugtask-searches/xx-advanced-upstream-pending-bugwatch.txt (+1/-1)
To merge this branch: bzr merge lp:~ivo-kracht/launchpad/bug-353097
Reviewer Review Type Date Requested Status
Raphaël Badin (community) Approve
Francesco Banconi (community) code* Approve
Review via email: mp+110018@code.launchpad.net

Commit message

fix for bug 353097: Marking a Bug as "Requiring Forwarding" text is missleading...

Description of the change

I changed the misleading text to <<I want to add this upstream project to the bug report, but someone must find or report this bug in the upstream bug tracker>> as proposed by Curtis Hovey in the bug discussion and changed several tests which checked the text.

Pre-imp call with adeuring

= Launchpad lint =

Checking for conflicts and issues in changed files.

Linting changed files:
  lib/lp/bugs/browser/bugalsoaffects.py
  lib/lp/bugs/stories/bug-also-affects/xx-bug-also-affects.txt
  lib/lp/bugs/stories/bug-also-affects/xx-bugtracker-information.txt
  lib/lp/bugs/stories/bugtask-searches/xx-advanced-upstream-pending-bugwatch.txt

./lib/lp/bugs/stories/bug-also-affects/xx-bug-also-affects.txt
     261: want exceeds 78 characters.
./lib/lp/bugs/stories/bug-also-affects/xx-bugtracker-information.txt
       1: narrative uses a moin header.
./lib/lp/bugs/stories/bugtask-searches/xx-advanced-upstream-pending-bugwatch.txt
      40: Comment belongs in narrative.
      41: Comment belongs in narrative.
       1: narrative uses a moin header.
      11: narrative uses a moin header.

These errors existed before and have nothing to do with my changes so I didn't fix them.

Tests:
./bin/test bugs -t xx-bugtracker-information.txt
./bin/test bugs -t xx-bug-also-affects.txt

To post a comment you must log in.
Revision history for this message
Francesco Banconi (frankban) wrote :

Looks good, thank you.
Tests pass (also xx-advanced-upstream-pending-bugwatch.txt): waiting for Raphael review.

review: Approve (code*)
Revision history for this message
Raphaël Badin (rvb) wrote :

A great first contribution, thanks Ivo!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'lib/lp/bugs/browser/bugalsoaffects.py'
--- lib/lp/bugs/browser/bugalsoaffects.py 2012-02-01 15:46:43 +0000
+++ lib/lp/bugs/browser/bugalsoaffects.py 2012-06-13 10:04:24 +0000
@@ -511,8 +511,8 @@
511# mailing lists."511# mailing lists."
512512
513 UNLINKED_UPSTREAM = Item(513 UNLINKED_UPSTREAM = Item(
514 """I just want to register that it is upstream right now; \514 """I want to add this upstream project to the bug report, but someone\
515 I don't have any way to link it.515 must find or report this bug in the upstream bug tracker.
516516
517 Launchpad will record that.517 Launchpad will record that.
518 """)518 """)
519519
=== modified file 'lib/lp/bugs/stories/bug-also-affects/xx-bug-also-affects.txt'
--- lib/lp/bugs/stories/bug-also-affects/xx-bug-also-affects.txt 2012-04-04 05:46:26 +0000
+++ lib/lp/bugs/stories/bug-also-affects/xx-bug-also-affects.txt 2012-06-13 10:04:24 +0000
@@ -284,20 +284,22 @@
284 [ ]284 [ ]
285 ( ) I have already emailed an upstream bug contact:285 ( ) I have already emailed an upstream bug contact:
286 [ ]286 [ ]
287 ( ) I just want to register that it is upstream right now; I287 ( ) I want to add this upstream project to the bug report, but
288 don't have any way to link it.288 someone must find or report this bug in the upstream bug
289 tracker.
289290
290We can just link upstream without a URL to say that this has been dealt291We can just link upstream without a URL to say that this has been dealt
291with, but we can't reference it.292with, but we can't reference it.
292293
293 >>> user_browser.getControl('I just want to register').selected = True294 >>> user_browser.getControl('I want to add this upstream').selected = True
294 >>> print_upstream_linking_form(user_browser)295 >>> print_upstream_linking_form(user_browser)
295 ( ) I have the URL for the upstream bug:296 ( ) I have the URL for the upstream bug:
296 [ ]297 [ ]
297 ( ) I have already emailed an upstream bug contact:298 ( ) I have already emailed an upstream bug contact:
298 [ ]299 [ ]
299 (*) I just want to register that it is upstream right now; I300 (*) I want to add this upstream project to the bug report, but
300 don't have any way to link it.301 someone must find or report this bug in the upstream bug
302 tracker.
301303
302 >>> user_browser.getControl('Add to Bug Report').click()304 >>> user_browser.getControl('Add to Bug Report').click()
303 >>> print user_browser.url305 >>> print user_browser.url
@@ -638,8 +640,9 @@
638 [ ]640 [ ]
639 (*) I have already emailed an upstream bug contact:641 (*) I have already emailed an upstream bug contact:
640 [dark-master-o-bugs@mylittlepony.com]642 [dark-master-o-bugs@mylittlepony.com]
641 ( ) I just want to register that it is upstream right now; I643 ( ) I want to add this upstream project to the bug report, but
642 don't have any way to link it.644 someone must find or report this bug in the upstream bug
645 tracker.
643646
644The bug tracker is automatically created without asking for647The bug tracker is automatically created without asking for
645confirmation.648confirmation.
646649
=== modified file 'lib/lp/bugs/stories/bug-also-affects/xx-bugtracker-information.txt'
--- lib/lp/bugs/stories/bug-also-affects/xx-bugtracker-information.txt 2009-06-12 16:36:02 +0000
+++ lib/lp/bugs/stories/bug-also-affects/xx-bugtracker-information.txt 2012-06-13 10:04:24 +0000
@@ -19,8 +19,8 @@
19 [ ]19 [ ]
20 ( ) I have already emailed an upstream bug contact:20 ( ) I have already emailed an upstream bug contact:
21 [ ]21 [ ]
22 ( ) I just want to register that it is upstream right now; I22 ( ) I want to add this upstream project to the bug report, but someone
23 don't have any way to link it.23 must find or report this bug in the upstream bug tracker.
2424
25If a product doesn't use Launchpad, and doesn't have a bug tracker25If a product doesn't use Launchpad, and doesn't have a bug tracker
26specified, it will simply say that it doesn't use Launchpad to track26specified, it will simply say that it doesn't use Launchpad to track
@@ -38,8 +38,8 @@
38 [ ]38 [ ]
39 ( ) I have already emailed an upstream bug contact:39 ( ) I have already emailed an upstream bug contact:
40 [ ]40 [ ]
41 ( ) I just want to register that it is upstream right now; I41 ( ) I want to add this upstream project to the bug report, but someone
42 don't have any way to link it.42 must find or report this bug in the upstream bug tracker.
4343
44For products using Launchpad, the linking upstream widgets won't even44For products using Launchpad, the linking upstream widgets won't even
45appear.45appear.
4646
=== modified file 'lib/lp/bugs/stories/bugtask-searches/xx-advanced-upstream-pending-bugwatch.txt'
--- lib/lp/bugs/stories/bugtask-searches/xx-advanced-upstream-pending-bugwatch.txt 2012-02-17 23:00:56 +0000
+++ lib/lp/bugs/stories/bugtask-searches/xx-advanced-upstream-pending-bugwatch.txt 2012-06-13 10:04:24 +0000
@@ -28,7 +28,7 @@
2828
29 >>> browser.getLink('Also affects project').click()29 >>> browser.getLink('Also affects project').click()
30 >>> browser.getControl(30 >>> browser.getControl(
31 ... 'I just want to register that it is upstream').selected = True31 ... 'I want to add this upstream project').selected = True
32 >>> browser.getControl('Add to Bug Report').click()32 >>> browser.getControl('Add to Bug Report').click()
33 >>> browser.title33 >>> browser.title
34 'Bug #... : Bugs : alsa-utils'34 'Bug #... : Bugs : alsa-utils'