Server bugs

Author: renz45

server/models/category.js

@@ -1,14 +1,14 @@
 var _ = require('lodash');
 
 var categories = [
-  {"id": 1, "name": "Testing", "icon": "tumblr"},
-  {"id": 2, "name": "Personal Note", "icon": "pencil"},
-  {"id": 3, "name": "Bash", "icon": "terminal"},
-  {"id": 4, "name": "Idea", "icon": "lightbulb"},
-  {"id": 5, "name": "Use with Caution","icon": "warning"},
-  {"id": 6, "name": "Question", "icon": "question"},
-  {"id": 7, "name": "Best Practice","icon": "thumbs up outline"},
-  {"id": 8, "name": "Code Snippet", "icon": "code"}
+  {"id": 1, "name": "Testing"},
+  {"id": 2, "name": "Personal Note"},
+  {"id": 3, "name": "Bash"},
+  {"id": 4, "name": "Idea"},
+  {"id": 5, "name": "Use with Caution"},
+  {"id": 6, "name": "Question"},
+  {"id": 7, "name": "Best Practice"},
+  {"id": 8, "name": "Code Snippet"}
 ]
 
 module.exports = {

server/models/note.js

@@ -2,12 +2,12 @@ var _ = require('lodash');
 var Category = require('./category')
 var User = require('./user')
 var notes = [
-  {"id":1 ,"userId": 13, "categoryId": 8, "link" :"https://www.youtube.com/watch?feature=player_detailpage&v=ZhfUv0spHCY#t=1870", "description": "NgModel Best Practice", "content" : "Always use dot syntax when using NgModel! Treat Scope as read-only in templates & write-only in controllers. The purpose of the scope is to refer to the model, not be the model. The model is your javascript objects. When doing bidirectional binding with ngModel make sure you don't bind directly to the scope properties. This will cause unexpected behavior in the child scopes.", "title" : "NgModel BP", "icon" : "basic info"},
-  {"id":2 ,"userId": 2, "categoryId": 3, "link" : "https://docs.angularjs.org/api/ng#directive", "description" : "Markers on a **DOM element** that tell AngularJS's HTML compiler `$compile` to attach a specified behavior to that DOM element.", "title" : "Directives", "icon" : "code", "content": "Markers on a **DOM element**"},
-  {"id":3 ,"userId": 1, "categoryId": 6, "link" : "", "description" : "Clarify the confusion between Service the term and `service` the angular method and to explain the 5 different Service recipes in Angular.", "title" : "Service Service? Really Angular?","content": "There are 5 Recipes used to create a Service. One of those *was* unfortunately named, Service. So yes, amongst its fellow peers such as Provider Service and Factory Service, there is in fact a Service Service.", "icon" : "question"},
-  {"id":4 ,"userId": 2, "categoryId": 6, "link" : "", "description" : "QUESTIONABLE DESCRIPTION GOES HERE", "title" : "TEST TEST TEST", "content": "QUESTIONABLE CONTENT GOES HERE", "icon" : "question"},
-  {"id":5 ,"userId": 4, "categoryId": 6, "link" : "", "description" : "Define Service", "title" : "What is a Service", "content": "Service: Angular services are objects that are wired together using dependency injection (DI). You can use services to organize and share code across your app.", "icon" : "question"},
-  {"id":6 ,"userId": 5, "categoryId": 6, "description" : "Steps for Creating a Service", "title" : "How do you create a Service?", "content": "You can register a service to our Angular module `app` with a one of the following 5 recipes: \\n 	- **factory**  \\n 	- **provider**  \\n 	- **service**  \\n 	- **value**  \\n 	- **constant** ", "icon" : "question"}
+  {"id":1 ,"userId": 13, "categoryId": 8, "description": "NgModel Best Practice", "content" : "Always use dot syntax when using NgModel! Treat Scope as read-only in templates & write-only in controllers. The purpose of the scope is to refer to the model, not be the model. The model is your javascript objects. When doing bidirectional binding with ngModel make sure you don't bind directly to the scope properties. This will cause unexpected behavior in the child scopes.", "title" : "NgModel BP"},
+  {"id":2 ,"userId": 2, "categoryId": 3, "description" : "Markers on a **DOM element** that tell AngularJS's HTML compiler `$compile` to attach a specified behavior to that DOM element.", "title" : "Directives", "content": "Markers on a **DOM element**"},
+  {"id":3 ,"userId": 1, "categoryId": 6, "description" : "Clarify the confusion between Service the term and `service` the angular method and to explain the 5 different Service recipes in Angular.", "title" : "Service Service? Really Angular?","content": "There are 5 Recipes used to create a Service. One of those *was* unfortunately named, Service. So yes, amongst its fellow peers such as Provider Service and Factory Service, there is in fact a Service Service."},
+  {"id":4 ,"userId": 2, "categoryId": 6, "description" : "QUESTIONABLE DESCRIPTION GOES HERE", "title" : "TEST TEST TEST", "content": "QUESTIONABLE CONTENT GOES HERE"},
+  {"id":5 ,"userId": 4, "categoryId": 6, "description" : "Define Service", "title" : "What is a Service", "content": "Service: Angular services are objects that are wired together using dependency injection (DI). You can use services to organize and share code across your app."},
+  {"id":6 ,"userId": 5, "categoryId": 6, "description" : "Steps for Creating a Service", "title" : "How do you create a Service?", "content": "You can register a service to our Angular module `app` with a one of the following 5 recipes: \\n 	- **factory**  \\n 	- **provider**  \\n 	- **service**  \\n 	- **value**  \\n 	- **constant** "}
 ]
 var lastId = 6;
 
@@ -51,7 +51,7 @@ module.exports = {
     for(var i=0, l=notes.length; i < l; i++) {
       if(notes[i].id === id){
         deletedNote = notes[i];
-        notes.splice(0, i);
+        notes.splice(i, 1);
         break;
       }
     }

server/models/user.js

@@ -1,26 +1,13 @@
-var categories = [
-  {"id": 1, "name": "Testing", "icon": "tumblr"},
-  {"id": 2, "name": "Personal Note", "icon": "pencil"},
-  {"id": 3, "name": "Bash", "icon": "terminal"},
-  {"id": 4, "name": "Idea", "icon": "lightbulb"},
-  {"id": 5, "name": "Use with Caution","icon": "warning"},
-  {"id": 6, "name": "Question", "icon": "question"},
-  {"id": 7, "name": "Best Practice","icon": "thumbs up outline"},
-  {"id": 8, "name": "Code Snippet", "icon": "code"}
-]
+var Category = require('../models/category');
 
 module.exports = function(app) {
   // Return a list of available node types
   app.get('/categories', function(req, res) {
-    res.json(categories);
+    res.json(Category.all());
   });
 
   app.get('/categories/:id', function(req, res) {
     var categoryId = parseInt(req.param('id'), 10);
-    var selectedCategory = _.find(categories, function(category){
-      return category.id === categoryId;
-    });
-
-    res.json(selectedCategory || {});
+    res.json(Category.get(categoryId) || {});
   });
 };

server/routes/category.js

@@ -7,11 +7,19 @@ module.exports = function(app) {
   });
 
   app.post('/notes', function(req, res) {
-    res.json(Note.create(req.body));
+    // Add a delay here to simulate the delay of a live server
+    // So things like button isSubmitting states can be demonstrated
+    setTimeout(function(){
+      res.json(Note.create(req.body));
+    }, 1000);
   });
 
-  app.put('/notes', function(req, res) {
-    res.json(Note.update(req.body));
+  app.put('/notes/:id', function(req, res) {
+    // Add a delay here to simulate the delay of a live server
+    // So things like button isSubmitting states can be demonstrated
+    setTimeout(function(){
+      res.json(Note.update(req.body));
+    },1000)
   });
 
   app.get('/notes/:id', function(req, res) {
@@ -20,7 +28,6 @@ module.exports = function(app) {
   });
 
   app.delete('/notes/:id', function(req, res) {
-    var noteId = parseInt(req.param('id'), 10);
-    res.json(Note.delete(noteId) || {});
+    res.json(Note.delete(parseInt(req.param('id'), 10)) || {});
   });
 };