[Dave]

I would escape the variables in the SQL query and use vB:: for the database calls as well. You can try the following:

PHP Code:

$postcnt = vB::$vbulletin->userinfo['posts']; 
$usrgrp = vB::$vbulletin->userinfo['usergroupid']; 
$name = vB::$vbulletin->userinfo['username']; 
$newmem = vB::$vbulletin->db->query_first("SELECT username FROM " . TABLE_PREFIX . "user WHERE userid = " . $members['maxid'] . " AND usergroupid NOT IN (3,4,8,17)");  
$usid = vB::$vbulletin->db->query_first("SELECT userid FROM " . TABLE_PREFIX . "user WHERE userid = " . $members['maxid'] . " AND usergroupid NOT IN (3,4,8,17)");  

if ($postcnt < 1 && $usrgrp == 2) { 
    $output = "<center>Welcome " . $name . ", you should introduce yourself -> <a href='http://rpgchat.com/forumdisplay.php/227-Welcome-Center'><font color='red'><u><b>Welcome Center</font></u></b></a></center>";     
} else { 
    $output =  "<center>Welcome Back " . $name . ", our newest member is " . $newmem . ", why not <a href='rpgchat.com/private.php?do=newpm&u='" . $usid . "'><u><b><font color='red'>Greet Them?</font></b></u></center></a>"; 
}