Fix for ticket #201.

1) create common update function so processing consistent between the
user edit and admin edit.
2) created common private helper function to build the user edit form
the same way.
So a user can now change their user name if the new one doesn't exist.
This commit is contained in:
Tim Almdal
2009-04-03 17:33:07 +00:00
parent d35a0cdb61
commit 9ed0dd51bf
3 changed files with 46 additions and 50 deletions

View File

@@ -26,21 +26,12 @@ class Users_Controller extends REST_Controller {
}
$form = user::get_edit_form($user);
$form->edit_user->password->rules("-required");
if ($form->validate()) {
// @todo: allow the user to change their name
$user->full_name = $form->edit_user->full_name->value;
if ($form->edit_user->password->value) {
$user->password = $form->edit_user->password->value;
}
$user->email = $form->edit_user->email->value;
$user->url = $form->edit_user->url->value;
if ($form->edit_user->locale) {
$desired_locale = $form->edit_user->locale->value;
$user->locale = $desired_locale == "none" ? null : $desired_locale;
}
$user->save();
$valid = $form->validate();
if ($valid) {
$valid = user::update($user, $form);
}
if ($valid) {
print json_encode(
array("result" => "success",
"resource" => url::site("users/{$user->id}")));