Snitz Forums 2000
Snitz Forums 2000
Home | Profile | Register | Active Topics | Members | Search | FAQ
Username:
 Password:
Save Password
Forgot your Password?

 All Forums
 Snitz Forums 2000 MOD-Group
 MOD Add-On Forum (W/Code)
 New, faster search.asp for SQL Server DBs		  New Topic  Topic Locked
 Printer Friendly
Previous Page
Author Previous Topic Topic Next Topic
Page: of 3

ruirib
Snitz Forums Admin

Portugal
26364 Posts
Posted - 10 July 2003 :  01:48:50  Show Profile  Send ruirib a Yahoo! Message
I have confirmed the situation. It can be fixed by changing line 245 in the downloaded search.asp, so that where you have

strSql2 = strSql2 & " ON F.FORUM_ID = T.FORUM_ID) INNER JOIN " & strActivePrefix & "REPLY R"

you will have

strSql2 = strSql2 & " ON F.FORUM_ID = T.FORUM_ID) LEFT JOIN " & strActivePrefix & "REPLY R"


Haven't evaluated the impact this can have on speed. I wouldn't expect it to have that much, but it has can slow it down even if a tiny little bit. I can only do some testing later. Will post the results then. I will also update the downloaded file as soon as this has been completely tested. If anyone cares to test it as well, that will be welcome.

P.S.: A very quick test showed that the page with this change was even faster for me. I can only confirm this later, however, with a more thourough test.

Snitz 3.4 Readme | Like the support? Support Snitz too
Edited by - ruirib on 10 July 2003 01:53:49
Go to Top of Page

SiSL
Average Member

Turkey
671 Posts
Posted - 29 July 2003 :  07:32:18  Show Profile  Visit SiSL's Homepage
If strUseMemberDropDownBox = 0 Then another error occurs.

Microsoft OLE DB Provider for SQL Server error '80040e14' 

Incorrect syntax near the keyword 'UNION'. 

/forum/search.asp, line 445



I have no idea about SQL specific commands, so I cannot find where to correct it.

Infact for larger forums, ACTIVE.ASP needs something like this.
CHIP Online Forum

My Mods
Select All Code | Fix a vulnerability for your private messages | Avatar Categories W/ Avatar Gallery Mod | Complaint Manager
Admin Level Revisited | Merge Forums | No More Nested Quotes Mod
Edited by - SiSL on 29 July 2003 07:36:55
Go to Top of Page

ruirib
Snitz Forums Admin

Portugal
26364 Posts
Posted - 08 August 2003 :  09:43:18  Show Profile  Send ruirib a Yahoo! Message
Ok, found it. In lines 319 and 320, remove the parenthesis in red:

                strSql31 = strSql31 & " AND (M.MEMBER_ID = " & cLng(intSearchMember) & " "
		strSql32 = strSql32 & " AND (M.MEMBER_ID = " & cLng(intSearchMember) & " "

This will fix it. I've also uploaded a new zip file, with the search.asp page corrected.

Snitz 3.4 Readme | Like the support? Support Snitz too
Go to Top of Page

hitched
Starting Member

United Kingdom
7 Posts
Posted - 01 September 2003 :  16:39:19  Show Profile  Visit hitched's Homepage
Ruirib,
If this was done with stored procedures would it be much quicker?

Dean
Go to Top of Page
Page: of 3 Previous Topic Topic Next Topic  
Previous Page
 New Topic  Topic Locked
 Printer Friendly
Jump To:
Snitz Forums 2000 © 2000-2021 Snitz™ Communications Go To Top Of Page
This page was generated in 0.12 seconds. Powered By: Snitz Forums 2000 Version 3.4.07