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


Petr Vobornik

Freeipa-devel mailing list

Reply via email to