is_member_of improvement.

Requested by Boofo

Some people want to check for multiple groups at once, instead of having to slam OR is_member_of(anothercheck) together for each group, if you apply this hack it will be simple to check for multiple groups.

The format for the function is now,

is_member_of($bbuserinfo, ID1, ID2, ID3,.....);

Just seperate each group by a comma, no need for extra quotes or anything.

Find the function in includes/functions.php ~line 190 and replace it with the following.

This should not affect any current is_member_of conditionals you're using.

PHP Code:

function is_member_of()
{
    static $argpad, $user_memberships;

    $usergroupids = func_get_args();
    $userinfo = array_shift($usergroupids);
    
    // check primary usergroup against given usergroupids
    if(in_array($userinfo['usergroupid'], $usergroupids))
    {
        return true;
    }
    
    // if we dont already have membergroupids, fetch them.
    if(!is_array($user_memberships["$userinfo[userid]"]))
    {
        $user_memberships["$userinfo[userid]"] = fetch_membergroupids_array($userinfo);    
    }
    
    foreach($usergroupids as $usergroupid)
    {
        if(in_array($usergroupid, $user_memberships["$userinfo[userid]"]))
        {
            return true;
        }
    }
    
    return false;
} 


[amykhar]

Originally Posted by mcyates

replaced it all and i got this:

Fatal error: Cannot redeclare is_member_of() (previously declared in /usr/home/yates238/public_html/includes/functions.php:176) in /usr/home/yates238/public_html/includes/functions.php on line 208

Then you didn't replace it all. Go back and make sure you got the whole function.

[mcyates]

Originally Posted by amykhar

Then you didn't replace it all. Go back and make sure you got the whole function.

LeeCHeSSS script worked, but the orignal one didn't. I'll try your hack now. Thanks for the support.

[noppid]

A template only mod to do this is available here if you would like to avoid editing php files.

http://www.vbulletintemplates.com/mo...ead.php?t=7570

[Boofo]

Has this been added to 3.5 RC 1?

[amykhar]

It's built in. Go read the code and you'll see that Jelsoft was wise and merciful and no hack is needed any more.

[Boofo]

Great! Then we can still use it in templates the way Merk descibed, now, right?

[amykhar]

Yep. But, slightly changed:

<if condition="is_member_of($vbulletin->userinfo,6,5)">
Hello World!
</if>

[Paul M]

$bbuserinfo should still work in templates shouldn't it ?

[amykhar]

Originally Posted by Paul M

$bbuserinfo should still work in templates shouldn't it ?

Yes. But, I try to upgrade to the new style when adding new conditionals.