Description: ------------ The method LiveUser_Admin_Perm_DB_Common::removeUserAreaAdmin() takes two arguments, $authId and $area_id. However, it is called from LiveUser_Admin_Perm_DB_Common::removeUser() with only the authId... // remove user area admin relation $result = $this->removeUserAreaAdmin($authId); LiveUser_Admin_Perm_DB_Common::removeUserAreaAdmin() should be changed to accept an area_id optionally, that way the removeUser() method doesn't have to call removeUserAreaAdmin() for each Area that User may be an admin of. Suggest changing prototype to.... function removeUserAreaAdmin($authId, $area_id = null) and changing $query to.... $query = ' DELETE FROM ' . $this->prefix . 'area_admin_areas WHERE perm_user_id=' . $permId .' '; if (is_numeric($area_id)) $query .= 'AND area_id=' . (int)$area_id;

Also remove check for area_id at top of method... change if (!is_numeric($authId) && !is_numeric($area_id)) to if (!is_numeric($authId))

This bug has been fixed in CVS. In case this was a documentation problem, the fix will show up at the end of next Sunday (CET) on pear.php.net. In case this was a pear.php.net website problem, the change will show up on the website in short time. Thank you for the report, and for helping us make PEAR better. Makes sense to have that behaviour.