[memobug]

Thanks TTG
Here's my update
1. I found a minor bug in the HTML printout that causes Thread block 4 to be labeled the same as thread block 3. You might want to edit it
replace

			$HTML_threadspreview4 .="	<td colspan=\"7\"><b><i>$title3</i></b></td>\n";

with the following:

			$HTML_threadspreview4 .="	<td colspan=\"7\"><b><i>$title4</i></b></td>\n";

It's somewhere around line 850 - I have added so much I can't be specific, but about midway down in the block that starts "if ($numtopics4)"

2. I have added the user-selected articles header back in. The text version is working well but for the HTML version to be usable I would need to have a way of ensuring that truncating the message doesn't split HTML tags or tag pairs, or it could mess up the rest of the message.

update - I got around this by using a tag like -MORE- right in the post. Then I made the commbull trim the article right at the tag, and there is no danger of cutting in a bad place!

3. I got rid of 222 lines of code in commbull.php by removing the redundant iterations of the same code for thread blocks 2, 3, 4 as were used in block 1.

4. I found a problem when I designated some 24 forum members in a special public group called proofreaders. When I tried to send the newsletter to them for proofing commbull complained that there were no users. I had to fix two queries to enable it to recognize the auxilliary user groups.

All in all, with the exception of not having any calendar or birthday events working, it came off pretty well. I sent out 2800 newsletters this afternoon.

Matt