before i start, id like to state that am not critizising your mod, am only suggesting improvements you could make to improve your mod.
i would suggest that you close any connections to the database. for example:
If blnAdmin = False Then 'Clean up Call closeDatabase()
Response.Redirect("default.asp") End If |
the function in red is built into the forum, it is used to close any connections created by the forum (eg adoCon and rsCommon).
you may also want to use
rsCommon instead of
rsEmailNotify. as the rsCommon is already created by the forum.
and then add the function to close the database to the bottom of the page
i also noticed that you show the Author ID
if you change your SQL statement to this you'll get the members username:
strSQL = "SELECT tblEmailNotify.*, " & strDBTable & "Author.Username FROM tblEmailNotify, " & strDBTable & "Author WHERE Topic_ID =" & lngTopicID & " and " & strDBTable & "Author.Author_ID = tblEmailNotify
.Author_ID " |
hope this helps
Edited by Scotty32 - 13 January 2007 at 9:52am