Re: 'diff' for Net::LDAP entries

[Graham B. <gb...@po...>]

It looks good. I'll assume you tested it :)

Just one point. We might want to allow the attribute names to be passed
and default to the union of the attributes in $new and $old if
not specified.

Graham.

On Sat, Aug 26, 2000 at 04:42:22PM +0100, John Berthels wrote:
> Hi.
> 
> Attached is a gzipped patch which attempts to implement a ->diff() method
> for Net::LDAP::Entry. There is a little POD in the patch too. Patch is
> against vanilla 0.20.
> 
> The intention is that you can do things like:
> 
> 	my $old_entry = ...read from LDAP server
> 	my $new_entry = ...read from LDIF
> 
> 	my $diff_entry = $old_entry->diff( $new_entry );
> 	$diff_entry->update( $ldap_server );
> 
> The only benefit of this over an add/delete sequence is that it should
> occur as one LDAP operation. Hence a power failure shouldn't leave you
> with the potential embarassement of an entry missing from the server.
> 
> No attempt is made to reconcile changes in the DN - it just fails.
> 
> regards,
> 
> jb
>