| Author |
 Topic  |
|
Giumer
Junior Member
Italy
163 Posts |
 Posted - 02 April 2009 : 11:52:59
|
hi i am for send e_mail error this :
Microsoft VBScript runtime error '800a005e'
Invalid use of Null: 'cInt'
/forum/pop_mail.asp, line 68
plz test
<moved from="Help: General / Current Version (v3.4.xx)" /> |
=======================
http://www.Giumer.it/forum
======================= |
Edited by - AnonJr on 02 April 2009 13:34:04 |
|
|
AnonJr
Moderator
United States
5768 Posts |
Posted - 02 April 2009 : 12:36:43
|
Once more, with clarity please...
Is this an upgraded forum or a clean install? What database type are you using? A little more information than "plz test" whould help. |
 |
|
|
HuwR
Forum Admin
United Kingdom
20584 Posts |
Posted - 02 April 2009 : 13:00:51
|
| pop_mail is not working here, it gives the error giumer posted |
|
|
|
AnonJr
Moderator
United States
5768 Posts |
Posted - 02 April 2009 : 13:32:45
|
| Interesting... it worked during testing with an Access back-end and using a clean database. I wonder if its only an issue in upgrades due to a missing default? |
|
|
|
AnonJr
Moderator
United States
5768 Posts |
Posted - 02 April 2009 : 13:46:13
|
I suspect there were similar issues or some other problem as I see line 70 is not evaluating it as a numeric value:
It is probably safe to change line 68 to:
No matter what, it would still help to know if this was an upgrade or a clean install, and what db is being used. |
Edited by - AnonJr on 02 April 2009 13:48:02 |
|
|
|
modifichicci
Average Member
Italy
787 Posts |
|
|
AnonJr
Moderator
United States
5768 Posts |
Posted - 02 April 2009 : 18:41:29
|
quote:
Originally posted by modifichicci
or
If intMPosts < intMaxPostsToEMail and intAllowEmail <> 1 Then
Won't work because its failing on the cInt() conversion - most likely due to the field not existing in previous versions... hence why I wanted to know if this was an upgrade or a clean install.
Since intAllowEmail is only used in the If ... Then statement I mentioned, and since its rejecting any value other than one that would be explicitly set, I would think it would be ok to do without the cInt().
Another option would be to write a small dbs script that set the value to 1 for all current members. If I had thought about it at the time I would have included that as a part of the upgrade process... hindsight is always 20/20. |
|
|
|
HuwR
Forum Admin
United Kingdom
20584 Posts |
Posted - 03 April 2009 : 11:35:28
|
well, obviously this was not a clean install  and the error occurs here |
|
|
|
AnonJr
Moderator
United States
5768 Posts |
Posted - 03 April 2009 : 11:53:03
|
quote:
Originally posted by HuwR
well, obviously this was not a clean install and the error occurs here
That much I figured.  If his was a clean install then it means there may be something wrong with the setup. If it wasn't than its probably just that the existing members don't have that particular field set.
After looking back at some of the dev discussions, I remember thinking about going and setting all existing members to "1" - but I obviously never got around to it...  |
|
|
|
modifichicci
Average Member
Italy
787 Posts |
|
|
HuwR
Forum Admin
United Kingdom
20584 Posts |
Posted - 06 April 2009 : 12:15:12
|
so what exactly am I supposed to change to fix this ?
fixed here.
I would suggest we provide a script to use in the 'Alternative MOD Setup' to correct the problem for anyone else upgrading. |
|
|
|
AnonJr
Moderator
United States
5768 Posts |
Posted - 06 April 2009 : 12:48:40
|
I'll try to whip something together after my 2nd "oh good, you're here - this will only take a minute" meeting.  |
|
|
|
HuwR
Forum Admin
United Kingdom
20584 Posts |
Posted - 06 April 2009 : 13:14:13
|
 I'm going to have a lot of those tomorrow as I have been away since thursday |
|
|
|
Carefree
Advanced Member
Philippines
4207 Posts |
Posted - 07 April 2009 : 02:56:52
|
I have no more of those "Oh good..." meetings. After 20 years with Uncle Sam, I had more than my share. Then another 20 years on the civilian side and I'm through with meetings. If someone wants my presence at a "meeting", they'll have to wait until I'm in a box and don't have any say about attending.
When I was in the Philippines, we had a captain who LOVED staff meetings. We'd have a meeting between the senior NCOs and him at 0700, another at noon, and still another at 1700. Directions would change at almost every meeting. Our troops didn't have time to implement one set of instructions before the instructions got changed again. Further, the NCOs weren't getting enough time to supervise things and discipline/morale, etc. were going down the tubes. Finally we all got together and nominated the most senior of us to tell the Captain that we needed to curtail some of the meetings and to explain why. His response? "We'll have a staff meeting and discuss it." |
|
|
|
Carefree
Advanced Member
Philippines
4207 Posts |
Posted - 07 April 2009 : 03:17:19
|
Back on the original topic, you could add a routine to populate the field with numeric values directly within the pop_mail.asp and not have to have people run a different dbs script. Any reason this wouldn't work?
|
|
|
|
AnonJr
Moderator
United States
5768 Posts |
Posted - 07 April 2009 : 10:04:57
|
| That's only going to run if someone hits pop_mail.asp, but it will run every time someone hits pop_mail.asp. This really only needs to be addressed once after the upgrade, hence a DBS script. (or some genius developer could have remembered to make it part of the upgrade... ) |
|
|
|
Topic |
|
