holm/egroupware
1
1
Fork 0
forked from extern/egroupware
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fixed IDE warnings

Browse Source
This commit is contained in:
Ralf Becker 2014-03-28 07:49:49 +00:00
parent fe83c07b50
commit ca3f43838c
1 changed files with 15 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
phpgwapi/inc/class.acl.inc.php
View File

@ -18,7 +18,8 @@
* This can manage rights to 'run' applications, and limit certain features within an application. * This can manage rights to 'run' applications, and limit certain features within an application.
* It is also used for granting a user "membership" to a group, or making a user have the security equivilance of another user. * It is also used for granting a user "membership" to a group, or making a user have the security equivilance of another user.
* It is also used for granting a user or group rights to various records, such as todo or calendar items of another user. * It is also used for granting a user or group rights to various records, such as todo or calendar items of another user.
* $acl =& CreateObject('phpgwapi.acl',5); // 5 is the user id *
* $acl = new acl(5); // 5 is the user id
*/ */
class acl class acl
{ {
@ -129,13 +130,12 @@ class acl
) )
); );
return $xml_functions; return $xml_functions;
break;
case 'soap': case 'soap':
return $this->soap_functions; return $this->soap_functions;
break;
default: default:
return array(); return array();
break;
} }
} }
@ -300,7 +300,7 @@ class acl
return True; return True;
} }
$rights = 0; $rights = 0;
foreach($this->data as $idx => $value) foreach($this->data as $value)
{ {
if ($value['appname'] == $appname) if ($value['appname'] == $appname)
{ {
@ -350,7 +350,7 @@ class acl
} }
$rights = 0; $rights = 0;
foreach($this->data as $idx => $value) foreach($this->data as $value)
{ {
if ($value['appname'] == $appname && if ($value['appname'] == $appname &&
($value['location'] == $location || $value['location'] == 'everywhere') && ($value['location'] == $location || $value['location'] == 'everywhere') &&
@ -423,7 +423,7 @@ class acl
*/ */
function delete_repository($app, $location, $accountid='') function delete_repository($app, $location, $accountid='')
{ {
static $cache_accountid; static $cache_accountid = array();
$where = array( $where = array(
'acl_appname' => $app, 'acl_appname' => $app,
@ -531,9 +531,9 @@ class acl
*/ */
function get_app_list_for_id($location, $required, $accountid = '') function get_app_list_for_id($location, $required, $accountid = '')
{ {
static $cache_accountid; static $cache_accountid = array();
if($cache_accountid[$accountid]) if(isset($cache_accountid[$accountid]))
{ {
$account_id = $cache_accountid[$accountid]; $account_id = $cache_accountid[$accountid];
} }
@ -572,9 +572,9 @@ class acl
*/ */
function get_location_list_for_id($app, $required, $accountid = '') function get_location_list_for_id($app, $required, $accountid = '')
{ {
static $cache_accountid; static $cache_accountid = array();
if($cache_accountid[$accountid]) if(isset($cache_accountid[$accountid]))
{ {
$accountid = $cache_accountid[$accountid]; $accountid = $cache_accountid[$accountid];
} }
@ -655,9 +655,9 @@ class acl
*/ */
function get_user_applications($accountid = '', $use_memberships=true, $add_implicit_apps=true) function get_user_applications($accountid = '', $use_memberships=true, $add_implicit_apps=true)
{ {
static $cache_accountid; static $cache_accountid = array();
if($cache_accountid[$accountid]) if(isset($cache_accountid[$accountid]))
{ {
$account_id = $cache_accountid[$accountid]; $account_id = $cache_accountid[$accountid];
} }
@ -719,7 +719,6 @@ class acl
{ {
$grantor = $row['acl_account']; $grantor = $row['acl_account'];
$rights = $row['acl_rights']; $rights = $row['acl_rights'];
$granted_to = (int) $row['acl_location'];
if(!isset($grants[$grantor])) if(!isset($grants[$grantor]))
{ {