[dev] [Patch] Check for requied attributes

[Tarjei Huse]

On Mon, 2004-08-02 at 06:38, Chuck Hagenbuch wrote: 
> Quoting Tarjei Huse <tarjei+a_lists.phpgw at nu.no>:
> 
> > Hi, patch update. I've added some comments.
> >>  >
> >> > > The attached patch adds functionality to turbas ldapdriver to check if
> >> > > an entry beeing added contains all the attributes needed.
> >> > >
> >> > > This is especially practical with regard to adding entries from other
> >> > > applications (like imp) as they cannot tell the user that f.x. the
> >> > > Lastname field is required. As the patch is no, it adds just a "N/A"
> >> > > string instead.
> 
> I'd definitely commit this if the field could contain an empty string 
> instead of
> N/A. Can it? Or does that go counter to how LDAP handles required fields?
I'm sorry, but it will not accept an empty string, a space is ok though.
I have changed the patch so it's possible to set the string in a
configfile. 

Also, I've done my homework and modified the code so that it now checks
to see if the attribute in question is a string or not. This makes the
patch a bit larger (+60 LOC). I'd like input from other ldapusers on
what ldapsyntaxes should be ok.

Also, the patch now uses the existing ldapconnection instead of making
it's own.

Please comment.

Tarjei



> -chuck
> 
> --
> "Regard my poor demoralized mule!" - Juan Valdez
-- 
Tarjei Huse
Rådgiver
Bergfald & Co AS
Telefon: 23 00 05 90
Mobiltelefon: 920 63 413
www.bergfald.no
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ldap.checkrequired.config.v2.patch
Type: text/x-patch
Size: 1132 bytes
Desc: not available
Url : http://lists.horde.org/archives/dev/attachments/20040803/a265508f/ldap.checkrequired.config.v2-0001.bin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ldap.php.checkrequired.v2.patch
Type: text/x-patch
Size: 4831 bytes
Desc: not available
Url : http://lists.horde.org/archives/dev/attachments/20040803/a265508f/ldap.php.checkrequired.v2-0001.bin