#123 adds case to route for when user already registered

db_build.sql

@@ -7,7 +7,7 @@ CREATE TABLE users (
     firstname   TEXT       NOT NULL,
     lastname    TEXT       NOT NULL,
     github      TEXT       NOT NULL,
-    email       TEXT       NOT NULL,
+    email       TEXT       UNIQUE,
     username    TEXT       UNIQUE,
     password    TEXT       NOT NULL
 );

src/queries.js

@@ -37,7 +37,9 @@ queries.registerUser = (payload, cb) => {
                              VALUES ($1, $2, $3, $4, $5, $6) RETURNING id, username`;
       const values = [payload.firstname, payload.lastname, payload.github, payload.email, payload.username, hash];
       dbConn.query(sql, values, (err, data) => {
-        if (err) cb(err);
+        if (err) {
+          cb(err);
+        }
         else {
           const userinfo = data.rows[0];
           cb(null, userinfo);

src/routes.js

@@ -63,9 +63,13 @@ const registerSubmit = {
       reply.view('register', { password: true });
     } else {
       queries.registerUser(req.payload, (err, userinfo) => {
-        if (err) console.log(err);
-        req.cookieAuth.set({username: userinfo.username, userid: userinfo.id});
-        reply.redirect('/');
+        if (err) {
+          console.log('register error');
+          reply.view('register', { alreadyReg: true })
+        } else {
+          req.cookieAuth.set({username: userinfo.username, userid: userinfo.id});
+          reply.redirect('/');
+        }
       });
     }
   }

views/register.hbs

@@ -22,3 +22,7 @@
 {{# if password}}
 <p style="color:red">Passwords do not match</p>
 {{/if}}
+
+{{# if alreadyReg}}
+<p style="color:red">User already registered</p>
+{{/if}}