################################################################## ## 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 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," "); + fprintf(actout," %s\n", pa[z]); fprintf(actout," \n", CGIPATH, TmpBuf, ActionUser, pa[z], Domain, Username, Mytime); - fprintf(actout," %s\n", pa[z]); ++z; if(pa[z] != NULL) { + fprintf(actout," %s\n", pa[z]); fprintf(actout," \n", CGIPATH, TmpBuf, ActionUser, pa[z], Domain, Username, Mytime); - fprintf(actout," %s", pa[z]); } else { fprintf(actout," "); }