Renames controller functions

Author: andela-nmartin

server/controllers/documents.js

@@ -7,7 +7,7 @@
 
    module.exports = {
      // get document by id
-     getDocument: function(req, res) {
+     get: function(req, res) {
        var id = req.params.id;
        Document.find({
          _id: id
@@ -21,7 +21,7 @@
      },
 
      // to get the mongo cluster of all the documents stored
-     getAllDocuments: function(req, res) {
+     getAll: function(req, res) {
        Document.find({}, function(err, documents) {
          if (err) {
            res.send(err);
@@ -31,7 +31,7 @@
        });
      },
 
-     createDocument: function(req, res) {
+     create: function(req, res) {
        var document = new Document({
          ownerId: req.decoded._id,
          title: req.body.title,
@@ -47,7 +47,7 @@
      },
 
      // update document by id
-     updateDocument: function(req, res) {
+     update: function(req, res) {
        var id = req.params.id;
        Document.findById(req.params.id).exec(function(err, document) {
          if (err) {
@@ -95,7 +95,7 @@
      },
 
      // delete document by id
-     deleteDocument: function(req, res) {
+     delete: function(req, res) {
        Document.findById(req.params.id).exec(function(err, document) {
          if (err) {
            res.status(500).send({

server/controllers/roles.js

@@ -5,7 +5,7 @@
 
   module.exports = {
     // gets all the saved roles from the db
-    getRoles: function(req, res) {
+    get: function(req, res) {
       Role.find({}, function(err, roles) {
         if (err) {
           res.send(err);
@@ -16,7 +16,7 @@
     },
 
     // creates a role in the db
-    createRole: function(req, res) {
+    create: function(req, res) {
       var role = new Role({
         id: req.body.id,
         title: req.body.title

server/controllers/users.js

@@ -17,7 +17,7 @@
 
   module.exports = {
     // to add a user to the db
-    createUser: function(req, res) {
+    create: function(req, res) {
       var user = new User({
         name: {
           first: req.body.firstname,
@@ -96,7 +96,7 @@
     },
 
     // to get the mongo cluster of all the users stored on the db
-    getAllUsers: function(req, res) {
+    getAll: function(req, res) {
       User.find({}, function(err, users) {
         if (err) {
           res.send(err);
@@ -107,7 +107,7 @@
     },
 
     // get user by id
-    getUser: function(req, res) {
+    get: function(req, res) {
       var id = req.params.id;
       User.find({
         _id: id
@@ -147,7 +147,7 @@
     },
 
     // update user by id
-    updateUser: function(req, res) {
+    update: function(req, res) {
       var id = req.params.id;
       // update function
       var updateMe = function(id) {
@@ -200,7 +200,7 @@
     },
 
     // delete user by id
-    deleteUser: function(req, res) {
+    delete: function(req, res) {
       // delete function
       var deleteMe = function(id) {
         User.findOneAndRemove({

server/routes/index.js

@@ -5,27 +5,27 @@ var DocCtrl = require('../controllers/documents'),
 
 module.exports = function(app, express) {
   var api = express.Router();
-  api.post('/users', UserCtrl.createUser);
-  api.get('/users', UserCtrl.getAllUsers);
-  api.get('/users/roles', RolesCtrl.getRoles);
-  api.post('/users/roles', RolesCtrl.createRole);
+  api.post('/users', UserCtrl.create);
+  api.get('/users', UserCtrl.getAll);
+  api.get('/users/roles', RolesCtrl.get);
+  api.post('/users/roles', RolesCtrl.create);
   api.post('/users/login', UserCtrl.login);
   // middleware
   api.use(auth.authenticate);
   // routes that need checking for a legitimate token
-  api.get('/documents', DocCtrl.getAllDocuments);
+  api.get('/documents', DocCtrl.getAll);
   api.get('/documents/user', DocCtrl.getAllDocumentsByRoleUser);
   api.get('/documents/admin', DocCtrl.getAllDocumentsByRoleAdministrator);
   api.get('/documents/date', DocCtrl.getAllDocumentsByDate);
   api.get('/users/logout', UserCtrl.logout);
-  api.post('/documents', DocCtrl.createDocument);
+  api.post('/documents', DocCtrl.create);
   api.get('/users/:id/documents', DocCtrl.getAllDocumentsParticularUser);
-  api.get('/users/:id', UserCtrl.getUser);
-  api.put('/documents/:id', DocCtrl.updateDocument);
-  api.put('/users/:id', UserCtrl.updateUser);
-  api.delete('/users/:id', UserCtrl.deleteUser);
-  api.get('/documents/:id', DocCtrl.getDocument);
-  api.delete('/documents/:id', DocCtrl.deleteDocument);
+  api.get('/users/:id', UserCtrl.get);
+  api.put('/documents/:id', DocCtrl.update);
+  api.put('/users/:id', UserCtrl.update);
+  api.delete('/users/:id', UserCtrl.delete);
+  api.get('/documents/:id', DocCtrl.get);
+  api.delete('/documents/:id', DocCtrl.delete);
   api.get('/me', function(req, res) {
     res.send(req.decoded);
   });