View Single Post
naris
AlliedModders Donor
Join Date: Dec 2006
Old 10-20-2010 , 10:04   Re: Server Redirect 1.1 [Last update: Februari 7, 2009]
Reply With Quote #215

Quote:
Originally Posted by thetwistedpanda View Post
the most recent updates to this plugin inserted the 'id' parameter into the queries, which shifted the original indexes up by one. But you're the only one that had this issue, where as it seems to have worked for everyone else, but everyone should have this problem. This is why I'm quite confused.
As you noticed, the query field ids have indeed been bumped up by 1 because the query built in CreateServerListQuery() was changed.

However, the menu still works because the code in Query_ActiveServers() had been changed to use the new field ids, yet the code in LoadServerRedirectList_Query() had been left unchanged even though it uses the same query created in CreateServerListQuery().

That resulted in any plugins hooking LoadServerRedirectList() or LoadServerRedirectListFiltered(), such as serverredirect_ads, getting incorrect values and using the address instead of the display_name, which would also prevent {NAME} and other tags from being expanded properly.

Last edited by naris; 10-20-2010 at 10:08.
naris is offline