[Freeipa-devel] [PATCH] 270 Fixed posix group checkbox.

Petr Vobornik pvoborni at redhat.com
Tue Sep 20 11:57:53 UTC 2011


On 09/16/2011 03:59 PM, Endi Sukma Dewata wrote:

> Attached is the updated patch. I fixed the test case and did some
> cleanup. This is actually the problem I mentioned in the meeting (so
> patch 271 is actually fine).
>
> It looks like currently the checkbox widget is only used with boolean
> attributes. And it will only work with boolean attributes anyway because
> right now the save() will always return a boolean value. But for now I'm
> trying to use the original loaded/default value unless it's 'TRUE' or
> 'FALSE' which will be converted into boolean.
>
> In the future we might want to use this with non-boolean attributes. So
> we need to be able to map a pair of non-boolean values into checked and
> unchecked state during load()/reset() and vice versa during save().
>
> --
> Endi S. Dewata

ACK

-- 
Petr Vobornik




More information about the Freeipa-devel mailing list