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

Request #7025 BC breakage request - wrong defaults location
Submitted: 2006-03-05 07:40 UTC Modified: 2006-04-11 09:50 UTC
From: prahal at yahoo dot com Assigned: lsmith
Status: Closed Package: LiveUser_Admin (version 0.3.7)
PHP Version: 5.1.2 OS: all
Roadmaps: (Not assigned)    
Subscription  


 [2006-03-05 07:40 UTC] prahal at yahoo dot com (Alban Browaeys)
Description: ------------ in Admin.php , getUsers has itscontainer parameter set to 'perm' default. Though it is the first parameter and the second have to be defined, this "default" definition is of no use. The same problem arise for Admin->perm->updateUser, Admin->auth->updateUser, etc. Well I find it a bit weird that you did not though about this thus feel free to close the bug if it was the wanted behaviour. Alban

Comments

 [2006-03-05 07:45 UTC] lsmith
I do not exactly understand what you mean .. function getUsers($container = 'perm', $param = array()) both parameters are optional ... In the auth and perm container there are no optional parameters in the updateUser() method .. function updateUser($data, $filters)
 [2006-03-07 08:23 UTC] lsmith
I am still not sure if I understand your argument. Thinking about things a bit more I have began wondering if the LiveUser_Admin::getUser() API should not be changed to move the first parameter into the second parameter array similar to what we have done for LiveUser_Admin_Perm_Medium::getRights() with the 'by_group' setting.
 [2006-04-11 09:50 UTC] lsmith (Lukas Smith)
This bug has been fixed in CVS. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better. I have changed the getUsers(), addUser() and updateUser() API to be more in line with the container APIs