[bira]

Actually I prefer it in a separate file, FireFly - less calls and code parsing. There's no need to parse the entire showthread.php code for this function, hence I'd rather separate them and be more economic.

If, however, you prefer to only edit showthread.php, then what you did is not enough. Do the following:

1) Find

if ($goto=="nextnewest") {

And change it to:

if (($goto=="nextnewest") && (!isset($postuserid))) {

2) Find

if ($goto=="nextoldest") {

And change it to:

if (($goto=="nextoldest") && (!isset($postuserid))) {

3) Find

// draw nav bar
$navbar=makenavbar($threadid,"thread",0);

And change it to:

// draw nav bar
if (isset($postuserid)) {
	$navbar=makenavbar($threadid,"thread",1);
} else {
	$navbar=makenavbar($threadid,"thread",0);
}

4) Find

if ($thread[pollid]) {

And change it to:

if (($thread[pollid]) && (!isset($postuserid))) {

5) Find

$postscount=$DB_site->query_first("SELECT COUNT(*) AS posts FROM post WHERE post.threadid='$threadid' AND post.visible=1");

And change it to:

$postscount=$DB_site->query_first("SELECT COUNT(*) AS posts FROM post WHERE post.threadid='$threadid' ".iif(isset($postuserid), "AND post.userid='$postuserid'", "")." AND post.visible=1");

6) Find

$getpostids=$DB_site->query("
	SELECT post.postid FROM post
	WHERE post.threadid='$threadid' AND post.visible=1
	ORDER BY dateline $postorder LIMIT ".($limitlower-1).",$perpage
");

And change it to (this is the change you already made):

$getpostids=$DB_site->query("
	SELECT post.postid FROM post
	WHERE post.threadid='$threadid' ".iif(isset($postuserid), "AND post.userid='$postuserid'", "")." AND post.visible=1
	ORDER BY dateline $postorder LIMIT ".($limitlower-1).",$perpage
");

7) Find

$pagenav = getpagenav($totalposts,"showthread.php?s=$session[sessionhash]&threadid=$threadid&perpage=$perpage");

And change it to

$pagenav = getpagenav($totalposts,"showthread.php?s=$session[sessionhash]&threadid=$threadid".iif(isset($postuserid), "&postuserid=$postuserid", "")."&perpage=$perpage");

These are the changes I could pick up off the top of my head. Personally I prefer to separate the two, so as not to add more procedures to showthread, and at the same time to deduct unnecessary procedures from showposts

Cheers,

Bira