phpgroupware-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Phpgroupware-tracker] [bugs #9443] accounts - return_members


From: Dirk Schaller
Subject: [Phpgroupware-tracker] [bugs #9443] accounts - return_members
Date: Thu, 24 Jun 2004 07:16:03 -0400
User-agent: Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1; .NET CLR 1.1.4322)

This mail is an automated notification from the bugs tracker
 of the project: phpGroupWare.




/**************************************************************************/
[bugs #9443] Full Item Snapshot:

URL: <http://savannah.gnu.org/bugs/?func=detailitem&item_id=9443>
Project: phpGroupWare
Submitted by: Dirk Schaller
On: Thu 06/24/04 at 11:15

Category:  API - phpGWapi
Item Group:  0.9.16.000 release
Severity:  1 - None
Priority:  7 - High
Resolution:  None
Assigned to:  ceb
Status:  Open
Component Version:  CVS
Platform Version:  Other
Reproducibility:  None


Summary:  accounts - return_members

Original Submission:  class: accounts
method: return_members
line: 350

Every return value of the get_ids_for_location() call will be added to the 
$members array. When the return value is false, the array gets a new entry with 
faƶse as value. That makes problems in some account list, when the developer 
doesnt check for right account id's (in this cases the account id is false (0) 
and the list contains an empty [] or the current logged in user are twotimes in 
a list.

Solution:

instead of:
$members[] = 
$GLOBALS['phpgw']->acl->get_ids_for_location($app_users[$i],1,'phpgw_group');

use:
$mem = 
$GLOBALS['phpgw']->acl->get_ids_for_location($app_users[$i],1,'phpgw_group');
if(is_array($mem))
{
        $members[] = $mem;
}











For detailed info, follow this link:
<http://savannah.gnu.org/bugs/?func=detailitem&item_id=9443>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/







reply via email to

[Prev in Thread] Current Thread [Next in Thread]