| Author |
 Topic  |
|
|
pweighill
Junior Member
United Kingdom
453 Posts |
 Posted - 19 August 2002 : 08:45:24
|
In subscription_list.asp, the following line numbers have a cint(). This should be changed to a clng().
Lines: 138, 142, 146 |
Edited by - pweighill on 19 August 2002 08:46:50 |
|
|
HuwR
Forum Admin
United Kingdom
20584 Posts |
Posted - 19 August 2002 : 08:56:04
|
| i agree with 146, but you are ublikely to have more than 32,767 categories or forums |
 |
|
|
RichardKinser
Snitz Forums Admin
USA
16655 Posts |
Posted - 19 August 2002 : 13:00:55
|
| Lines 135 & 136 should have cLng added to them right? |
|
|
|
ruirib
Snitz Forums Admin
Portugal
26364 Posts |
Posted - 19 August 2002 : 13:13:16
|
quote:
Originally posted by RichardKinser
Lines 135 & 136 should have cLng added to them right?
I think so, as the ID values can be higher that 32,767 (although less likely for memberID, it can do no harm using CLng). |
Snitz 3.4 Readme | Like the support? Support Snitz too |
|
|
|
bax
Junior Member
141 Posts |
Posted - 19 August 2002 : 21:29:23
|
| Is this considered a documented bug for 3.4, and therefore should be changed? My lines 135 and 136 do not include the cLng(). |
|
|
|
RichardKinser
Snitz Forums Admin
USA
16655 Posts |
Posted - 19 August 2002 : 21:42:56
|
| Yes, go ahead and make the changes listed here. These fixes will be included in v3.4.01 which will be a required upgrade if you are using v3.4 so you can either do it now, or wait until v3.4.01 is released. |
|
|
|
RichardKinser
Snitz Forums Admin
USA
16655 Posts |
Posted - 21 August 2002 : 01:11:07
|
| fixed in v3.4.01 |
|
|
|
Rasyr
Junior Member
USA
199 Posts |
Posted - 21 August 2002 : 07:56:46
|
| hmm... putting clng() on my lines 135 & 136 broke the page for me, so I removed them |
**TANSTAAFL** |
|
|
|
RichardKinser
Snitz Forums Admin
USA
16655 Posts |
Posted - 21 August 2002 : 08:03:58
|
ok, DO NOT add clng() to lines 135 & 136
fixed in v3.4.01 |
|
|
| |
Topic |
|
Forum Locked
Topic Locked
