| Author |
 Topic  |
|
|
Da_Stimulator
DEV Team Forum Moderator
USA
3373 Posts |
 Posted - 07 November 2001 : 23:40:53
|
This bug only occurs when implementing the bug fix related to single quotes(') in member names. This happens when a user trys to log into there profile.
Line 678 of pop profile.asp change from
strSql = strSql & " WHERE "&Strdbntsqlname&" = '" & ChkString(STRdbntUserName, "decode") & "' "
to
[code]strSql = strSql & " WHERE "&Strdbntsqlname&" = '" & ChkString(STRdbntUserName, "archive") & "' "code]
Fixed for 3.4
---------------
-Da_Stimulator
Stims Snitz Test area - Running 3.3.03, 4 beta, and Huw's modified code
Need a Mod? Check out the Mod Resource
Edited by - da_stimulator on 07 November 2001 23:55:00 |
|
|
Da_Stimulator
DEV Team Forum Moderator
USA
3373 Posts |
Posted - 07 November 2001 : 23:45:35
|
note the above did not work, working on a solution
---------------
-Da_Stimulator
Stims Snitz Test area - Running 3.3.03, 4 beta, and Huw's modified code
Need a Mod? Check out the Mod Resource |
 |
|
|
Da_Stimulator
DEV Team Forum Moderator
USA
3373 Posts |
Posted - 07 November 2001 : 23:56:14
|
ok Additionally at line 686 you have to remove the ChkQuoteOK function
Old:
if rs.BOF or rs.EOF or not(ChkQuoteOk(STRdbntUserName)) or not(ChkQuoteOk(Password)) then
New:
if rs.BOF or rs.EOF or not(ChkQuoteOk(Password)) then
---------------
-Da_Stimulator
Stims Snitz Test area - Running 3.3.03, 4 beta, and Huw's modified code
Need a Mod? Check out the Mod Resource |
|
|
|
Davio
Development Team Member
Jamaica
12217 Posts |
Posted - 08 November 2001 : 13:08:18
|
Stim, you're doing a good job so far. But just a little concern I have, when you find a bug, could you leave it open a while so we can confirm that it is a bug that needs to be fixed, and that your suggested fix is ok before implementing it into the source code?
So if it doesn't work, everyone can't say it was Da_Stimulator's fault. They can point fingers at all the Admins and DEV Team members, since we would have to confirm it works first.  
- David |
|
|
|
Da_Stimulator
DEV Team Forum Moderator
USA
3373 Posts |
Posted - 08 November 2001 : 13:12:59
|
Sure :) But this isnt a 'bug' anymore - as we decided not to fix the apostrophe thing, since it complicates things too much
---------------
-Da_Stimulator
Stims Snitz Test area - Running 3.3.03, 4 beta, and Huw's modified code
Need a Mod? Check out the Mod Resource |
|
|
|
Deleted
deleted
4116 Posts |
Posted - 01 December 2001 : 10:17:59
|
Well, whats the final decision? The alpha code is already different:
strSql = strSql & " WHERE "&Strdbntsqlname&" = '" & ChkString(STRdbntUserName, "SQLString") & "' "
Didn't implement it in v40b03patch001 yet.
Think Pink
Test Site not ready yet | Post v40b03 Patches |
|
|
| |
Topic |
|
Forum Locked
Topic Locked
