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.
--
Endi S. Dewata
_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
