RE: [qmailadmin] Working on new 1.0.7 version

[Michael Bowe]

> -----Original Message-----
> From: Ken Jones [mailto:[EMAIL PROTECTED]] 
> Sent: Friday, 10 January 2003 7:49 AM
> To: [EMAIL PROTECTED]
> Subject: [qmailadmin] Working on new 1.0.7 version
 
> I'm putting together a new 1.0.7 version. 
> So anyone with updated code, or bug reports.
> Please post the information here.

In the mailing list menus, you have menus to show 
subscribers / moderators / digest subscribers

This patch fixes two issues with this area :

 1. Make the menu layout more consistent with other parts
    of qmailadmin. ie put the email address on the left and
    the action button on the right 

 2. In the show subscribers menu, the column headings were
    the reverse of the data. ie the action button was in the
    address column, and the address was in the action column.

------------------------------------------------
Michael Bowe (B.App.Sc)
Managing Director - Pipeline Internet
96 Pakington Street, Geelong West. VIC. 3218
Tel (03) 5229 7643
Fax (03) 5229 0282
Mobile 0419 242 136
http://www.pipeline.com.au/
------------------------------------------------  

##################################################################
## In the mailing list menus, you have menus to show 
## subscribers / moderators / digest subscribers
##
## This patch fixes two issues with this area :
##   1. Make the menu layout more consistent with other parts
##      of qmailadmin. ie put the email address on the left and
##      the action button on the right 
##   2. In the show subscribers menu, the column headings were
##      the reverse of the data. ie the action button was in the
##      address column, and the address was in the action column.
##
##  Michael Bowe <[EMAIL PROTECTED]> 14/1/2003
##
##################################################################

--- mailinglist.c       Wed Aug  7 08:04:59 2002
+++ ../../qmailadmin-1.0.6/mailinglist.c        Tue Jan 14 12:46:13 2003
@@ -609,9 +609,9 @@
 
     /* Display subscriber/moderator/digest list, along with delete button */
     if(mod == 1) {
-        strcpy(TmpBuf, "228"); strcpy(TmpBuf1, "087"); strcpy(TmpBuf2, "220");
+        strcpy(TmpBuf, "228"); strcpy(TmpBuf1, "220"); strcpy(TmpBuf2, "087");
     } else if(mod == 2) {
-        strcpy(TmpBuf, "244"); strcpy(TmpBuf1, "245"); strcpy(TmpBuf2, "246");
+        strcpy(TmpBuf, "244"); strcpy(TmpBuf1, "246"); strcpy(TmpBuf2, "245");
     } else {
         strcpy(TmpBuf, "230"); strcpy(TmpBuf1, "222"); strcpy(TmpBuf2, "084");
     }
@@ -635,14 +635,14 @@
     }
     for(z = 0; z < index; ++z) {
       fprintf(actout," <TR align=middle>");
+      fprintf(actout,"  <TD align=left>%s</TD>\n", pa[z]);
       fprintf(actout,"  <TD align=middle><A 
href=%s/com/%s?modu=%s&newu=%s&dom=%s&user=%s&time=%d><IMG 
src=/images/qmailadmin/delete.png border=0></A></TD>\n",
         CGIPATH, TmpBuf, ActionUser, pa[z], Domain, Username, Mytime);
-      fprintf(actout,"  <TD align=left>%s</TD>\n", pa[z]);
       ++z;
       if(pa[z] != NULL) {
+      fprintf(actout,"  <TD align=left>%s</TD>\n", pa[z]);
         fprintf(actout,"  <TD align=middle><A 
href=%s/com/%s?modu=%s&newu=%s&dom=%s&user=%s&time=%d><IMG 
src=/images/qmailadmin/delete.png border=0></A></TD>\n",
           CGIPATH, TmpBuf, ActionUser, pa[z], Domain, Username, Mytime);
-        fprintf(actout,"  <TD align=left>%s</TD>", pa[z]);
       } else {
         fprintf(actout,"  <TD COLSPAN=2> </TD>");
       }