[Freeipa-devel] [PATCH] Add LDAPMultiQuery base class and make it the base of LDAPDelete

Adam Young ayoung at redhat.com
Fri Oct 1 14:20:55 UTC 2010


On 10/01/2010 10:02 AM, Rob Crittenden wrote:
> Pavel Zuna wrote:
>> In other words: make *-del commands accept 1 or more primary keys of
>> entries to be deleted. We can now delete more entries at a time with a
>> single command.
>>
>> Ticket #20
>
> The code looks ok but I have to nack the patch because I'm assuming 
> this is going to break existing unit tests (returns different values 
> now) and should have its own unit tests for doing multiple deletes.
>
> rob
>
> _______________________________________________
> Freeipa-devel mailing list
> Freeipa-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/freeipa-devel

Should I revert?




More information about the Freeipa-devel mailing list