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

[Endi Sukma Dewata]

On 9/20/2011 6:57 AM, Petr Vobornik 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().
>
> ACK

Pushed to master and ipa-2-1.

-- 
Endi S. Dewata