diff options
author | Petr Vobornik <pvoborni@redhat.com> | 2013-03-21 15:47:50 +0100 |
---|---|---|
committer | Petr Vobornik <pvoborni@redhat.com> | 2013-04-05 10:44:44 +0200 |
commit | 3393c577f99661f6dc8a9a6a265bd420ceb74d0d (patch) | |
tree | 5c661d20712f1518656d5512f4d42b35e0053e1a /install | |
parent | 4dcc9476875e293aa0b639f004704bb9995b7182 (diff) | |
download | freeipa-3393c577f99661f6dc8a9a6a265bd420ceb74d0d.tar.gz freeipa-3393c577f99661f6dc8a9a6a265bd420ceb74d0d.tar.xz freeipa-3393c577f99661f6dc8a9a6a265bd420ceb74d0d.zip |
Run permission target switch action only for visible widgets
Permission details page was incorrectly evaluated as dirty (update button enabled) right after load when permission type={subtree,filter} and some attrs are set.
Can be reproduced by opening 'Modify Automount maps' permission.
The culprit is that attrs widget is populated and dirty-checked even targets where it doesn't belong.
Fixed by running target_mapping action only for visible targets.
https://fedorahosted.org/freeipa/ticket/3527
Diffstat (limited to 'install')
-rw-r--r-- | install/ui/src/freeipa/aci.js | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/install/ui/src/freeipa/aci.js b/install/ui/src/freeipa/aci.js index b6825d136..ceda40f8d 100644 --- a/install/ui/src/freeipa/aci.js +++ b/install/ui/src/freeipa/aci.js @@ -835,7 +835,7 @@ IPA.permission_target_policy = function (widget_name) { } } - if (target_info.action) target_info.action(); + if (visible && target_info.action) target_info.action(); }; |