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

Submitted by:  Dirk Schaller <dschaller>
Submitted on:  Thu 24 Jun 2004 11:15:54 AM UTC  
 
Item Group: 0.9.16.000 releaseCategory: API - phpGroupware API
Severity: 1 - WishPriority: 7 - High
Status: NonePrivacy: Public
Assigned to: Bettina Gille <ceb>Open/Closed: Closed
Component Version: SVNOperating System: Other
Reproducibility: NonePlanned Release: None
Fixed Release: 

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

fixed. thanks

Bettina Gille <ceb>
Project AdministratorIn charge of this item.
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>

 

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

CC list is empty

 

Do you think this task is very important?
If so, you can click here to add your encouragement to it.
This task has 0 encouragements so far.

Only logged-in users can vote.

 

Please enter the title of George Orwell's famous dystopian book (it's a date):

 

 

Follows 1 latest change.

Date Changed By Updated Field Previous Value => Replaced By
Thu 24 Jun 2004 01:13:29 PM UTCcebOpen/ClosedOpen=>Closed

Back to the top


Powered by Savane 3.1-cleanup1