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

[Rob Crittenden]

Rob Crittenden wrote:
> Simo Sorce wrote:
>> On Tue, 2008-05-20 at 11:37 -0400, Rob Crittenden wrote:
>>> Simo Sorce wrote:
>>>> 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.
>>>>
>>> There are only like 5 multi-valued attributes possible in the UI. The 
>>> phone numbers and cn.
>>
>> what about custom attributes ?
>>
> 
> They are limited to single-value in the UI.

I went ahead and pushed what I had. And did a second push because I 
missed adding the new file to Makefile.am.

rob
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/x-pkcs7-signature
Size: 3245 bytes
Desc: S/MIME Cryptographic Signature
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20080520/9c66e3e2/attachment.bin>