Merge lp:~abentley/launchpad/flag-enables-privacy-checks into lp:launchpad
| Status: | Merged |
|---|---|
| Approved by: | Aaron Bentley on 2012-10-17 |
| Approved revision: | no longer in the source branch. |
| Merged at revision: | 16166 |
| Proposed branch: | lp:~abentley/launchpad/flag-enables-privacy-checks |
| Merge into: | lp:launchpad |
| Diff against target: | 0 lines |
| To merge this branch: | bzr merge lp:~abentley/launchpad/flag-enables-privacy-checks |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Richard Harding (community) | 2012-10-17 | Approve on 2012-10-17 | |
|
Review via email:
|
|||
Commit Message
Provide disclosure.
Description of the Change
= Summary =
Ensure that enabling project privacy on qastaging doesn't make it unusable for non-users.
== Proposed fix ==
Introduce disclosure.
== Pre-implementation notes ==
Brief discussion with deryck
== LOC Rationale ==
Part of Private Projects
== Implementation details ==
Updates ViewProduct to use userCanView for anonymous users.
Adds disclosure.
== Tests ==
bin/test -t test_anonymous_
== Demo and Q/A ==
- Create a proprietary product.
- Log in as an unprivileged user.
- Attempt to view the product. You should be unable to see it.
- Log out.
- Attempt to view the product. You should be unable to see it.
- Enable disclosure.
- Log in as an unprivileged user.
- Attempt to view the product. You should be able to see it.
- Log out.
- Attempt to view the product. You should be able to see it.
= Launchpad lint =
Checking for conflicts and issues in changed files.
Linting changed files:
lib/lp/
lib/lp/
lib/lp/
lib/lp/
