Allowing only certain users to visit script

[BamaStangGuy]

I have a new product I am creating and one of the setting options allows you to add userids, seperated by a comma, that you want to have access to the script.

How can I take those userids and compare them to the current users id trying to access the script to make sure that they match?

Right now this isn't working:

PHP Code:

$aids = explode(',',$vbulletin->options['users_allowed']);
if ((!$vbulletin->options['vbincome_enabled']) OR in_array($bbuserinfo[userid], $aids))
{
    print_no_permission();
} 


What is the correct way to do this?

[jwocky]

Do you want to check the user who is currently viewing the script and decide if that user is allowed to view the script?

[Opserty]

Use $vbulletin->userinfo['userid'] instead of $bbuserinfo[userid]

I think you need to add a not (!) before the in_array().

[BamaStangGuy]

Originally Posted by jwocky

Do you want to check the user who is currently viewing the script and decide if that user is allowed to view the script?

Yes

Originally Posted by Opserty

Use $vbulletin->userinfo['userid'] instead of $bbuserinfo[userid]

I think you need to add a not (!) before the in_array().

Thanks I'll give that a go.

[Dean C]

What!? You mean vBSEO doesn't do all of this for you and more

PHP Code:

$aids = explode(',',$vbulletin->options['users_allowed']);
if (!empty($vbulletin->options['vbincome_enabled']) OR !in_array($bbuserinfo[userid], $aids))
{
    print_no_permission();
} 


Checking for !($value) will match false, as the poster below me pointed out.

[Opserty]

Are you sure you can use isset there? Because even if the option is set to say, false it will still read that the variable is set, won't it?

[Dean C]

Originally Posted by Opserty

Are you sure you can use isset there? Because even if the option is set to say, false it will still read that the variable is set, won't it?

Woopsie, too used to checking to see if array keys exist