[Freeipa-devel] [PATCH] 755 webui-ci: case-insensitive record check
Fraser Tweedale
ftweedal at redhat.com
Thu Sep 25 01:30:37 UTC 2014
On Wed, Sep 24, 2014 at 09:16:52AM -0500, Endi Sukma Dewata wrote:
> On 9/24/2014 8:26 AM, Petr Vobornik wrote:
> >On 24.9.2014 04:43, Endi Sukma Dewata wrote:
> >>On 9/22/2014 9:49 AM, Petr Vobornik wrote:
> >>>[PATCH] webui-ci: case-insensitive record check
> >>>
> >>>Indirect association are no longer lower cased, which caused a issue
> >>>in CI.
> >>
> >>Is the use of "|=" operator intentional? I don't see the "has" variable
> >>defined anywhere else in this method.
> >>
> >> has |= self.has_record(pkey.lower(), parent, table_name)
> >>
> >>If this has been tested to work, then ACK.
> >>
> >
> >it's defined on the previous line:
> >
> > has = self.has_record(pkey, parent, table_name)
> > has |= self.has_record(pkey.lower(), parent, table_name)
>
> Ah, I see. I thought the old line was being replaced.
> ACK.
IMO the following reads better::
has = self.has_record(pkey, parent, table_name) \
|| self.has_record(pkey.lower(), parent, table_name)
(Just a comment - no reason to block the patch.)
> --
> Endi S. Dewata
>
> _______________________________________________
> Freeipa-devel mailing list
> Freeipa-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/freeipa-devel
More information about the Freeipa-devel
mailing list