[Ocean]

Originally Posted by Colin F

I would use the single tiny squigly lines-on-top (') instead of the double ones (") here:

implode(", ", $vieweduserids)

making it:

implode(', ', $vieweduserids)

also, you might want to save the ID's in the database using the format 1,2,3,4,5,6,7 as you woudn't have to explode() and implode() again.


Then again, if anything I said doesn't make sense, excuse me. I'm extremely tired...

I tried making the character change - it didn't work. I even tried using that character change for all instances of the "implode" function, and that didn't do anything either.

As for your second idea, if you think that is the better way to do it, I'm all for it. However, I'm not sure how to modify the code to make it work that way.


This is the original addition to the ShowThread.php file:

PHP Code:

 
 
$showinvisible = 1; 
// Get who has already viewed this thread 
$currentthread = $DB_site->query_first("SELECT whoviewed FROM ".TABLE_PREFIX."thread WHERE threadid = $threadid"); 
$vieweduserids = explode(" ", $currentthread['whoviewed']); 
$userinfo = fetch_userinfo($bbuserinfo['userid']); 
if (!$userinfo['invisible'] OR $showinvisible) 
{ 
    if (!empty($currentthread['whoviewed'])) 
    { 
        if (!in_array($bbuserinfo['userid'], $vieweduserids)) 
        { 
            $DB_site->query("UPDATE ".TABLE_PREFIX."thread SET whoviewed = CONCAT(whoviewed, \" \", \"" .$bbuserinfo['userid'] . "\") WHERE threadid = $threadid"); 
        } 
    } 
    else 
    { 
        $DB_site->query("UPDATE ".TABLE_PREFIX."thread SET whoviewed = \"" . $bbuserinfo['userid'] . "\" WHERE threadid = $threadid"); 
    } 
} 
// Who has viewed this thread? 
if (empty($currentthread['whoviewed'])) 
{ 
    $thread['viewers'] = '('.strtolower($vbphrase['none']).')'; 
} 
else 
{ 
    $result = $DB_site->query("SELECT userid, username FROM ".TABLE_PREFIX."user WHERE userid IN (" . implode(", ", $vieweduserids) . ")"); 
    $viewers = array(); 
    while ($user = $DB_site->fetch_array($result)) 
    { 
            array_push($viewers, "<a href=\"member.php?".$session['sessionurl'] . "u=".$user['userid'] . "\">" . htmlspecialchars($user['username']) .    "</a>"); 
    } 
    $thread['viewers'] = implode(", ", $viewers); 
} 


If you could show me what I need to do in order to make the original code work with the comma delineated format - I can then try again to modify it by adding a second section and see if it works.

(p.s. Thank you for your patience, and your help!)