Skip to content
Snippets Groups Projects
Commit e215e6fa authored by Peter Rotich's avatar Peter Rotich
Browse files

Use base form API instead of DynamicFormEntry form

Include all validation erors
parent 18bc1ebc
No related branches found
No related tags found
No related merge requests found
...@@ -66,20 +66,23 @@ class UsersAjaxAPI extends AjaxController { ...@@ -66,20 +66,23 @@ class UsersAjaxAPI extends AjaxController {
function addUser() { function addUser() {
$errors = $info = array(); $valid = true;
$user = null; $form = UserForm::getUserForm()->getForm($_POST);
if (!$form->isValid())
$form = UserForm::getInstance(); $valid = false;
if ($form->isValid()) {
if (($f=$form->getField('email')) if (($field=$form->getField('email'))
&& User::lookup(array('emails__address'=>$f->getClean()))) && $field->getClean()
$f->addError('Email is assigned to another user'); && User::lookup(array('emails__address'=>$field->getClean()))) {
elseif (($user = User::fromForm($form->getClean()))) $field->addError('Email is assigned to another user');
Http::response(201, $user->to_json()); $valid = false;
} }
if (!$info || !$info['error']) if ($valid && ($user = User::fromForm($form->getClean())))
$info += array('error' =>'Error adding user - try again!'); Http::response(201, $user->to_json());
$info = array('error' =>'Error adding user - try again!');
return self::_lookupform($form, $info); return self::_lookupform($form, $info);
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment