print_no_permission(); Problem
 

I am making a mod and my mod is loaded in iframes. The code for the frames is in a template and the template is called by my php file. I have everything working except rather then the No Permission page loading in the frames, I want the frames to just not load. I need the frames to not load when the mod is not activated or if the member is not in the right usergroup. All I need is the code or atleast a nudge in the right direction for the else statement.

As I said I have the code that will display the mod, now i just need the else code that will keep the frames template from loading so that not even the permissions page shows.

If my code is needed to be seen then just say and I will post the part that is catching me.

Wrap it in a conditional. Something like:

$array = array(1,2,3,4,5,6);

if(is_member_of($array))
{
display_code_here();
}

That way, if they do not succed in the conditional, the server will not run the display code.

Ok I am not sure how that would work. I have the code to check if the member is in the right group. I have it set up so that if the mod is active then it checks if the member is in the right group. now in the else is where I need to not display the frame.

You don't need to write an else clause, if the conditional does not match then it simply won't execute the code between the braces.

I think maybe you are over complicating matters? It's not exactly clear what you are trying to achieve.

This is the code I have.
I need to replace
with the code that will get rid of the frames if the mod is not enabled or if it is and the member is just not in the required group.

Make a new template? Like imagewtf_noframes and then check if they have frames or something?

I think you may be going about it the wrong way. Heres the way I would have done it.

Example template imagewtf_frame:
Example template imagewtf_shell:
Notice the $innerhtml variable, now if the modification is not active or the user is not in the required usergroup then $innerhtml will be empty and therefore will not appear. I don't see why you need to use frames unless you are trying to achieve something else.

Also some general pointers.

1. You can store settings in vBulletin Options (Enter debug mode then navigate to your options). If you add them to vBulletin options then you can access by using $vbulletin->options['....'] instead of performing a number of queries just to obtain settings for your modification.
2. If you are querying for just a single piece of data from MySQL use the $db->query_first(...) database function, it will reduce the lines of code you need significantly. For example:
   
   PHP Code:

   ---

   $settings = $db->query_first("SELECT * FROM imagewtf_settings WHERE settingname='allowedusergroups' LIMIT 1");
// No need for this line: $settings = $db->query_read($query);
// No need for this line: $setting = $db->fetch_array($settings);
$set = $setting['switch2']; 


   ---

3. Use the is_member_of($userinfo, $usergroups); function as this checks both primary and secondary usergroups.
   
   PHP Code:

   ---

   // $allowedug = explode(",", $set);
// if(in_array ( $usergroupid, $allowedug ))
// ------------- NOW BECOMES ---------------

$allowedug = exmplode(',', $set);
if(is_member_of($vbulletin->userinfo, $allowedug))
{
//... 


   ---