[mykes]

The hack created an interesting bug that I'm not sure has been reported.

In showthread.php, the old code looks like this (pre-hack):

if ($threadedmode != 1) // if linear or hybrid
{
  exec_header_redirect("showthread.php?$session[sessionurl_js]p=$getlastpost[postid]$highlightwords#post$getlastpost[postid]");
}

It's been changed to this (Geek Articles Hack version):

if ($threadedmode != 1) // if linear or hybrid
{
  exec_header_redirect("showthread.php?$session[sessionurl_js]p=$getlastpost[postid]$highlightwords#post$getlastpost[postid]&conly=$commentsonly");
				}

With the original code commented in, and the Geek Articles hack code commented out, things work proper. Comment in the articles hack code and things don't work right.

So let me define "don't work right."

If you are on forumhome page and click on the graphic next to a thread link "go to first unread/new post" then it takes you to some post in the thread maybe 2 or 3 posts BEFORE where the first unread/new post actually is. Or even in the middle of some post.

Here's an example of the URL that's generated, so you can see what's wrong:

showthread.php?p=2032241#post2032241&conly=

(I think having the &conly after #post confuses the browser)

The fixed code that works on my site looks like this:

	            // === GEEK ARTICLES step 5.4 ===
	            // WAS:
//				if ($threadedmode != 1) // if linear or hybrid
//				{
//					exec_header_redirect("showthread.php?$session[sessionurl_js]p=$getlastpost[postid]$highlightwords#post$getlastpost[postid]");
//				}
                // NOW:
				if ($threadedmode != 1) // if linear or hybrid
				{
					exec_header_redirect("showthread.php?$session[sessionurl_js]p=$getlastpost[postid]$highlightwords&conly=$commentsonly#post$getlastpost[postid]");
				}
				// === /GEEK ARTICLES ===