Merge lp:~wallyworld/launchpad/oops-bug-filing-location into lp:launchpad

Proposed by Ian Booth
Status: Merged
Approved by: Ian Booth
Approved revision: no longer in the source branch.
Merged at revision: 12095
Proposed branch: lp:~wallyworld/launchpad/oops-bug-filing-location
Merge into: lp:launchpad
Diff against target: 57 lines (+6/-6)
3 files modified
lib/launchpad_loggerhead/app.py (+4/-4)
lib/lp/code/browser/sourcepackagerecipe.py (+1/-1)
lib/lp/registry/model/person.py (+1/-1)
To merge this branch: bzr merge lp:~wallyworld/launchpad/oops-bug-filing-location
Reviewer Review Type Date Requested Status
Steve Kowalik (community) code* Approve
Review via email: mp+44005@code.launchpad.net

Commit message

[r=stevenk][ui=none][bug=691343] Change some project based bugs urls to just launchpad ones

Description of the change

= Summary =

Trivial fix to remove references such as http://bug.launchpad.net/launchpad-code and replace with http://bug.launchpad.net/launchpad

= Implementation =

Change some urls.

= Tests =

None

= Lint =

(all existing issues)

Checking for conflicts and issues in changed files.

Linting changed files:
  lib/launchpad_loggerhead/app.py
  lib/lp/code/browser/sourcepackagerecipe.py
  lib/lp/registry/model/person.py

./lib/launchpad_loggerhead/app.py
      99: E231 missing whitespace after ':'
     237: E301 expected 1 blank line, found 0
     336: E301 expected 1 blank line, found 0
     227: Line exceeds 78 characters.
     250: Line exceeds 78 characters.
     288: Line exceeds 78 characters.
     332: Line exceeds 78 characters.
     333: Line exceeds 78 characters.
     354: Line exceeds 78 characters.

To post a comment you must log in.
Revision history for this message
Steve Kowalik (stevenk) wrote :

Do it, except the URLs currently redirect anyway.

review: Approve (code*)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'lib/launchpad_loggerhead/app.py'
2--- lib/launchpad_loggerhead/app.py 2010-08-03 00:22:37 +0000
3+++ lib/launchpad_loggerhead/app.py 2010-12-17 04:53:55 +0000
4@@ -257,8 +257,8 @@
5 <h1>Oops!</h1>
6 <p>Something broke while generating the page.
7 Please try again in a few minutes, and if the problem persists file a bug at
8-<a href="https://bugs.launchpad.net/launchpad-code"
9->https://bugs.launchpad.net/launchpad-code</a>
10+<a href="https://bugs.launchpad.net/launchpad"
11+>https://bugs.launchpad.net/launchpad</a>
12 and quote OOPS-ID <strong>%(oopsid)s</strong>
13 </p></body></html>'''
14
15@@ -271,7 +271,7 @@
16 """Wraps start_response (from a WSGI request) to keep track of whether
17 start_response was called (and whether the callable it returns has been
18 called).
19-
20+
21 Used by oops_middleware.
22 """
23
24@@ -324,7 +324,7 @@
25 [], URL=construct_url(environ))
26 error_utility.raising(sys.exc_info(), request)
27 return request.oopsid
28-
29+
30
31 def oops_middleware(app):
32 """Middleware to log an OOPS if the request fails.
33
34=== modified file 'lib/lp/code/browser/sourcepackagerecipe.py'
35--- lib/lp/code/browser/sourcepackagerecipe.py 2010-12-13 08:23:15 +0000
36+++ lib/lp/code/browser/sourcepackagerecipe.py 2010-12-17 04:53:55 +0000
37@@ -95,7 +95,7 @@
38 'We\'re still working on source package recipes. '
39 'We would love for you to try them out, and if you have '
40 'any issues, please '
41- '<a href="http://bugs.launchpad.net/launchpad-code">'
42+ '<a href="http://bugs.launchpad.net/launchpad">'
43 'file a bug</a>. We\'ll be happy to fix any problems you encounter.')
44
45
46
47=== modified file 'lib/lp/registry/model/person.py'
48--- lib/lp/registry/model/person.py 2010-12-16 19:27:54 +0000
49+++ lib/lp/registry/model/person.py 2010-12-17 04:53:55 +0000
50@@ -307,7 +307,7 @@
51 Note that it performs poorly at least some of the time, and if
52 EmailAddress and Person are already being queried, its probably better to
53 query Account directly. See bug
54- https://bugs.launchpad.net/launchpad-registry/+bug/615237 for some
55+ https://bugs.launchpad.net/launchpad/+bug/615237 for some
56 corroborating information.
57 """
58