diff options
| author | Petr Vobornik <pvoborni@redhat.com> | 2012-03-09 15:52:05 +0100 |
|---|---|---|
| committer | Petr Vobornik <pvoborni@redhat.com> | 2012-03-15 16:08:02 +0100 |
| commit | 3ca0f6aee5b04758f8473359cc14979d27d31530 (patch) | |
| tree | 3d7b2ef8a65219e66e3f7155b1a06da5608c3f8b /install/ui/group.js | |
| parent | 51601ac794ce589981c0cc3501d91518cea27f15 (diff) | |
| download | freeipa-3ca0f6aee5b04758f8473359cc14979d27d31530.tar.gz freeipa-3ca0f6aee5b04758f8473359cc14979d27d31530.tar.xz freeipa-3ca0f6aee5b04758f8473359cc14979d27d31530.zip | |
Fixed evaluating checkbox dirty status
Problem:
When value in checkbox is modified twice in a row (so it is at its original value) an 'undo' button is still visible even when it shouldn't be.
Cause:
IPA server sends boolean values as 'TRUE' or 'FALSE' (strings). Checkbox_widget converts them to JavaScript? boolean (true, false). Save method in checkbox_widget is returning array with a boolean. So test_dirty method always evaluates to dirty because 'FALSE' != false.
This patch is fixing the problem.
https://fedorahosted.org/freeipa/ticket/2494
Diffstat (limited to 'install/ui/group.js')
| -rw-r--r-- | install/ui/group.js | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/install/ui/group.js b/install/ui/group.js index 93b2fe0ff..5ad9ba1f1 100644 --- a/install/ui/group.js +++ b/install/ui/group.js @@ -140,7 +140,7 @@ IPA.group_nonposix_checkbox_widget = function (spec) { }; IPA.widget_factories['nonposix_checkbox'] = IPA.group_nonposix_checkbox_widget; -IPA.field_factories['nonposix_checkbox'] = IPA.checkbox_fields; +IPA.field_factories['nonposix_checkbox'] = IPA.checkbox_field; IPA.group_adder_dialog = function(spec) { |