I posted this to <http://qmail.lightup.net/list.php?f=4>, but it got garbled and I 
don't know if anyone on the qmail dev team reads those messages.  I'm hoping that 
someone on the list can get these changes into the release version of qmailadmin.

Where should I send messages related to bug fixes or feature updates?

A few changes which should make it easier for novice users to add forwarding addresses:

(diff on add_forward)
30,31c30,31
<                                       <td align=right>
<                                       <input type=text size=16 name=newu 
maxlength=128> </td>
---
>                                       <td align=left>
>                                       <input type=text size=48 name=newu 
>maxlength=128> </td>
36c36
<                                       <td align=right> <input type=text size=16 
name=alias maxlength=128> </td>
---
>                                       <td align=left> <input type=text size=16 
>name=alias maxlength=128>@##D </td>

This does two things.  One, it make the forward email address box larger (since it 
will typically be longer than the Local Name).  It also adds the domain outside of the 
box for Local Name (making it extra clear that the Local Name box is an address for 
the current domain).

Also, I noticed that localization string 010 in the English file wasn't showing up 
because it had a tab instead of a space between the 010 and the message.  I've updated 
that text as follows:

10c10
< 010   Forward Email Address should be of the form <B>[EMAIL PROTECTED]</B>.  <BR>Local 
name should be of the form: <B>popaccount</B> <BR><b> IE, </b> <BR>Forward: 
[EMAIL PROTECTED] LocalName: sales <BR>That would send all mail sent to sales@##D to 
[EMAIL PROTECTED]
---
> 010 Forward Email Address should be of the form <B>[EMAIL PROTECTED]</B>.  <BR>Local 
>name should be of the form: <B>popaccount</B> <BR><b> IE, </b> <BR>Forward: 
>[EMAIL PROTECTED] LocalName: sales <BR>That would send all mail sent to sales@##D to 
>[EMAIL PROTECTED]

It just stays consistent with using [EMAIL PROTECTED]  I've noticed that qmailadmin 
didn't like long localization strings, so I increased tmpbuf in get_html_text of 
util.c to 512.  Then I found out that localization strings don't get the same macro 
expansion as send_template, so the "sales@##D" bit doesn't work.

I guess someone would have to make serious changes to get macro expansion to work on 
the macros in get_html_text, so you could change the message to be just "sales" 
instead of "sales@##D".

--
Tom Collins, CTO       InstallCo Computer Services
[EMAIL PROTECTED]      <http://www.installco.com/>

Reply via email to