[Freeipa-devel] [PATCH] display name as separate attributes in ipa-finduser

Rob Crittenden rcritten at redhat.com
Thu Aug 21 15:21:02 UTC 2008


Martin Nagy wrote:
> On Thu, 21 Aug 2008 10:37:24 -0400, Rob Crittenden
> <rcritten at redhat.com> wrote:
>> +        try:
>> +            l = attr.index('givenname')
>> +            try:
>> +                attr.remove('sn')
>> +                attr.insert(l+1, 'sn')
>> +            except ValueError:
>> +                pass
>> +        except ValueError:
>> +            pass
> 
> Please remove the nested try, the code will be equivalent.
>

Updated patch attached.

rob
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: freeipa-78-name.patch2
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20080821/cc13834b/attachment.ksh>
-------------- 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/20080821/cc13834b/attachment.bin>


More information about the Freeipa-devel mailing list