[imp] IMAP ACLs [PATCH]

Jan Schneider jan at horde.org
Mon Feb 3 23:55:05 PST 2003


Zitat von Chris Hastie <lists at oak-wood.co.uk>:

> On Mon, 3 Feb 2003, Jan Schneider <jan at horde.org> wrote
> >Zitat von Chris Hastie <lists at oak-wood.co.uk>:
> >
> >> As presented, use of the rfc2086 driver is hard-coded into
> imp/acl.php,
> >> since there are no alternatives. It would be fairly easy to make this
> a
> >> configuration option in imp/config/servers.php in the same way as with
> >> Quotas. As is, the code will throw you back to the Options page with
> an
> >> error message if the server being used is not an IMAP server, or if
> the
> >> result of the CAPABILITY command to the server were successfully
> >> obtained and did not indicate ACL support.
> >
> >I think adding a configuration option to servers.php is the way to go.
> If
> >you send a patch for servers.php.dist please make sure to document the
> new
> >option well in the comment header. For now it only makes sense to add it
> to
> >the Cyrus example if I understand you correctly.
> 
> I could do with a hint on how to pick up any values defined in
> servers.php. EG if I add
> 
> $servers['cyrus'] = array(
>    ...
>    'acl' => array(
>              'driver' => 'rfc2086'
>           )
> )
> 
> how do I access this value? I did have a quick look at this and it
> seemed I needed to do some other work, maybe in imp/lib/Session.php, to
> get the value into $_SESSION. Which was my main reason for not doing it
> - I wanted to change as little as possible elsewhere.

lib/Session.php is the correct place. Take a look at how we handle the
'quota' option in IMP_Session::createSession() and lib/Quota.php.
 
Jan.

--
http://www.horde.org - The Horde Project
http://www.ammma.de - discover your knowledge
http://www.tip4all.de - Deine private Tippgemeinschaft


More information about the imp mailing list