diff --git a/app.js b/app.js index 75ecd22..77420d7 100644 --- a/app.js +++ b/app.js @@ -27,7 +27,7 @@ app.use(session({ secret: 'secret', resave: false, saveUninitialized: true -})) +})); app.use(express.static(path.join(__dirname, 'public'))); // res.locals is an object passed to hbs engine @@ -39,14 +39,14 @@ app.use(function(req, res, next) { app.use('/', index); // catch 404 and forward to error handler -app.use(function(req, res, next) { +app.use(function(_req, _res, next) { var err = new Error('Not Found'); err.status = 404; next(err); }); // error handler -app.use(function(err, req, res, next) { +app.use(function(err, req, res, _next) { // set locals, only providing error in development res.locals.message = err.message; res.locals.error = req.app.get('env') === 'development' ? err : {}; diff --git a/public/stylesheets/style.css b/public/stylesheets/style.css index 69bc44c..1f0365f 100644 --- a/public/stylesheets/style.css +++ b/public/stylesheets/style.css @@ -16,10 +16,12 @@ a { .dropdown-menu .cart { min-width: 250px; padding: 15px 25px 0; + margin: 0; } .dropdown-menu .cart .row > div { padding: 0 0 10px 0; + margin: 0; } .dropdown-menu .cart .totalPrice{ diff --git a/routes/index.js b/routes/index.js index 850c948..8bd5d98 100644 --- a/routes/index.js +++ b/routes/index.js @@ -6,9 +6,7 @@ var fs = require('fs'); var Cart = require('../models/cart'); var products = JSON.parse(fs.readFileSync('./data/products.json', 'utf8')); -router.get('/', function (req, res, next) { - var productId = products && products[0].id; - +router.get('/', function (_req, res, _next) { res.render('index', { title: 'NodeJS Shopping Cart', @@ -17,12 +15,12 @@ router.get('/', function (req, res, next) { ); }); -router.get('/add/:id', function(req, res, next) { +router.get('/add/:id', function(req, res, _next) { var productId = req.params.id; var cart = new Cart(req.session.cart ? req.session.cart : {}); var product = products.filter(function(item) { - return item.id == productId; + return item.id === productId; }); cart.add(product[0], productId); req.session.cart = cart; @@ -30,7 +28,7 @@ router.get('/add/:id', function(req, res, next) { inline(); }); -router.get('/cart', function(req, res, next) { +router.get('/cart', function(req, res, _next) { if (!req.session.cart) { return res.render('cart', { products: null @@ -44,7 +42,7 @@ router.get('/cart', function(req, res, next) { }); }); -router.get('/remove/:id', function(req, res, next) { +router.get('/remove/:id', function(req, res, _next) { var productId = req.params.id; var cart = new Cart(req.session.cart ? req.session.cart : {});