[Freeipa-devel] [PATCH] 216 Remove memberPrincipal for deleted replicas

[Simo Sorce]

On Fri, 2012-03-02 at 12:34 +0100, Martin Kosek wrote:
> +                principals.remove(member_principal)
> +                mod = [(ldap.MOD_REPLACE, 'memberPrincipal',
> principals)] 

Any special reason why you use a search and then a replace instead of a
delete by value ?

A delete by value seem a lot less error prone to me, and should give you
the same resuls.

Simo.

-- 
Simo Sorce * Red Hat, Inc * New York