View Full Version : [fixed] private forums showing up in forumjump on failed admincp/modcp login
Matze0007
Thu 31st Oct '02, 10:08am
Hiho =)
If someone trys to access the admin/mod cp using a valid admin/mod username but wrong password, all private forums which the admin/mod has access to will appear in the forumjump box on the wrong password page.
To fix this open admin/sessions.php and find:
eval("standarderror(\"".gettemplate("error_wrongpassword")."\");");
Above it add:
$bbuserinfo['usergroupid'] = 1;
filburt1
Sat 2nd Nov '02, 12:33pm
Confirmed, and interesting forums here :D
filburt1
Sat 2nd Nov '02, 12:33pm
One more problem: they generate 404 errors because they try to go to mod/forumdisplay.php.
Mystics
Sat 2nd Nov '02, 3:42pm
Thx @Matze0007 for the Info, bug is confirmed in my test forums.
But I would use this fix:
Open admin/sessions.php and find:eval("standarderror(\"".gettemplate("error_wrongpassword")."\");");and add above it:$useforumjump = 0;With this, the forumjump box doesn't show up at all, and therefore the 404-Bug is also 'fixed' ;)
(I don't think you ever will need the forumjump menu at this location!)
Mystics
Scott MacVicar
Sat 2nd Nov '02, 7:22pm
well you shouldnt be getting a forumjump full stop there, i'm adding
$useforumjump = 0;
above
eval("standarderror(\"".gettemplate("error_wrongpassword")."\");");
in sessions.php as there is no point of having it
Scott MacVicar
Sat 2nd Nov '02, 7:23pm
lol i forgot to push submit before i went out, I see mystics has already posted the fix.
I've commited this fix to cvs.
Erwin
Sun 3rd Nov '02, 3:03am
Thanks for the fix. What an interesting bug! :)
DWZ
Sun 3rd Nov '02, 6:46am
Thanks, worked great :)
Chris M
Sun 3rd Nov '02, 11:10am
:)
Satan
DWZ
Sun 3rd Nov '02, 8:53pm
Originally posted by PPN
I've commited this fix to cvs. Just a thought - but shouldn't someone put [fixed] in the topic?
Paul
Sun 3rd Nov '02, 10:45pm
Originally posted by DWZ
Just a thought - but shouldn't someone put [fixed] in the topic?
Make it more interesting--put [fixed in vb4.1] in the topic ;) See how many people pee themselves ;)
Scott MacVicar
Mon 4th Nov '02, 11:56am
putting fixed in the title is just a reminder to a dev so they dont try and fix it twice :)
anyway done now
vBulletin® v3.8.0 Release Candidate 1, Copyright ©2000-2008, Jelsoft Enterprises Ltd.