[RESOLVED] Class/array/return problem

[benbox]

Hey all!

I've got this problem and I can't understand what's going wrong..

I've got a class (simplified below).. the array is set up fine, print_r() works perfectly. echo $user_permissions[25]['auth_read'] proves this works too, returning the correct value..

However my function allowed($forum_id, $access_mode) never works.. always returns false.. and I can't understand why, when both should be doing EXACTLY the same thing..

I've also tried returning count($user_permissions) in the function allowed().. which returns 0, so I'm assuming that function isn't getting the array.. but why?

Does anyone have any ideas? or know why this might be happening?

Many thanks 😃

<?php
class PageAccess
{	

private $user_permissions;

public function __construct($required_page_level = 0) {	

	// COLLATE USER PERMISSIONS
	$user_permissions = array();

	// code which sets up array $user_permissions[id][mode] = 1 or 0

	echo $user_permissions[25]['auth_read'] . '<br>'; // returns the correct value
	echo $this->allowed(25, 'auth_read'); // returns false everytime

	echo '<p>';
	print_r($user_permissions);


}

public function allowed($forum_id, $access_mode) {
	global $user_permissions;	
	//return count($user_permissions);
                            return $user_permissions[$forum_id][$access_mode] ? 1 : 0;
}

} // end class
?>

[Installer]

You've got three different "$user_permissions" floating around there. For starters, try something more like this:

class PageAccess 
{     

    private $user_permissions = array(); 

public function __construct($required_page_level = 0) 
{     
    // code which sets up array $user_permissions[id][mode] = 1 or 0 
    echo $this->user_permissions[25]['auth_read'] . '<br>'; // returns the correct value 
    echo $this->allowed(25, 'auth_read'); // returns false everytime 

    echo '<p>'; 
    print_r($this->user_permissions); 


} 

public function allowed($forum_id, $access_mode) 
{ 
     return $this->user_permissions[$forum_id][$access_mode] ? 1 : 0; 
} 

} // end class 

Since this is "simplified" I won't try to guess at any other problems with it, although it looks like there may be some.

[benbox]

Ahh, of course! Thanks for pointing that out, I've been playing around with this for several hours and was getting nowhere.. and it was like you said having the 3 different $user_permission variables.. class and 2 different locals..

So like you've done, I've just set them all to the class ($this->) and it's now working!

I usually do that automatically without thinking, so it never occured to me to check that :X

Cheers! 😃

[Installer]

🙂