Re: [webmin-devel] making config change to existing module, adding popup "select users" list

[Jamie C. <jca...@we...>]

On Fri, 2005-12-09 at 09:52, Mr Duck wrote:
> All,
> 
>    I am a perl programmer, and am making some modifications
> to the mailboxes/Read User Mail module, which I use to keep
> an eye on virus and SPAM activity for the company for which
> I work.
> 
>    Anyway, I've already implemented a few changes including
> configuration options for these changes, but there is one
> more than I want to add, and am not sure how to add the
> proper/correct config options to make it happen.
> 
>    Basically, I want to provide a listing of usernames that
> will _not_ be listed in the Read User Mail user list.  For example,
> there is no reason to show the uucp user or the games user
> or the users for the different daemons that this server runs.
> So, I want to include them in a list that will be checked
> when showing the users.

That's a good idea for a feature..

>    What I would like in the config is to have a text box and
> then a [ ... ] button that I can click which will pop up the
> user selection popup window, which will populate the textbox.
> 
>    I know how to add the textbox item to the config settings,
> and I know how to program the mailboxes_lib.pl file to ignore
> users in the list, but I don't know how to implement the
> system userlist popup button.

It would be good if the config.info format supported a multi-user
popup option type, but unfortunately it only supports a single-user
selection field at the moment.

The only way to do this is to implement your own custom config field
type, which is documented at http://www.webmin.com/modules-config.html
in the section on config type 15.

>    Is this clear?  Is there a way to do this from within a
> config option?
> 
>    Please ask questions if more information is needed.
> 
>    One other thing to ask:
> 
>    The things I'm adding basically allow the admin to change how
> many columns the usernames get listed in, if the admin wants the
> size of the mailbox to be listed below the username instead,
> and then the ignore list for users not to show.
> 
>    What is the likelihood that Jamie would be willing to
> (or want to) incorporate these changes into the module?

As long as your changes are well written, don't break any existing
functionality, and are optional, I would be glad to include them in the
module.

 - Jamie