function not evaluating correctly

[Hypertext]

Heres my function, it always returns 0...

function user_type($page) {
    $user = $logged_user;
    $level = mysql_query("SELECT * FROM users WHERE user='$user'");
    $level_a = mysql_fetch_array($level);
    $final_level = $level_a['usergroup'];
    $check_level = mysql_query("SELECT * FROM userstructure WHERE name='$final_level'");
    $check_level_a = mysql_fetch_array($check_level);
    if ( $check_level_a[''.$page.''] == '1' ) {
        return "1";
    } else {
        return "0";
    }
} 

Heres how I use it in a backend page: ex. testimonials:

if(user_type("testimonials") == "0") {
    header("Location: notallowed.php");
    exit();
} 

Moved by Hitman (Forum Moderator) from Designing & Development: Please post in the correct forum next time, thanks.

[Dentafrice]

God, learn how to code. What the heck is this?

$check_level_a[''.$page.''] == '1' 

Your going to need to use a global I believe,

However, within user-defined functions a local function scope is introduced. Any variable used inside a function is by default limited to the local function scope.

<?php
function user_type ($page)
{
    global $logged_user;
    $level = mysql_query("SELECT * FROM users WHERE user='$user'"); // QUIT USING WILDCARDS =\
    $level_a = mysql_fetch_array($level); // You took my structure :@
    $final_level = $level_a["usergroup"];
    $check_level = mysql_query("SELECT * FROM userstructure WHERE name='$final_level'");
    $check_level_a = mysql_fetch_array($check_level);
    if ($check_level_a[$page] == "1") {
        return "1";
    } else {
        return "0";
    }
}

$user_type = user_type("testimonials");
if($user_type == "0") {
    header("Location: notallowed.php"); // This is stupid, it relies on the browser to redirect, plus it ***** up with sessions.
    exit();
}
?>

[Hypertext]

I realised the syntax in ''$var''

the global worked. thanks =)

[Dentafrice]

Ummm.. what?

[Agnostic Bear]

$level = mysql_query("SELECT * FROM users WHERE user='$user'"); // QUIT USING WILDCARDS =\

wildcards are fine unless you're pulling out a stupid amount of data which I highly doubt.

$level_a = mysql_fetch_array($level); // You took my structure :@

what structure

header("Location: notallowed.php"); // This is stupid, it relies on the browser to redirect, plus it ***** up with sessions.

All redirects rely on the browser. And it doesn't mess up with sessions.

[Dentafrice]

If the stupid sessions goes bazackwhap, it throws out a nice little error about sending **** before the sessions headers already sent all that meatwhack.

I showed him my source code to something, and now his _a, _n, looks like mine.

[Hypertext]

Meh, I'll throw out an error, it exit();'s anyway tbh.

[Bojangles]

If the stupid sessions goes bazackwhap, it throws out a nice little error about sending **** before the sessions headers already sent all that meatwhack.

I showed him my source code to something, and now his _a, _n, looks like mine.

Some people never change I guess:rolleyes:

[Hypertext]

lol, it's better than what I used to have:

$query
$result
$row
$querya
$resulta
$row
$queryasox
$resultex
$rowarc

I changed the language almost.

[Bojangles]

Lmao why did you use $rowarc o.0