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

Use base form API instead of DynamicFormEntry form

Include all validation erors
parent 8d442a61
Branches
Tags
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.
Please register or to comment