Re: [Postfixadmin-devel] SF.net SVN: postfixadmin:[512] trunk/delete.php

[David G. <da...@co...>]

Christian Boltz wrote :
> Hello,
> 
> Am Montag, 12. Januar 2009 schrieb Gin...@us...:
> > Revision: 512
> 
> > delete.php: this probably allows you to delete domains
> 
> > Modified: trunk/delete.php
> 
> > +elseif ($fTable == "alias_domain")
> > +{
> > +   authentication_require_role('global-admin');
> 
> Is there a special reason why you limited deletion of alias domains to 
> superadmins?
> 
> Every domain admin is allowed to create alias domains (given he has at 
> least 2 domains ;-) - so he should also be allowed to delete domain 
> aliases again. (A check if both involved domains are owned by the admin 
> is of course necessary.)
> 

I wasn't sure how to tell if a non-global admin owned a domain.. so I
just went with the easy option.

David.

-- 
David Goodwin 

[ david at codepoets dot co dot uk ]
[ http://www.codepoets.co.uk       ]