updated

API/room.route.js

@@ -8,12 +8,15 @@ let Player = require('./player.model');
 postRoutes.route('/add').get(function (req, res) {
     let r = new Room();
     let p = new Player();
+    p.colour = Math.trunc(Math.random()*8);
+    p.score = 0;
     r.members.push(p);
     r.save()
         .then(() => {
             let ret = {};
             ret._id = r._id;
             ret.playerId = p._id;
+            ret.colour = p.colour;
             res.send(ret);
             console.log("Created Room");
         })
@@ -31,39 +34,26 @@ postRoutes.route('/join').post(function (req, res) {
             res.json(err);
             console.log("Error Joining Room");
         } else {
-            let p = new Player();
-            r.members.push(p);
-            r.save().then(() => {
-                res.send(p);
-                console.log("Joined Room");
-                console.log(r.members);
-            }).catch(() => {
-                console.log("Unable to save to db");
-                res.status(400).send("Unable to save to the database")
-            })
+            if (r) {
+                let p = new Player();
+                p.colour = Math.trunc(Math.random()*8);
+                p.score = 0;
+                r.members.push(p);
+                r.save().then(() => {
+                    res.send(p);
+                    console.log(p);
+                    console.log("Joined Room");
+                    console.log(r.members);
+                }).catch(() => {
+                    console.log("Unable to save to db");
+                    res.status(400).send("Unable to save to the database")
+                })
+            } else {
+                res.send(null);
+            }
         }
     })
 
 });
 
-// Defined get data(index or listing) route
-postRoutes.route('/').get(function (req, res) {
-    Post.find(function (err, posts) {
-        if (err) {
-            res.json(err);
-        }
-        else {
-            res.json(posts);
-        }
-    });
-});
-
-// Defined delete | remove | destroy route
-postRoutes.route('/delete/:id').delete(function (req, res) {
-    Post.findByIdAndRemove({ _id: req.params.id }, function (err) {
-        if (err) res.json(err);
-        else res.json('Successfully removed');
-    });
-});
-
 module.exports = postRoutes;