bugphpGroupWare - Bugs: bug #9443, accounts - return_members

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #9443: accounts - return_members

Submitter:  Dirk Schaller <dschaller>
Submitted:  Thu 24 Jun 2004 11:15:54 AM UTC
   
 
Category:  API - phpGroupware API Item Group:  0.9.16.000 release
Severity:  1 - Wish Priority:  7 - High
Status:  None Privacy:  Public
Assigned to:  ceb Open/Closed:  Closed
Component Version:  SVN Operating System:  Other
Reproducibility:  None Planned Release:  None
Fixed Release: 

Thu 24 Jun 2004 01:13:29 PM UTC, comment #1: 

fixed. thanks

Bettina Gille <ceb>
Group administrator
Thu 24 Jun 2004 11:15:54 AM UTC, 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;
}

Dirk Schaller <dschaller>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

CC list is empty

 

There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

 

Follows 1 latest change.

Date Changed by Updated Field Previous Value => Replaced by
2004-06-24 ceb Open/ClosedOpen Closed

Back to the top

Powered by Savane 3.13-4b48.
Corresponding source code