diff options
| author | Simon Rettberg | 2019-09-09 12:13:57 +0200 |
|---|---|---|
| committer | Simon Rettberg | 2019-09-09 12:13:57 +0200 |
| commit | 18269245e7cb18c69f9743b49a6aad79ffa58b68 (patch) | |
| tree | 5b1d9f268c74d7cfb5ee33ba26d02c08bc0cc200 /modules-available/adduser/page.inc.php | |
| parent | [dnbd3] proxy view: Show cache-map when clicking percentage of image (diff) | |
| download | slx-admin-18269245e7cb18c69f9743b49a6aad79ffa58b68.tar.gz slx-admin-18269245e7cb18c69f9743b49a6aad79ffa58b68.tar.xz slx-admin-18269245e7cb18c69f9743b49a6aad79ffa58b68.zip | |
[adduser] Check if adding user succeeded
Fixes #3573
Diffstat (limited to 'modules-available/adduser/page.inc.php')
| -rw-r--r-- | modules-available/adduser/page.inc.php | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/modules-available/adduser/page.inc.php b/modules-available/adduser/page.inc.php index cffe33f9..c8acb554 100644 --- a/modules-available/adduser/page.inc.php +++ b/modules-available/adduser/page.inc.php @@ -47,7 +47,12 @@ class Page_AddUser extends Page 'phone' => $phone, 'email' => $email, ); - Database::exec('INSERT INTO user SET login = :login, passwd = :pass, fullname = :fullname, phone = :phone, email = :email', $data); + $ret = Database::exec('INSERT INTO user + SET login = :login, passwd = :pass, fullname = :fullname, phone = :phone, email = :email', $data, true); + if ($ret === false) { + Message::addError('user-already-exists', $login); + return; + } $id = Database::lastInsertId(); // Make it superadmin if first user. This method sucks as it's a race condition but hey... $ret = Database::queryFirst('SELECT Count(*) AS num FROM user'); |