From 6a59d68d248f4d35b44e4b5016582737c3004783 Mon Sep 17 00:00:00 2001
From: Marcin Haba <marcin.haba@bacula.pl>
Date: Sun, 6 Aug 2017 10:13:32 +0200
Subject: [PATCH] baculum: Remove assigning to api host when user is deleted

---
 gui/baculum/protected/Web/Portlets/Users.php | 6 +++++-
 gui/baculum/protected/Web/Portlets/Users.tpl | 6 +++---
 2 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/gui/baculum/protected/Web/Portlets/Users.php b/gui/baculum/protected/Web/Portlets/Users.php
index 265ae88096..fca312f78b 100644
--- a/gui/baculum/protected/Web/Portlets/Users.php
+++ b/gui/baculum/protected/Web/Portlets/Users.php
@@ -21,7 +21,7 @@
  */
 
 
-Prado::using('System.Web.UI.WebControls.TConditional');
+Prado::using('System.Web.UI.ActiveControls.TActiveRepeater');
 Prado::using('Application.Web.Portlets.Portlets');
 
 class Users extends Portlets {
@@ -96,6 +96,10 @@ class Users extends Portlets {
 			case 'rmuser': {
 					if ($user != $_SERVER['PHP_AUTH_USER']) {
 						$this->getModule('basic_webuser')->removeUser($user);
+						if (array_key_exists('users', $this->web_config) && array_key_exists($user, $this->web_config['users'])) {
+							unset($this->web_config['users'][$user]);
+						}
+						$this->getModule('web_config')->setConfig($this->web_config);
 						$this->setUsers();
 					}
 				break;
diff --git a/gui/baculum/protected/Web/Portlets/Users.tpl b/gui/baculum/protected/Web/Portlets/Users.tpl
index 393b08c213..07fed30b1f 100644
--- a/gui/baculum/protected/Web/Portlets/Users.tpl
+++ b/gui/baculum/protected/Web/Portlets/Users.tpl
@@ -1,6 +1,6 @@
 <a class="big" href="javascript:void(0)" id="add_user_btn"><img src="/themes/Baculum-v1/add.png" alt="Add"><%[ Add new user ]%></a>
 <div id="add_user" style="display: none">
-	<p><%[ Username: ]%><input id="newuser" type="text" /><%[ Password: ]%><input id="newpwd" type="password" /><%[ API host: ]%><com:TDropDownList ID="HostsList" OnInit="SourceTemplateControl.initHosts" />
+	<p><%[ Username: ]%><input id="newuser" type="text" /><%[ Password: ]%><input id="newpwd" type="password" />
 	<a href="javascript:void(0)" onclick="Users.addUser()">
 		<img src="<%=$this->getPage()->getTheme()->getBaseUrl()%>/icon_ok.png" alt="<%[ Save ]%>" title="<%[ Save ]%>"/>
 	</a>
@@ -8,7 +8,7 @@
 		<img src="<%=$this->getPage()->getTheme()->getBaseUrl()%>/icon_err.png" alt="<%[ Close ]%>" title="<%[ Close ]%>" />
 	</a></p>
 </div>
-<com:TRepeater ID="UsersList">
+<com:TActiveRepeater ID="UsersList">
 	<prop:HeaderTemplate>
 	<table id="users_list" class="window-section-detail-smallrow">
 		<tr>
@@ -61,7 +61,7 @@
 	<prop:FooterTemplate>
 		</table>
 	</prop:FooterTemplate>
-</com:TRepeater>
+</com:TActiveRepeater>
 <p><em><%[ Please note that for each user (excluding administrator) there should exist separate Bconsole config file in form: ]%> <strong><com:TLabel ID="BconsoleCustomPath" /></strong></em></p>
 <com:TCallback ID="UserAction" OnCallback="TemplateControl.userAction" ClientSide.OnComplete="Users.hide_loader();" />
 <script type="text/javascript">
-- 
2.39.5