[Tickets #2834] patch for qmail-ldap

[bugs@bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/?id=2834
-----------------------------------------------------------------------
 Ticket             | 2834
 Updated By         | m.zdila at episoftware.com
 Summary            | patch for qmail-ldap
 Queue              | Vacation
 Version            | HEAD
 State              | Assigned
 Priority           | 2. Medium
 Type               | Enhancement
 Owners             | 
-----------------------------------------------------------------------


m.zdila at episoftware.com (2005-10-24 11:44) wrote:

hi

thanks for prompt reply :-)

>> * LDAP attribute for marking active vacation message can be array -
>> more entries - eg: 'reply', 'nolocal'. The value of the 'active'
>> attribute is then specified as: reply|nolocal
>
> What is the unset() call for?

it's because ldap_get_attributes adds to returned array the number of
attributes in the entry (see:
http://www.php.net/manual/en/function.ldap-get-attributes.php). We don't
want to have the value of "count" in the result (eg.: 2|reply|nolocal)

>
>> * the vacation message format is modified for qmail-ldap (for now
>> hardcoded in 2 places)
>> Example:
>> "%HEADER%\nSubject: " . _("On vacation message (Autoreply to:
>> %SUBJECT%)") . "\n\n" . _("I'm on vacation and will not be reading my
>> mail for a while.") . "\n" . _("Your mail will be dealt with when I
>> return.") . "\n";
>
> This breaks any other driver, or am I missing something? Where is 
> %HEADER% being replaced?

1. see http://www.qmail-ldap.org/wiki/LDAP_REPLYTEXT
2. this patch will break only other LDAP using mailsystems (hardcoding is
done only in Driver/ldap.php). Solution TODO:
a) specify mailsystem type in the configuration (exim, qmail-ldap)
b) use callback for encoding/decoding vacation message

if you would like me to do it, just say a or b :-)