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

Martin Nagy mnagy at redhat.com
Thu Aug 21 15:24:56 UTC 2008


On Thu, 21 Aug 2008 11:21:02 -0400, Rob Crittenden
<rcritten at redhat.com> wrote:

> 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
ack




More information about the Freeipa-devel mailing list