![]() |
i have an idea...overgrow, post your forumdisplay.php as an attachment and i'll look at that part...
you and freddie will get credit now as well, of course ;) |
>>overgrow, post your forumdisplay.php as an attachment
Sorry, can't post whole vB files as it violates the license. Freddie made this super easy though.. I see no reason why this wasn't suggested as the first way to code the hack. Here is the important part of forumdisplay.. add the first part and modify the query by including $previewselect and $previewjoin. PHP Code:
One of my users asked why this didn't work for search results, so that is pretty easy to move over. Search.php find: PHP Code:
PHP Code:
find user.userid AS postuserid, replace with user.userid AS postuserid, $previewselect find LEFT JOIN icon AS threadicon ON thread.iconid=threadicon.iconid $dotjoin replace with LEFT JOIN icon AS threadicon ON thread.iconid=threadicon.iconid $dotjoin $previewjoin find PHP Code:
PHP Code:
I noticed this was your first project so I hope you write this all up into a new hack so it's not confusing for the other users. thanks~ |
ouch...forgot about the license... :O
okay...well this is the error i get (again, i'm not sure whether it's related to this 'dotqueries' thing ;)): Code:
Database error in vBulletin 2.2.2: |
Not sure what to tell you.. this is right out of my forumdisplay.php and it works. I just turned on "Use Dot Icons" in the Cpanel, which affects this query, and they both worked at the same time.. which was the main goal. The second join slowed it down, too much for my site, but probably OK for others.
|
PHP Code:
|
'thread.firstpostid' isn't a default field in vBulletin. It makes the hack easy as you know what to join on but you have to make other modifications to newthread to save the first postid when the thread is created. I saw it in the queries you guys posted but I don't see it in the hack instructions so I was wondering how you were making this work without creating that field.
|
If it is slowing you down, make sure you have put an index on "thread.firstpostid" otherwise you are going to have a full thread table scan which is bad(tm).
|
i made it work...i even made an install script to add and populate the extra field :) works like a charm, expect an update tonight
|
update released!
thread previews on searches will be in the next update |
I'm using Parker's fantastic hack with no problems. How much does this hack improve on Parker's? I'm just wondering whether it is worth the changeover... :)
|
All times are GMT. The time now is 07:23 PM. |
Powered by vBulletin® Version 3.8.12 by vBS
Copyright ©2000 - 2025, vBulletin Solutions Inc.
X vBulletin 3.8.12 by vBS Debug Information | |
---|---|
|
|
![]() |
|
Template Usage:
Phrase Groups Available:
|
Included Files:
Hooks Called:
|