[HOW TO - vB4] Create a New Tab in the navbar

[Lynne]

Do a search in this thread for "is_member_of" because that is what you would use in the condition and I know this has been covered before in this thread.

[mhudson]

ah thanks Lynne. I kinda skimmed though the thread 30 pages is a lot to take in.

all sorted now works perfectly

[Spyike]

Can somebody tell me what is wrong with my code? I keep getting a syntax error and cannot seem to see where I made an error..

if (is_member_of($vbulletin->userinfo, 1))
{
$tabselected = '';
$tablinks = '';
if (THIS_SCRIPT == 'spyikereg')
{
    $vbulletin->options['selectednavtab']='spyreg';
    $tabselected = ' class="selected"';
} 
$template_hook['navtab_middle'] .= '<li'.$tabselected.'><a class="navtab" href="http://spyike.org/register.php">Register</a>'.$tablinks.'</li>' ;

[Lynne]

Originally Posted by Spyike

Can somebody tell me what is wrong with my code? I keep getting a syntax error and cannot seem to see where I made an error..

if (is_member_of($vbulletin->userinfo, 1))
{
$tabselected = '';
$tablinks = '';
if (THIS_SCRIPT == 'spyikereg')
{
    $vbulletin->options['selectednavtab']='spyreg';
    $tabselected = ' class="selected"';
} 
$template_hook['navtab_middle'] .= '<li'.$tabselected.'><a class="navtab" href="http://spyike.org/register.php">Register</a>'.$tablinks.'</li>' ;

You forgot the ending } to your is_member_of condition.

[cellarius]

1. Why don't you give the exact error message instead if making us search & guess?
2. Use indenting to make your code more readable.
3. You're missing a closing curly bracket at the end, that's for sure.

// Edit: Lynne beat me to it

[Spyike]

Originally Posted by Lynne

You forgot the ending } to your is_member_of condition.

Where does it belong..?

Thank you for the quick response.

[HMBeaty]

Originally Posted by Spyike

Where does it belong..?

Thank you for the quick response.

Your code should look like:

PHP Code:

if (is_member_of($vbulletin->userinfo, 1))
{
    $tabselected = '';
    $tablinks = '';

    if (THIS_SCRIPT == 'spyikereg')
    {
        $vbulletin->options['selectednavtab']='spyreg';
        $tabselected = ' class="selected"';
    } 

    $template_hook['navtab_middle'] .= '<li'.$tabselected.'><a class="navtab" href="http://spyike.org/register.php">Register</a>'.$tablinks.'</li>' ;
} 


[Spyike]

Originally Posted by HMBeaty

Your code should look like:

Greatly appreciate it, thanks.

[HMBeaty]

Originally Posted by Spyike

Greatly appreciate it, thanks.

No problem

[Lynne]

Originally Posted by Spyike

Where does it belong..?

Thank you for the quick response.

It belongs at the very, very end. It should be your last line in the plugin.

edit: Well, I guess while I was typing Brooks was being the young whippersnapper he is and answering way, waaaaay faster than me!