[Freeipa-devel] [PATCH] check for duplicate phone numbers

Simo Sorce ssorce at redhat.com
Tue May 20 15:36:33 UTC 2008


On Tue, 2008-05-20 at 11:32 -0400, Rob Crittenden wrote:
> Simo Sorce wrote:
> > On Mon, 2008-05-19 at 17:14 -0400, Rob Crittenden wrote:
> >> So what is the final resolution of this? Is this patch good enough for
> >> now?
> > 
> > patch itself is ok, but I'd extend the test to every attribute. Putting
> > in twice your name in 2 separate values is going to cause the same
> > error.
> > 
> > Simo.
> > 
> 
> Ok, I'll add cn in when I commit the patch.

Uhm no I was not thinking about doing a per attribute specific patch,
but something more generic that is applied to any multivalued attribute.
Maybe this is something for v2 ?

Simo.

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




More information about the Freeipa-devel mailing list