[akanevsky]

Instead of doing:

PHP Code:

$table = "color_groups";
if (!table_exists($table))
{
$vbulletin->db->query("CREATE TABLE `" . TABLE_PREFIX . "color_groups` (
  `color_id` int(15) NOT NULL auto_increment,
  `name` varchar(255) NOT NULL default '',
  `color` varchar(255) NOT NULL default '',
  `orderid` int(10) NOT NULL default '0',
  PRIMARY KEY  (`color_id`),
  UNIQUE KEY `name` (`name`)
) TYPE=MyISAM;");
} 


You should do:

PHP Code:

$vbulletin->db->query_write("CREATE TABLE IF NOT EXISTS `" . TABLE_PREFIX . "color_groups` (
  `color_id` int(15) NOT NULL auto_increment,
  `name` varchar(255) NOT NULL default '',
  `color` varchar(255) NOT NULL default '',
  `orderid` int(10) NOT NULL default '0',
  PRIMARY KEY  (`color_id`),
  UNIQUE KEY `name` (`name`)
) TYPE=MyISAM"); 


Besides, I see no reason why you can't use usergroupcache to derive the list of color usergroups.