http://pear.php.net/bugs/7975 [Closed] update of auth_user_id not possible in liveuser_perm_users table en-us pear-webmaster@lists.php.net pear-webmaster@lists.php.net hourly 1 2000-01-01T12:00+00:00 http://pear.php.net/bugs/7975 LiveUser_Admin Bug Reported by mahono 2006-06-23T17:12:24-00:00 PHP: Irrelevant OS: Package Version: CVS Description: ------------ Unfortunately it is not possible to update the auth_user_id using $admin->updateUser($data, $perm_user_id) Only the perm_type is updated. Once I change an auth_user_id, the user disappears because the join/mapping fails. Test script: --------------- I suggest changing the end of updateUser() after ($result===true) check this way (tested and works as expected): if (is_null($type) && !array_key_exists('auth_user_id', $data)) { return true; } $updateData = array(); if ($type) { $updateData['perm_type'] = $type; } if (array_key_exists('auth_user_id', $data) && $permData['auth_user_id'] != $data['auth_user_id'] ) { $updateData['auth_user_id'] = $data['auth_user_id']; } $filters = array('perm_user_id' => $permUserId); return $this->perm->updateUser($updateData, $filters);]]> LiveUser_Admin Bug Reported by mahono 2006-06-23T17:12:24-00:00 PHP: Irrelevant OS: Package Version: CVS Description: ------------ Unfortunately it is not possible to update the auth_user_id using $admin->updateUser($data, $perm_user_id) Only the perm_type is updated. Once I change an auth_user_id, the user disappears because the join/mapping fails. Test script: --------------- I suggest changing the end of updateUser() after ($result===true) check this way (tested and works as expected): if (is_null($type) && !array_key_exists('auth_user_id', $data)) { return true; } $updateData = array(); if ($type) { $updateData['perm_type'] = $type; } if (array_key_exists('auth_user_id', $data) && $permData['auth_user_id'] != $data['auth_user_id'] ) { $updateData['auth_user_id'] = $data['auth_user_id']; } $filters = array('perm_user_id' => $permUserId); return $this->perm->updateUser($updateData, $filters);]]> 2006-06-23T17:12:24-00:00 http://pear.php.net/bugs/7975#1154983205 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.]]> 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.]]> 2006-08-07T20:40:05-00:00 http://pear.php.net/bugs/7975#1151254336 ok, I think best will be to remove the if (is_null($type) && ...) block (first 4 lines in my code) and insert these lines before the line "$filters = array(...": if (count($updateData) == 0) { return true; }]]> ok, I think best will be to remove the if (is_null($type) && ...) block (first 4 lines in my code) and insert these lines before the line "$filters = array(...": if (count($updateData) == 0) { return true; }]]> 2006-06-25T16:52:16-00:00 http://pear.php.net/bugs/7975#1151253610 like I said: it works.. but the check (permData['auth_user_id'] != $data['auth_user_id']) should be moved to the "is_null($type) && ..." check so the data will really only get updated if auth_user_id changed to avoid the query if not needed. or simply do a count() check before updateUser() ..?]]> like I said: it works.. but the check (permData['auth_user_id'] != $data['auth_user_id']) should be moved to the "is_null($type) && ..." check so the data will really only get updated if auth_user_id changed to avoid the query if not needed. or simply do a count() check before updateUser() ..?]]> 2006-06-25T16:40:10-00:00