Re: Patch: Handle distribution lists on alias expansion



Toralf Lund wrote:
>the test ought
> to be written to cover all cases. You might use
> LibBalsaAddressBook::dist_list_mode, of course, but that requires the
> address book to be found (is there a way to lookup the address book based
> on an address), and it's not really what we want for LDIF/LDAP. In fact,
> the "dist list" parameter is fairly meaningless for those as person (or
> organization) entries and distribution lists are completely distinct for
> those types. What we really want is probably to keep more of the structure
> from the address book data.

1. If you only can, ignore dist_list_mode. This field is a semi-quick
hack. 

2. You can treat LDIF format as the fundamental one. It is most
flexible - and also basically identical to LDAP responses. I am sure
one can obtain similar effects with VCARD format, though.

3. I would consider adding another class LibBalsaDistList. The address
book should have entries which can be ordinary addresses or dist
lists.

Does it sound sensible?

/pawel
-- 
Pawel Salek, pavels@kjemi.uio.no, +47 (228) 55430
Theoretical Chemistry, Dept of Chemistry, Univ of Oslo
http://www.theochem.kth.se/~pawsa/




[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]