On Sat, 23 Feb 2008 00:29:19 +0100 Guy Brand <gb@xxxxxxxxxxxxxxxxx> wrote: > One more before the feature freeze. > > Attached is a patch which allows the superuser or manager setting to > be a list instead of a single value. So for example, you can use: > > $conf['superuser'] = 'john,@root,doe'; > > and have users john and do, and group 'root' be superusers (admin > level access) of your wiki. This helps in situation where grouping > users in a dedicated group is a simple but unpractical solution. It > works the same way for managers (through $conf['manager']). > > The auth_aclcheck testcase is also updated and a new testcase for > auth_ismanager is provided. > > Comments and critics are welcome. A question on one part of your patch: hunk ./inc/auth.php 296 - $groups[$i] = '@'.auth_nameencode($groups[$i]); + $groups[$i] = auth_nameencode('@' . $groups[$i]); Why are you doing this? Andi -- http://www.splitbrain.org