Merge pull request #2105 from djpadz/userbug

Flashing an error if a nonexistent user is added to a project.
This commit is contained in:
Frédéric Guillot 2016-04-09 15:43:06 -04:00
commit 42813d702d
1 changed files with 3 additions and 1 deletions

View File

@ -83,7 +83,9 @@ class ProjectPermission extends Base
$project = $this->getProject();
$values = $this->request->getValues();
if ($this->projectUserRole->addUser($values['project_id'], $values['user_id'], $values['role'])) {
if (empty($values['user_id'])) {
$this->flash->failure(t('User not found.'));
} elseif ($this->projectUserRole->addUser($values['project_id'], $values['user_id'], $values['role'])) {
$this->flash->success(t('Project updated successfully.'));
} else {
$this->flash->failure(t('Unable to update this project.'));