Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 0.4.0

Bug #2994 Bug in getGroup in MDB_Complex when you try get subGroup
Submitted: 2004-12-20 17:41 UTC
From: fabio at bs2 dot com dot br Assigned:
Status: Wont fix Package: LiveUser_Admin
PHP Version: Irrelevant OS:
Roadmaps: (Not assigned)    
Subscription  
Comments Add Comment Add patch


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 40 - 18 = ?

 
 [2004-12-20 17:41 UTC] fabio at bs2 dot com dot br
Description: ------------ Bug in getGroup in MDB_Complex when you try get subGroup Notice when you set the second param with false; it returns array() when you set the second param with true; Reproduce code: --------------- $groupFilter = array('where_group_id' => {some subGroup}); $luAdmin->perm->getGroups($groupFilter); // undefined index $luAdmin->perm->getGroups($groupFilter ,true); // return array(); Actual result: -------------- Notice Undefined index: 5 403 : usermanager/liveuser/admin/perm/container/mdb_complex.php 403 : usermanager/liveuser/admin/perm/container/mdb_complex.php getgroups 59 : usermanager/user_list.action.php liveuser_admin_perm_container_mdb_complex->getgroups 27 : usermanager/_index.php require_once

Comments

 [2004-12-20 18:10 UTC] smith at backendmedia dot com
We know this is broken and stated this in the changelog ("the complex container is currently not fully implemented"). However as you can see on our roadmap (http://pear.limbourg.com/27.html) we plan to fix this in the next release. We are sorry for the inconvinience. Please also submit LiveUser_Admin bugs in the LiveUser_Admin category in the future. Anyways we do appreciate the report. We are well aware that we are flipping things upside down and that this will confuse users a bit. So we thank you for trying this stuff out!
 [2004-12-20 19:04 UTC] smith at backendmedia dot com
Helgi just told me the obvious thing I missed in this report. This report was still for the old 0.13.x release. Most of what I said still applies though, however please note that the admin API has drastically changed since the last release (BC breaks everywhere) and that the get* methods in the complex container are not yet implemented. So I will mark this bug report as wont fix since the old methods you were using will no longer exist in the future version of Liveuser (or LiveUser_Admin to be more exact). However like I said we will provide similar functionality eventually.