Author |
Topic |
|
RichardKinser
Snitz Forums Admin
USA
16655 Posts |
|
Hamlin
Advanced Member
United Kingdom
2386 Posts |
Posted - 17 August 2002 : 09:08:45
|
It seems you no longer get the email boxes if you are not logged in, was this part of the fix?
I just get this text now
quote:
Send an EMAIL Message
Close Window
Where as before if I was not logged in I would get the chance to enter my infomation... |
|
|
theory
Starting Member
29 Posts |
Posted - 17 August 2002 : 09:10:41
|
Does this apply to the Internationalized version? |
|
|
RichardKinser
Snitz Forums Admin
USA
16655 Posts |
Posted - 17 August 2002 : 09:21:32
|
quote: Originally posted by Hamlin
It seems you no longer get the email boxes if you are not logged in, was this part of the fix?
I just get this text now
quote:
Send an EMAIL Message
Close Window
Where as before if I was not logged in I would get the chance to enter my infomation...
No, not really. But it could. I've never unchecked the "Require Logon for sending Mail" in the e-mail options so I didn't test this fix with it. But, you should know that if you don't require logon for mail, the person can put in any username and any e-mail address they want, it is not checked against the database. |
|
|
RichardKinser
Snitz Forums Admin
USA
16655 Posts |
Posted - 17 August 2002 : 09:22:40
|
quote: Originally posted by theory
Does this apply to the Internationalized version?
No, Bozden will have to release a fix for the internationalized version, or someone else who is familiar with it. You can probably download the file and compare the changes with your file and just make the changes yourself. |
|
|
RichardKinser
Snitz Forums Admin
USA
16655 Posts |
Posted - 17 August 2002 : 09:54:39
|
I had to add some missing text in the file, if you have already downloaded it, please redownload it again. |
|
|
Deleted
deleted
4116 Posts |
Posted - 17 August 2002 : 11:43:26
|
quote: Originally posted by theory
Does this apply to the Internationalized version?
I'm working on it now. I'll release patch005 ASAP. |
Stop the WAR! |
|
|
mortioli
Average Member
United Kingdom
898 Posts |
Posted - 17 August 2002 : 12:13:19
|
Is it possible to provide the codes needed to be changed?
I think I've changed my pop_mail, so would be great if the changes could be pointed out.
Thanks! |
|
|
D3mon
Senior Member
United Kingdom
1685 Posts |
Posted - 17 August 2002 : 12:41:05
|
Just wondering if it would pose a security threat to publish just the affected lines? If so, I'll copy over my original and re-modify it. |
Snitz 'Speedball' : Site Integration Mod : Friendly Registration Mod "In war, the victorious strategist only seeks battle after the victory has been won" |
|
|
Deleted
deleted
4116 Posts |
Posted - 17 August 2002 : 13:48:37
|
Fixed in v4b03patch005 (released now, download link here). |
Stop the WAR! |
|
|
Anacrusis
Junior Member
USA
219 Posts |
|
RichardKinser
Snitz Forums Admin
USA
16655 Posts |
Posted - 17 August 2002 : 16:07:19
|
If you've changed your file, just download this one and compare the 2 and make the necessary changes to yours. |
|
|
mortioli
Average Member
United Kingdom
898 Posts |
Posted - 17 August 2002 : 16:49:18
|
I've tried to compare them, but theres to much which has changed between the two. |
|
|
RichardKinser
Snitz Forums Admin
USA
16655 Posts |
Posted - 17 August 2002 : 16:56:55
|
the major parts to fix are:
on line #43:
replace this:
set rs = Server.CreateObject("ADODB.RecordSet")
with this:
if Request.QueryString("ID") <> "" and IsNumeric(Request.QueryString("ID")) = True then intMemberID = cLng(Request.QueryString("ID")) else intMemberID = 0 end if
on line #53:
change this:
rs = my_Conn.Execute (strSql)
to this:
set rs = my_Conn.Execute (strSql)
then on line #51 & line #167:
replace this:
Request.QueryString("ID")
with this:
intMemberID |
|
|
mortioli
Average Member
United Kingdom
898 Posts |
Posted - 20 August 2002 : 16:52:07
|
Cheers |
|
|
|
Topic |
|