Add pages to change your user name and email.

4 files changed, 91 insertions, 8 deletions

diff --git a/server.js b/server.js
index 710fec4..6d036e7 100644
--- a/server.js
+++ b/server.js
@@ -328,6 +328,16 @@ app.get('/change_password', must_be_logged_in, function (req, res) {
 	res.render('change_password.ejs', { user: req.user, message: req.flash('message') });
 });
 
+app.get('/change_name', must_be_logged_in, function (req, res) {
+	LOG(req, "GET /change_name");
+	res.render('change_name.ejs', { user: req.user, message: req.flash('message') });
+});
+
+app.get('/change_mail', must_be_logged_in, function (req, res) {
+	LOG(req, "GET /change_mail");
+	res.render('change_mail.ejs', { user: req.user, message: req.flash('message') });
+});
+
 app.get('/subscribe', must_be_logged_in, function (req, res) {
 	LOG(req, "GET /subscribe");
 	sql_subscribe.run(req.user.user_id);
@@ -478,6 +488,56 @@ app.post('/change_password', must_be_logged_in, function (req, res) {
 	}
 });
 
+const sql_is_name_taken = db.prepare("SELECT EXISTS ( SELECT 1 FROM users WHERE name = ? )").pluck();
+const sql_change_name = db.prepare("UPDATE users SET name = ? WHERE user_id = ?");
+
+const sql_is_mail_taken = db.prepare("SELECT EXISTS ( SELECT 1 FROM users WHERE mail = ? )").pluck();
+const sql_change_mail = db.prepare("UPDATE users SET mail = ? WHERE user_id = ?");
+
+app.post('/change_name', must_be_logged_in, function (req, res) {
+	try {
+		let newname = clean_user_name(req.body.newname);
+		LOG(req, "POST /change_name", req.user, req.body, newname);
+		if (!is_valid_user_name(newname)) {
+			req.flash('message', "Invalid user name!");
+			return res.redirect('/change_name');
+		}
+		if (sql_is_name_taken.get(newname)) {
+			req.flash('message', "That name is already taken!");
+			return res.redirect('/change_name');
+		}
+		sql_change_name.run(newname, req.user.user_id);
+		req.flash('message', "Your name has been changed.");
+		return res.redirect('/profile');
+	} catch (err) {
+		console.log(err);
+		req.flash('message', err.message);
+		return res.redirect('/change_name');
+	}
+});
+
+app.post('/change_mail', must_be_logged_in, function (req, res) {
+	try {
+		let newmail = req.body.newmail;
+		LOG(req, "POST /change_mail", req.user, req.body);
+		if (!is_email(newmail)) {
+			req.flash('message', "Invalid mail address!");
+			return res.redirect('/change_mail');
+		}
+		if (sql_is_mail_taken.get(newmail)) {
+			req.flash('message', "That mail address is already taken!");
+			return res.redirect('/change_mail');
+		}
+		sql_change_mail.run(newmail, req.user.user_id);
+		req.flash('message', "Your mail address has been changed.");
+		return res.redirect('/profile');
+	} catch (err) {
+		console.log(err);
+		req.flash('message', err.message);
+		return res.redirect('/change_mail');
+	}
+});
+
 /*
  * GAME LOBBY
  */
diff --git a/views/change_mail.ejs b/views/change_mail.ejs
new file mode 100644
index 0000000..6c3ed62
--- /dev/null
+++ b/views/change_mail.ejs
@@ -0,0 +1,12 @@
+<%- include('header', { title: "Change mail address" }) %>
+<form action="/change_mail" method="post">
+<p>
+Name: <%= user.name %>
+<p>
+Mail: <%= user.mail %>
+<p>
+<label for="newmail">New mail address: </label><br>
+<input type="newmail" id="newmail" name="newmail" required>
+<p>
+<button type="submit">Change mail</button>
+</form>
diff --git a/views/change_name.ejs b/views/change_name.ejs
new file mode 100644
index 0000000..eac508f
--- /dev/null
+++ b/views/change_name.ejs
@@ -0,0 +1,12 @@
+<%- include('header', { title: "Change name" }) %>
+<form action="/change_name" method="post">
+<p>
+Name: <%= user.name %>
+<p>
+Mail: <%= user.mail %>
+<p>
+<label for="newname">New name: </label><br>
+<input type="newname" id="newname" name="newname" required>
+<p>
+<button type="submit">Change name</button>
+</form>
diff --git a/views/profile.ejs b/views/profile.ejs
index 7829870..74843f3 100644
--- a/views/profile.ejs
+++ b/views/profile.ejs
@@ -12,20 +12,19 @@ Your mail address is <%= user.mail %>.
 
 <br clear=left>
 
-<p>
-<%
+<ul style="list-style:'\bb  '">
+<li><%
 	if (user.notifications) {
 		%><a href="/unsubscribe">Disable mail notifications</a><%
 	} else {
 		%><a href="/subscribe">Enable mail notifications</a><%
 	}
 %>
-
-<p>
-<a href="/change_password">Change password</a>
-
-<p>
-<a href="/logout">Logout</a>
+<li><a href="/change_password">Change password</a>
+<li><a href="/change_mail">Change mail address</a>
+<li><a href="/change_name">Change name</a>
+<li><a href="/logout">Logout</a>
+</ul>
 
 <% if (open_games.length > 0) { %>
 <h2>Open Games</h2>