Interface for setting default group is hidden when user doesn't have
necessary rights or if there is some error while loading the state.

https://fedorahosted.org/freeipa/ticket/4356
--
Petr Vobornik
From 317d407dbb76a0a6d54075eea435d2809314ce9b Mon Sep 17 00:00:00 2001
From: Petr Vobornik <pvobo...@redhat.com>
Date: Wed, 28 May 2014 16:50:22 +0200
Subject: [PATCH] webui: handle "unknown" result of
 automember-default-group-show

Interface for setting default group is hidden when user doesn't have
necessary rights or if there is some error while loading the state.

https://fedorahosted.org/freeipa/ticket/4356
---
 install/ui/src/freeipa/automember.js | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/install/ui/src/freeipa/automember.js b/install/ui/src/freeipa/automember.js
index 621ed6d1f40affc6b6fa691e0ab9e01239d42ad9..5eff3e69bdb014a3057bf9532e420cede416e8bf 100644
--- a/install/ui/src/freeipa/automember.js
+++ b/install/ui/src/freeipa/automember.js
@@ -617,6 +617,7 @@ IPA.automember.default_group_widget = function(spec) {
         }
 
         that.update(group);
+        that.set_visible(true);
     };
 
     that.update = function(group) {
@@ -646,7 +647,11 @@ IPA.automember.default_group_widget = function(spec) {
     that.refresh = function() {
 
         var command = that.create_command('show');
+        command.retry = false;
         command.on_success = that.load;
+        command.on_error = function() {
+            that.set_visible(false);
+        };
 
         command.execute();
     };
@@ -677,6 +682,7 @@ IPA.automember.default_group_widget = function(spec) {
     that.create = function(container) {
 
         var title = that.get_title();
+        that.container = container;
 
         that.header = $('<div/>', {
             'class': 'automember-header-label',
@@ -689,6 +695,7 @@ IPA.automember.default_group_widget = function(spec) {
         that.group_select.create(that.group_select_node);
         that.group_select_node.appendTo(container);
         that.group_select.update([]); // preload groups
+        that.set_visible(false);
     };
 
     that.get_title = function() {
-- 
1.9.0

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to