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

[Bettina Gille]

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

/**************************************************************************/
[bugs #9443] Latest Modifications:

Changes by: 
                Bettina Gille <address@hidden>
'Date: 
                Thu 06/24/2004 at 13:13 (GMT)

            What     | Removed                   | Added
---------------------------------------------------------------------------
              Status | Open                      | Closed


------------------ Additional Follow-up Comments ----------------------------
fixed. thanks






/**************************************************************************/
[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/2004 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:  Closed
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;
}

Follow-up Comments
------------------


-------------------------------------------------------
Date: Thu 06/24/2004 at 13:13       By: ceb
fixed. thanks












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/