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;
} 


[Boofo]

You have once again proven yourself the "Master"! Thank you, very much, sir.

[high]* Boofo skulks away having proven to not be a good Grasshopper to Master. [/high]

[Floris]

Originally Posted by Boofo

You have once again proven yourself the "Master"! Thank you, very much, sir.

[high]* Boofo skulks away having proven to not be a good Grasshopper to Master. [/high]

Great work, and thank you for sharing. You are a good and kind person.

[LeeCHeSSS]

Wouldn't it be easier to allow the second argument to be an array with usergroup id's?

ie.

PHP Code:

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


That way using is_member_of is easier to use in "dynamic" situations...

[merk]

MY initial use of array() in the conditionals didnt work, maybe because of the way it is evaluated.

If you can work out if array() works in conditionals it would be an easy modification to the code

[LeeCHeSSS]

Try this:

PHP Code:

// ###################### Start is member of #######################
// returns true/false if a $userinfo belongs to $usergroupid
// $userinfo must contain (userid, usergroupid, membergroupids)
function is_member_of($userinfo, $usergroupid)
{
        static $user_memberships;

        if (!is_array($user_memberships["$userinfo[userid]"])) {
                $user_memberships["$userinfo[userid]"] = fetch_membergroupids_array($userinfo);
        }

        if (is_array($usergroupid)) {
                for ($i = 0; $i < sizeof($usergroupid); $i++) {
                        if ($userinfo['usergroupid'] == $usergroupid[$i]) {
                                return true;
                        } else if (in_array($usergroupid[$i], $user_memberships["$userinfo[userid]"])) {
                                return true;
                        }
                }
        } else {
                if ($userinfo['usergroupid'] == $usergroupid) {
                        return true;
                } else {
                        return in_array($usergroupid, $user_memberships["$userinfo[userid]"]);
                }
        }
} 


[Gary King]

Interesting hack, should be useful to many

[mcyates]

What exactly do i raplace????

// ###################### Start is member of #######################
// returns true/false if a $userinfo belongs to $usergroupid
// $userinfo must contain (userid, usergroupid, membergroupids)
function is_member_of($userinfo, $usergroupid)
{
	static $user_memberships;

	if ($userinfo['usergroupid'] == $usergroupid)
	{
		// user's primary usergroup is $usergroupid - return true
		return true;
	}
	else if (!is_array($user_memberships["$userinfo[userid]"]))
	{
		// fetch membergroup ids
		$user_memberships["$userinfo[userid]"] = fetch_membergroupids_array($userinfo);
	}

	// return true/false depending on membergroup ids
	return in_array($usergroupid, $user_memberships["$userinfo[userid]"]);
}

[chris2707]

Replace all of it unless you've previously hacked that function.

[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