Launchpad itself

Merge lp:~mwhudson/launchpad/i-dont-believe-for-a-minute-that-steveIsFixingThis into lp:launchpad

  1. i-dont-believe-for-a-minute-that-steveIsFixingThis
  2. Merge into devel
Proposed by Michael Hudson-Doyle
Status: Merged
Approved by: William Grant
Approved revision: no longer in the source branch.
Merged at revision: 14137
Proposed branch: lp:~mwhudson/launchpad/i-dont-believe-for-a-minute-that-steveIsFixingThis
Merge into: lp:launchpad
Diff against target: 26 lines (+0/-8)
1 file modified
lib/canonical/launchpad/webapp/authorization.py (+0/-8)
To merge this branch: bzr merge lp:~mwhudson/launchpad/i-dont-believe-for-a-minute-that-steveIsFixingThis
Reviewer Review Type Date Requested Status
William Grant code Approve
Review via email: mp+79047@code.launchpad.net

Commit message

[r=wgrant][no-qa] Remove the 'steveIsFixingThis' cruft that was old when I started with Canonical, 4.5 years ago

Description of the change

Steve is not fixing this.

To post a comment you must log in.
Revision history for this message
William Grant (wgrant) wrote :

Tempting to defer to SteveA, but perhaps not.

review: Approve (code)
Revision history for this message
Francis J. Lacoste (flacoste) wrote :

The truth of this is that this is already fixed... on the mythical DVD that also has the implementation for crowd control ;-)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'lib/canonical/launchpad/webapp/authorization.py'
2--- lib/canonical/launchpad/webapp/authorization.py 2011-04-27 16:01:06 +0000
3+++ lib/canonical/launchpad/webapp/authorization.py 2011-10-12 01:11:28 +0000
4@@ -41,9 +41,6 @@
5 from lp.app.interfaces.security import IAuthorization
6
7
8-steveIsFixingThis = False
9-
10-
11 LAUNCHPAD_SECURITY_POLICY_CACHE_KEY = 'launchpad.security_policy_cache'
12
13
14@@ -190,12 +187,7 @@
15 # This check shouldn't be needed, strictly speaking.
16 # However, it is here as a "belt and braces".
17
18- # XXX Steve Alexander 2005-01-12:
19- # This warning should apply to the policy in zope3 also.
20 if permission == 'zope.Public':
21- if steveIsFixingThis:
22- warnings.warn(
23- 'zope.Public being used raw on object %r' % object)
24 return True
25 if permission is CheckerPublic:
26 return True