Launchpad itself

Merge lp:~cjohnston/launchpad/1244636 into lp:launchpad

  1. 1244636
  2. Merge into devel
Proposed by Chris Johnston
Status: Merged
Approved by: William Grant
Approved revision: no longer in the source branch.
Merged at revision: 16973
Proposed branch: lp:~cjohnston/launchpad/1244636
Merge into: lp:launchpad
Diff against target: 20 lines (+1/-2)
1 file modified
lib/lp/bugs/browser/bugtarget.py (+1/-2)
To merge this branch: bzr merge lp:~cjohnston/launchpad/1244636
Reviewer Review Type Date Requested Status
William Grant code Approve
Review via email: mp+213574@code.launchpad.net

Commit message

Removes the private security bug notice from a public security bug

Description of the change

Currently Launchpad shows a message about security bugs being private by default even when reporting a public security bug. This is confusing behavior.

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

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'lib/lp/bugs/browser/bugtarget.py'
--- lib/lp/bugs/browser/bugtarget.py 2014-02-24 06:50:46 +0000
+++ lib/lp/bugs/browser/bugtarget.py 2014-04-01 03:43:26 +0000
@@ -63,7 +63,6 @@
63 InformationType,63 InformationType,
64 PRIVATE_INFORMATION_TYPES,64 PRIVATE_INFORMATION_TYPES,
65 PUBLIC_INFORMATION_TYPES,65 PUBLIC_INFORMATION_TYPES,
66 SECURITY_INFORMATION_TYPES,
67 ServiceUsage,66 ServiceUsage,
68 )67 )
69from lp.app.errors import (68from lp.app.errors import (
@@ -640,7 +639,7 @@
640 # Give the user some feedback on the bug just opened.639 # Give the user some feedback on the bug just opened.
641 for notification in notifications:640 for notification in notifications:
642 self.request.response.addNotification(notification)641 self.request.response.addNotification(notification)
643 if bug.information_type in SECURITY_INFORMATION_TYPES:642 if bug.information_type == InformationType.PRIVATESECURITY:
644 self.request.response.addNotification(643 self.request.response.addNotification(
645 structured(644 structured(
646 'Security-related bugs are by default private '645 'Security-related bugs are by default private '