diff options
author | Ana Krivokapic <akrivoka@redhat.com> | 2013-05-15 17:33:21 +0200 |
---|---|---|
committer | Petr Vobornik <pvoborni@redhat.com> | 2013-05-17 12:31:14 +0200 |
commit | 8d07054e1b2ff8680c225e7cd847eb885e7246a4 (patch) | |
tree | c49188aabdeffe3ba493e972a349290ae9ba2d92 /install/ui/src/freeipa/sudo.js | |
parent | 5b58348cd316dd817672cb81358ed557c28e09d3 (diff) | |
download | freeipa-8d07054e1b2ff8680c225e7cd847eb885e7246a4.tar.gz freeipa-8d07054e1b2ff8680c225e7cd847eb885e7246a4.tar.xz freeipa-8d07054e1b2ff8680c225e7cd847eb885e7246a4.zip |
Do not display success message on failure in web UI
https://fedorahosted.org/freeipa/ticket/3591
Diffstat (limited to 'install/ui/src/freeipa/sudo.js')
-rw-r--r-- | install/ui/src/freeipa/sudo.js | 14 |
1 files changed, 9 insertions, 5 deletions
diff --git a/install/ui/src/freeipa/sudo.js b/install/ui/src/freeipa/sudo.js index 638b2380f..cb620e41b 100644 --- a/install/ui/src/freeipa/sudo.js +++ b/install/ui/src/freeipa/sudo.js @@ -881,18 +881,22 @@ IPA.sudo.options_section = function(spec) { on_success: function(data) { //last successful result of batch results contains valid data var result; - for(var i = data.result.results.length - 1; i > -1; i--) { - result = data.result.results[i].result; - if(result) break; + var succeeded = 0; + + for (var i = data.result.results.length - 1; i > -1; i--) { + var error = data.result.results[i].error; + if (!result) result = data.result.results[i].result; + if (!error) succeeded++; } - if(result) { + if (result) { that.table.load(result); } else { that.reload(); } - IPA.notify_success('@i18n:objects.sudorule.option_removed'); + var msg = text.get('@i18n:objects.sudorule.option_removed').replace('${count}', succeeded); + IPA.notify_success(msg); }, on_error: function(data) { that.reload(); |