Merge lp:~salgado/launchpad/bug-538207 into lp:launchpad
Status: | Merged |
---|---|
Approved by: | Aaron Bentley |
Approved revision: | no longer in the source branch. |
Merged at revision: | not available |
Proposed branch: | lp:~salgado/launchpad/bug-538207 |
Merge into: | lp:launchpad |
Diff against target: |
135 lines (+23/-20) 6 files modified
lib/canonical/launchpad/doc/launchpadview.txt (+0/-10) lib/canonical/launchpad/templates/oops.pt (+1/-1) lib/canonical/launchpad/webapp/publisher.py (+0/-4) lib/canonical/launchpad/webapp/servers.py (+5/-1) lib/canonical/launchpad/webapp/tests/test_servers.py (+15/-2) lib/lp/app/templates/base-layout-macros.pt (+2/-2) |
To merge this branch: | bzr merge lp:~salgado/launchpad/bug-538207 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Aaron Bentley (community) | Approve | ||
Review via email: mp+21387@code.launchpad.net |
Description of the change
= Summary =
Move isRedirectInhibited from LaunchpadView to LaunchpadBrowse
so that views that don't inherit from LaunchpadView don't OOPS on edge.
= Launchpad lint =
Checking for conflicts. and issues in doctests and templates.
Running jslint, xmllint, pyflakes, and pylint.
Using normal rules.
Linting changed files:
lib/canonical
lib/lp/
lib/canonical
lib/canonical
lib/canonical
== Pylint notices ==
lib/canonical/
47: [F0401] Unable to import 'lazr.restful.
49: [F0401] Unable to import 'lazr.restful.
73: [F0401] Unable to import 'lazr.uri' (No module named uri)
lib/canonical/
18: [F0401] Unable to import 'lazr.restful.
20: [F0401] Unable to import 'lazr.restful.
21: [F0401] Unable to import 'lazr.restful.
The code looks good, but your comments about E1002 are irrelevant to the issue. TestWebServiceR equestTraversal , TestVhostWebser viceFactory, FeedsPublication, WebServicePubli cation, PrivateXMLRPCPu blication and ProtocolErrorPu blication are all derived from classes defined outside their file, and so their status as new-style classes is determined by the classes they derive from. __metaclass__ = type has absolutely no impact on these classes.