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
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.
Revision history for this message
Jeroen T. Vermeulen (jtv) wrote :

= 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

Revision history for this message
Henning Eggers (henninge) :
review: Approve (code)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'database/schema/security.cfg'
2--- database/schema/security.cfg 2009-10-02 09:27:02 +0000
3+++ database/schema/security.cfg 2009-10-05 15:40:32 +0000
4@@ -463,8 +463,10 @@
5 public.productseries = SELECT
6 public.sourcepackagename = SELECT
7 public.teamparticipation = SELECT
8+public.translationgroup = SELECT
9 public.translationimportqueueentry = SELECT, DELETE, UPDATE
10 public.translationrelicensingagreement = SELECT
11+public.translator = SELECT
12
13 [poexport]
14 # Rosetta export script