[sam] Re: sam + ldap patch

[Jan Schneider]

Zitat von Neil Sequeira <neil at ncsconsulting.com>:

> Jan Schneider wrote:
>
> > Zitat von Neil Sequeira <neil at ncsconsulting.com>:
> >
> >> Here's a start at an LDAP backend for sam.  It needs the >
> > Great! But please try to follow the coding standards.
> >
> > Jan.
> >
>
> An updated patch is attached.  It should follow the coding conventions
> and it fixes a bug displaying the blacklist.

Slightly changed committed, thanks. No one of use can test this driver
though, so it's up to you and other SAM users to keep this maintained.

Is there any reason that you commented out the capabilities array?

> BTW, this patch does a horrible job of handling errors (it silently
> ignores them!) because i'm not familiar with Horde (PEAR?) error
> reporting functions yet.  I hope to have it updated with better error
> reporting down the road (time permitting).

Great.

Jan.

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