[SDB]

Thanks guys

I've changed it to the following (which I've taken from somewhere, can't remember where now!) ..

$blockforums = "";
foreach($vbulletin->forumcache AS $forum) {
    $forumid = $forum['forumid'];
    $forumperms =& $vbulletin->userinfo['forumpermissions']["$forumid"];

    if (!isset($vbulletin->forumcache["$forumid"]) OR !($forumperms & $vbulletin->bf_ugp_forumpermissions['canview']) OR !($forumperms & $vbulletin->bf_ugp_forumpermissions['canviewothers']) OR !verify_forum_password($forumid, $vbulletin->forumcache["$forumid"]['password'], false))
    {
        $blockforums .= ','.$forum['forumid'];
    }
}


$threads = $vbulletin->db->query_read("SELECT threadid, title, lastpost, lastposter
									FROM " . TABLE_PREFIX . "thread
									WHERE visible = '1' AND open = '1'
							            AND forumid NOT IN (0$blockforums)
									ORDER BY lastpost DESC LIMIT 5
									");

Does that look ok?

Thanks again

Simon