[Tickets #11690] Re: Admin - User Management - adding and deleting do not work

[bugs at horde.org]

BITTE NICHT AUF DIESE NACHRICHT ANTWORTEN. NACHRICHTEN AN DIESE  
E-MAIL-ADRESSE WERDEN NICHT GELESEN.

Ticket-URL: http://bugs.horde.org/ticket/11690
------------------------------------------------------------------------------
  Ticket           | 11690
  Aktualisiert Von | burkhard at knippenberg.eu
  Zusammenfassung  | Admin - User Management - adding and deleting do not work
  Warteschlange    | Horde Base
  Version          | 5.0.1
  Typ              | Bug
  Status           | Unconfirmed
  Priorität        | 1. Low
  Milestone        |
  Patch            | 1
  Zuständige       |
------------------------------------------------------------------------------


burkhard at knippenberg.eu (2012-11-13 05:25) hat geschrieben:

Hi,

I can confirm the bug. In addition: Trying to add a user leads to a  
removal of the user from the database, if the username already exists.

Configuration:

$conf['auth']['admins'] = array('xxx');
$conf['auth']['checkip'] = true;
$conf['auth']['checkbrowser'] = true;
$conf['auth']['resetpassword'] = true;
$conf['auth']['alternate_login'] = false;
$conf['auth']['redirect_on_logout'] = false;
$conf['auth']['list_users'] = 'list';
$conf['auth']['params']['driverconfig'] = 'horde';
$conf['auth']['params']['encryption'] = 'ssha';
$conf['auth']['params']['show_encryption'] = false;
$conf['auth']['driver'] = 'sql';
$conf['auth']['params']['count_bad_logins'] = false;
$conf['auth']['params']['login_block'] = false;
$conf['auth']['params']['login_block_count'] = 5;
$conf['auth']['params']['login_block_time'] = 5;
$conf['signup']['params']['driverconfig'] = 'horde';
$conf['signup']['driver'] = 'Sql';
$conf['signup']['approve'] = true;
$conf['signup']['allow'] = true;

IMHO this bug should have a high priority!

> Hi,
>
> adding and removing users does not work on the Admin Page.
> seems that some input fields are missing in the add.inc and  
> remove.inc template files.
>
> the following changes solve the problem at the first glance, but I  
> don't have any ideas about side-effects and so on...
>
> below is a diff against the master branch.hope it helps.
>
> ag
>
> ------------------------------------------------------------
>
> diff --git a/horde/templates/admin/user/add.inc  
> b/horde/templates/admin/user/add.inc
> index a0d42ba..c19ff1e 100644
> --- a/horde/templates/admin/user/add.inc
> +++ b/horde/templates/admin/user/add.inc
> @@ -1,3 +1,4 @@
>  <?php
> +$vars->set('form', 'add');
>  $renderer = new Horde_Form_Renderer();
>  $addForm->renderActive($renderer, $vars, Horde::selfUrl(), 'post');
> diff --git a/horde/templates/admin/user/remove.inc  
> b/horde/templates/admin/user/remove.inc
> index 35540f5..7745e4e 100644
> --- a/horde/templates/admin/user/remove.inc
> +++ b/horde/templates/admin/user/remove.inc
> @@ -1,5 +1,6 @@
>  <form name="removeuser" action="<?php echo Horde::selfUrl() ?>"  
> method="post">
>  <input type="hidden" name="form" value="remove" />
> +<input type="hidden" name="user_name" value="<?php echo  
> htmlspecialchars($f_user_name) ?>" />
>  <?php Horde_Util::pformInput() ?>
>
>  <h1 class="header">
>
> ------------------------------------------------------------