Code review comment for lp:~jtv/launchpad/bug-443168

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

« Back to merge proposal