Permission problem

[subnet_rx]

I have read several articles, did some trial and error, and basically figured out many of my questions. At this point in my script, I have a question about usergroup_permissions. I got this code out of Psionic's tutorial, but it doesn't seem to work for me. Everyone is getting to the form. Here it is:

if ($_REQUEST['do']=='create_event' AND (!is_member_of($vbulletin->userinfo, 6) OR !is_member_of($vbulletin->userinfo, 76)))
{
	eval('$usernamecode = "' . fetch_template('newpost_usernamecode') . '";');
	if (!$_REQUEST['outcomes'])
	{
		eval('$picksbody = "' . fetch_template('picks_outcomes') . '";');
	}
	else
	{
		$o = $_REQUEST['outcomes'];
		for($i=0; $i<=$o; $i++)
		{
			$outcomes .= "<p><label>Outcome ".$i.":<input name=\"outcome[".$i."]\" type=\"text\" id=\"outcome".$i."\" size=\"64\" maxlength=\"127\" /></label></p>";
		}
		eval('$picksbody = "' . fetch_template('picks_new_event') . '";');
	}
}

Users in group 6 (me) and users not in either group still get the code executed in the function.

[Kirk Y]

Who exactly do you want to have access?

And this:

(!is_member_of($vbulletin->userinfo, 6) OR !is_member_of($vbulletin->userinfo, 76))

Should be:

(!is_member_of($vbulletin->userinfo, 6,76))

[subnet_rx]

Originally Posted by Kirk Y

Who exactly do you want to have access?

And this:

(!is_member_of($vbulletin->userinfo, 6) OR !is_member_of($vbulletin->userinfo, 76))

Should be:

(!is_member_of($vbulletin->userinfo, 6,76))

I want members of the groups 6 and 76 to have access. No one else.

[Paul M]

In which case remove the ! at the start of Kirks suggestion - atm that is saying anyone who is not a member of groups 6 or 76 (you can also remove the brackets around it).

[subnet_rx]

Well, I knew that's what it should mean, but in Psionic's tutorial, he has this:

Certain Usergroups Only (In this example, 6 and 7)

PHP Code:
if (!is_member_of($vbulletin->userinfo, 6) AND !is_member_of($vbulletin->userinfo, 7))
{
print_no_permission();
}

[Kirk Y]

Yes, that's allowing members of Usergroup 6 and 7 only; all other get a permission error. When dealing with the is_member_of function, the second argument can contain multiple usergroup ids, so you don't need to use two separate conditions..