summaryrefslogtreecommitdiffstats
path: root/modules-available/adduser/page.inc.php
diff options
context:
space:
mode:
Diffstat (limited to 'modules-available/adduser/page.inc.php')
-rw-r--r--modules-available/adduser/page.inc.php7
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');