Author |
Topic |
|
redbrad0
Advanced Member
USA
3725 Posts |
Posted - 09 January 2001 : 16:01:34
|
When the database setup my tables i name my memebers table different then the forum tables. it created the the ALLOWED_MEMBERS table with the prefix of the the forum, and the the in the post_info.asp on line 1414
my_Conn.execute ("DELETE FROM " & strMemberTablePrefix & "ALLOWED_MEMBERS WHERE FORUM_ID = " & fForumId)
It uses the member table prefix, this is an easy fix, either fix it before it sets up your database, or change in the post_info.asp file line 1414 to...
my_Conn.execute ("DELETE FROM " & strTablePrefix & "ALLOWED_MEMBERS WHERE FORUM_ID = " & fForumId)
Brad |
|
HuwR
Forum Admin
United Kingdom
20584 Posts |
Posted - 09 January 2001 : 16:09:58
|
The problem is setup.asp, all instances in the code refer to strMemberTablePrefix
The correct thing would be to fix setup.asp
But I would recomend renaming the table if it has already been created since it is referered to as strMemberTablePrefix in post.asp and inc_functions and post_info, so you will have to change it many times.
'Resistance is futile' |
|
|
redbrad0
Advanced Member
USA
3725 Posts |
Posted - 09 January 2001 : 18:26:49
|
well im sure im not the only one that had this problem, just in case you want to know im using sql server 7.
hopefully people will be able to change the code :O)
Brad |
|
|
HuwR
Forum Admin
United Kingdom
20584 Posts |
Posted - 09 January 2001 : 18:42:33
|
No, I'm sure your not the only one affected, I will leave a message for Mike in the moderators forum, and see if we can get it fixed fairly quickly, it could cause quite serious problems for multi forum sites.
Well spotted.
'Resistance is futile' |
|
|
HuwR
Forum Admin
United Kingdom
20584 Posts |
Posted - 10 January 2001 : 06:14:10
|
Actually, I had another think about this, it is the code that is wrong,
all instance of & strMemberTablePrefix & "ALLOWED_MEMBERS should be changed to & strTablePrefix & "ALLOWED_MEMBERS
if not, you will get priblems with forum ID's
'Resistance is futile' |
|
|
redbrad0
Advanced Member
USA
3725 Posts |
Posted - 10 January 2001 : 07:11:18
|
huwr,
I think it should be the strMemberTablePrefix, because on my site, and probley on many other sites, they use the member database for many other things. if they do not want to allow someone on the forum, they probley dont want to allow them on the rest of their website also. well thats how i feel, and i would rather it have the strMemberTablePrefix
just my $0.02
:O)
Brad |
|
|
HuwR
Forum Admin
United Kingdom
20584 Posts |
Posted - 10 January 2001 : 07:26:48
|
I thought about it and it shouldn't, the allowed member list is related to forums NOT members.
If you use strMemberTablePrefix, and you have a two foryms with the same ID, the user will be able to get into both forums on both boards, not the desired effect of a private forum.
it MUST be changed to strTablePrefix
'Resistance is futile' |
|
|
redbrad0
Advanced Member
USA
3725 Posts |
Posted - 10 January 2001 : 08:39:41
|
sorry your right huwr, i was not thinking that way, thanks for helping me clear my head :O) i still think this is something that should be fixed very quickly, its an easy fix. it could cause some major problems for some users
Brad |
|
|
HuwR
Forum Admin
United Kingdom
20584 Posts |
Posted - 10 January 2001 : 08:46:36
|
it's ok, you were obviously thinking the same way I was when I first posted, it wasn't until I went to bed that it suddenly dawned on me what I had said.
The files that require fixing are inc_functions.asp, post.asp and post_info.asp
just replace all instances of strMemberTablePrefix & "ALLOWED_MEMBERS with strTablePrefix & "ALLOWED_MEMBERS
'Resistance is futile' |
|
|
Reinsnitz
Snitz Forums Admin
USA
3545 Posts |
Posted - 02 February 2001 : 23:56:50
|
Updated in source of sf2k_31sr5a2.zip
Reinsnitz (Mike) ><)))'> "Blessed are those who hear the word of God and obey it." - Luke 11:28
|
|
|
|
Topic |
|