moving to a controller folder some logic that were in the routes for …

…users

controllers/userController.js

@@ -0,0 +1,43 @@
+const User = require('../models/user');
+
+module.exports.showUserSignUp = (req, res) => {
+    res.render('users/signup');
+}
+
+module.exports.userSignUp = async (req, res) => {
+    try {
+        const { email, username, password } = req.body;
+        const newUser = await new User({ email, username });
+        const registeredUser = await User.register(newUser, password);
+        req.login(registeredUser, err => {
+            if (err) {
+                return next(err);
+            }
+            req.flash('success', 'Successfully signed up');
+            res.redirect('/');
+        });
+    } catch (error) {
+        req.flash('error', error.message);
+        res.redirect('signup');
+    }
+}
+
+module.exports.showSignIn = (req, res) => {
+    res.render('users/signin')
+}
+
+module.exports.userSignIn = (req, res) => {
+    const redirectUrl = res.locals.returnTo ? res.locals.returnTo : '/';
+    req.flash('success', 'Signed in');
+    res.redirect(redirectUrl);
+}
+
+module.exports.userSignOut = (req, res, next) => {
+    req.logout(function (err) {
+        if (err) {
+            return next(err);
+        }
+        req.flash('success', 'Successfully signed out');
+        res.redirect('/');
+    });
+}

routes/userRoutes.js

@@ -1,52 +1,21 @@
 const express = require('express');
 const router = express.Router();
+const userController = require('../controllers/userController');
 const User = require('../models/user');
 const catchAsync = require('../utils/catchAsync');
 const passport = require('passport');
 const { storeReturnTo } = require('../middleware');
 
-router.get('/signup', (req, res) => {
-    res.render('users/signup');
-});
+router.get('/signup', userController.showUserSignUp);
 
-router.post('/signup', catchAsync(async (req, res) => {
-    try {
-        const { email, username, password } = req.body;
-        const newUser = await new User({ email, username });
-        const registeredUser = await User.register(newUser, password);
-        req.login(registeredUser, err => {
-            if (err) {
-                return next(err);
-            }
-            req.flash('success', 'Successfully signed up');
-            res.redirect('/');
-        });
-    } catch (error) {
-        req.flash('error', error.message);
-        res.redirect('signup');
-    }
-}));
+router.post('/signup', catchAsync(userController.userSignUp));
 
-router.get('/signin', (req, res) => {
-    res.render('users/signin')
-});
+router.get('/signin', userController.showSignIn);
 
 router.post('/signin', storeReturnTo, passport.authenticate('local', { failureFlash: true, failureRedirect: '/signin' })
-    , (req, res) => {
-        const redirectUrl = res.locals.returnTo ? res.locals.returnTo : '/';
-        req.flash('success', 'Signed in');
-        res.redirect(redirectUrl);
-    });
+    , userController.userSignIn);
 
-router.get('/signout', (req, res, next) => {
-    req.logout(function (err) {
-        if (err) {
-            return next(err);
-        }
-        req.flash('success', 'Successfully signed out');
-        res.redirect('/');
-    });
-});
+router.get('/signout', userController.userSignOut);
 
 
 module.exports = router;