Forum Display Enhancements - CES Intelligent Announcement Query

CES Intelligent Announcement Query
Version: 1.0.2

*** NEWS ***
1/28/2007 - v1.0.2 released
1/25/2007 - updated install instructions
1/24/2007 - v1.0.1 released
1/21/2007 - v1.0.0 released

Known Issues:
- none

What It Does:
Makes forumdisplay.php smarter. It will only use a query for Announcements IF there are any announcements for that forum. So more likely than not, this will improve your forumdisplay pages by virtually eliminating 1 query.

I actually wrote this for another mod I'm working on that contains a modified version of forumdisplay.php. Decided to release these plugins separately so people can benefit from it now.

Products to Install: 1
Plugins Included: 2
Files to Upload: 0
Template Edits: 0
Files to Edit: 1
Queries Added:* -1

*This only subtracts 1 query if you don't have any announcements configured for that forum.

*** Changelog ***
As of Version 1.0.2:
- added support for global announcements

As of Version 1.0.1:
- made the announcement check faster
- announcements will work after uninstall if file edits not undone

[thincom2000]

Version 1.0.1 was just uploaded. This contains instructions for an updated file edit that will do this check faster, and if someone is foolish enough to uninstall this product, announcements will now work without undoing the file edits.

[stomph]

Thanks for this hack!

There's a ) missing on line 477 in forumdisplay:
if ($announce_exists OR (!is_array($announces)))

[adhari_com]

Thanx for the great effort,
Although I think eliminating a single query would be replaced with the query calling the hook and also executing itself!

Just an opinion.

[thincom2000]

You are correct that the hooks creates a new query. However this is an additional query in the AdminCP Announcements page, and not one users would experience on your forumdisplay.

I would not have released this if it was that pointless.

Originally Posted by stomph

There's a ) missing on line 477 in forumdisplay:
if ($announce_exists OR (!is_array($announces)))

I have updated the installation instructions with the missing parenthesis. Thanks!

[TheFrienzNet]

As much as I love your other hacks, I love this one too;however, there is a bug once again. Global announcements do not show up.

[Get Shorty]

Originally Posted by TheFrienzNet

As much as I love your other hacks, I love this one too;however, there is a bug once again. Global announcements do not show up.

I don't use global announcements. Is that the only bug we're aware of?

I may install this one too...

[thincom2000]

Global Announcements fixed.

[Get Shorty]

Sweet. Installing...