Hi, I don't know if anyone might be able to use this but after I upgraded from 6 to 7 my users kept getting hard halts when trying to send emails. As it turned out, there is a new(?) field called M_ALLOWEMAIL in the FORUM_MEMBERS table which is numeric but contains blanks unless the user updates their profile. The code is trying to cast the value as an integer which causes the halt. I changed the pop_mail.asp file at line 68 as follows and the problem went away:
Change this intAllowEmail = cInt(rs("M_ALLOWEMAIL"))
to this if isNull(rs("M_ALLOWEMAIL")) then intAllowEmail = 0 Else intAllowEmail = cInt(rs("M_ALLOWEMAIL")) End If
That way, the default is to NOT send a user emails but don't blow up during the attempt. Hope that helps someone. Coda
No errors at all during the upgrade. I got a success message. It didn't set the field value to zero, it appears to have left the field empty or null, hence my problem. I am using an Access database. I hope I didn't open a can of worms or something. I thought it was a pretty simple little thing to fix. Coda
quote:Originally posted by Davio
The upgrade should have set the default value for the new field to 0. Did you get any errors during the upgrade? What database are you using?
Unless the 3.4.07 version was updated with the correct fix, I don't see why the upgrade doesn't set the default value of the new field to 0. I'll setup a 3.4.06 version and test the upgrade for the problem.
Do you guys think it might be something I did wrong? (I actually hope that it was me.) I have not made any real changes to the forum code other than the one I posted. I have renamed the database and changed the email port according to the instructions from my web host. Other than that, vanilla. Please let me know what you find and I do apologize for triggering a test. Have a good day. Coda
Topic Locked
Posted - 28 October 2009 : 22:01:30
