[BamaStangGuy]

Originally Posted by Paul M

There is more than one way to do it, I prefer the method below for a small number of variations (some would say it should be done using a few IFELSE's, I hate them and always find them confusing to follow).

If there were more variations I would probably use another switch/case.

PHP Code:

case $vbulletin->options['forumhome'] . '.php':
    $userinfo['activity'] = 'index';
    if ($values['page'] == 'history') $userinfo['activity'] = 'history';
    if ($values['page'] == '2005history') $userinfo['activity'] = '2005intro';
    if ($values['page'] == 'fordtrademark') $userinfo['activity'] = 'fordtrademark';
    break; 


Btw, the layout is a personal preference for IF's that only perform one action, definitely against vb standards

Thank you that worked