From 3393c577f99661f6dc8a9a6a265bd420ceb74d0d Mon Sep 17 00:00:00 2001 From: Petr Vobornik Date: Thu, 21 Mar 2013 15:47:50 +0100 Subject: 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 --- install/ui/src/freeipa/aci.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/install/ui/src/freeipa/aci.js b/install/ui/src/freeipa/aci.js index b6825d13..ceda40f8 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(); }; -- cgit