diff options
author | Petr Vobornik <pvoborni@redhat.com> | 2014-07-07 15:51:29 +0200 |
---|---|---|
committer | Petr Vobornik <pvoborni@redhat.com> | 2014-07-21 12:30:23 +0200 |
commit | 75a96fb4c2f58d9ad54a374136afa656ac9a737e (patch) | |
tree | dfca76e2bc2628abb0d178b363566d99ef0292f9 | |
parent | e60cfa28626d7e224e2b4aebbe8af8e3fdf1d1c0 (diff) | |
download | freeipa-75a96fb4c2f58d9ad54a374136afa656ac9a737e.tar.gz freeipa-75a96fb4c2f58d9ad54a374136afa656ac9a737e.tar.xz freeipa-75a96fb4c2f58d9ad54a374136afa656ac9a737e.zip |
webui: show managed fields as readonly and not disabled
Visible read-only fields are no longer displayed as disabled in
permission details facet.
https://fedorahosted.org/freeipa/ticket/4254
Reviewed-By: Endi Sukma Dewata <edewata@redhat.com>
-rw-r--r-- | install/ui/src/freeipa/aci.js | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/install/ui/src/freeipa/aci.js b/install/ui/src/freeipa/aci.js index ec3d82cf9..f7628097a 100644 --- a/install/ui/src/freeipa/aci.js +++ b/install/ui/src/freeipa/aci.js @@ -972,9 +972,7 @@ aci.permission_target_policy = function (spec) { var widget = that.permission_target.widgets.get_widget(target_info.name); var field = that.container.fields.get_field(target_info.name); that.permission_target.set_row_visible(target_info.name, visible); - var managed_f = aci.managed_fields.indexOf(target_info.name) > -1; - var enabled = !(managed_f && that.managed) && visible && !that.system; - field.set_enabled(enabled); + field.set_enabled(visible); field.set_required(visible && target_info.required); widget.set_visible(visible); }; @@ -1054,7 +1052,7 @@ aci.permission_managed_policy = function (spec) { var field = fields[i]; if (field.read_only) continue; var managed_f = aci.managed_fields.indexOf(field.name) > -1; - field.set_enabled(!system && !(managed_f && managed)); + field.set_writable(!system && !(managed_f && managed) && field.writable); } }; |