Re: Net::LDAP modify examples typo

[Mark W. <mew...@un...>]

+1 require

mark

Graham Barr wrote:

> Well there is sonething that needs to be fixed. I am in two minds to
> either require the [] or allow them to be dropped if the value is a single
> value.
>
> Graham.
>
> On Fri, Jul 28, 2000 at 06:16:04PM +0200, Arvid Requate wrote:
> > Hi,
> >
> > Some time ago I asked about a problem with the $ldap->modify command
> > in combination with the "changes" syntax, resulting in "incorrect PDU structure"
> > messages. Now I got it working and found that there seems to be a typo
> > in the perldoc of Net:LDAP, where it reads in the examples section
> > of the modify command:
> >
> > -----------------------------------------------------------------------
> > $ldap->modify( $dn,
> >   changes => [
> >     add     => [ sn => 'Barr' ],              # Add sn=Barr
> >     delete  => [ faxNumber => []],            # Delete all fax numbers
> >     delete  => [ telephoneNumber => ['911']], # delete phone number 911
> >     replace => [ email => 'gb...@po...']  # change email address
> >   ]
> > );
> > -----------------------------------------------------------------------
> >
> > I guess there should be brakets around the values 'Barr' and the email-
> > address, as it is stated in the preceding text that arrayreferences
> > are expected here.
> >
> > Best regards
> >
> >  Arvid Requate
> >