Merge lp:~jtv/launchpad/bug-443168 into lp:launchpad
Proposed by
Jeroen T. Vermeulen
Status: | Merged |
---|---|
Merged at revision: | not available |
Proposed branch: | lp:~jtv/launchpad/bug-443168 |
Merge into: | lp:launchpad |
Diff against target: |
14 lines 1 file modified
database/schema/security.cfg (+2/-0) |
To merge this branch: | bzr merge lp:~jtv/launchpad/bug-443168 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Henning Eggers (community) | code | Approve | |
Review via email: mp+12880@code.launchpad.net |
To post a comment you must log in.
= Bug 443168 =
This is a quick fix for a permissions check that we really don't want to
occur in the first place. See bug 443129 for more about that.
Sometimes, the approver needs to create new POFiles based on uploads by
non-admins. In that case it checks whether the current user would have
edit permissions on the POFile; if they would, the user becomes the
owner. If not, rosetta-admins becomes the owner. There's not much
point to this and we want to get rid of it. But for now we'll just give
it the database privileges it wants.
Jeroen