From acb0d56196536f50ebd354f50becf3fd796a92f0 Mon Sep 17 00:00:00 2001 From: DefNotArham Date: Tue, 23 Jun 2026 07:49:59 -0400 Subject: [PATCH 1/4] changed all vars to const --- models/Todo.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/models/Todo.js b/models/Todo.js index 7635d64..80da1db 100644 --- a/models/Todo.js +++ b/models/Todo.js @@ -1,9 +1,9 @@ -var mongoose = require('mongoose'); +const mongoose = require("mongoose"); -var TodoSchema = new mongoose.Schema({ +const TodoSchema = new mongoose.Schema({ name: String, completed: Boolean, - note: String + note: String, }); -module.exports = mongoose.model('Todo', TodoSchema); +module.exports = mongoose.model("Todo", TodoSchema); From 0b353efd70d57733ee0ba11e46886ae9a226e0af Mon Sep 17 00:00:00 2001 From: DefNotArham Date: Tue, 23 Jun 2026 07:50:13 -0400 Subject: [PATCH 2/4] . --- app.js | 60 ++++++++++++++++++++++++------------------------- bin/www | 40 +++++++++++++++------------------ routes/index.js | 8 +++---- routes/todos.js | 16 ++++++------- 4 files changed, 60 insertions(+), 64 deletions(-) diff --git a/app.js b/app.js index 4063552..a9fa2ef 100644 --- a/app.js +++ b/app.js @@ -1,46 +1,47 @@ -var express = require('express'); -var path = require('path'); -var favicon = require('serve-favicon'); -var logger = require('morgan'); -var cookieParser = require('cookie-parser'); -var bodyParser = require('body-parser'); +const express = require("express"); +const path = require("path"); +const favicon = require("serve-favicon"); +const logger = require("morgan"); +const cookieParser = require("cookie-parser"); +const bodyParser = require("body-parser"); -var routes = require('./routes/index'); -var users = require('./routes/users'); -var todos = require('./routes/todos'); +const routes = require("./routes/index"); +const users = require("./routes/users"); +const todos = require("./routes/todos"); // load mongoose package -var mongoose = require('mongoose'); +const mongoose = require("mongoose"); // Use native Node promises mongoose.Promise = global.Promise; // connect to MongoDB -mongoose.connect('mongodb://localhost/todo-api') - .then(() => console.log('connection succesful')) +mongoose + .connect("mongodb://localhost/todo-api") + .then(() => console.log("connection succesful")) .catch((err) => console.error(err)); -var app = express(); +const app = express(); // view engine setup -app.set('views', path.join(__dirname, 'views')); -app.set('view engine', 'ejs'); +app.set("views", path.join(__dirname, "views")); +app.set("view engine", "ejs"); // uncomment after placing your favicon in /public //app.use(favicon(path.join(__dirname, 'public', 'favicon.ico'))); -app.use(logger('dev')); +app.use(logger("dev")); app.use(bodyParser.json()); app.use(bodyParser.urlencoded({ extended: false })); app.use(cookieParser()); -app.use(express.static(path.join(__dirname, 'public'))); +app.use(express.static(path.join(__dirname, "public"))); -app.use('/', routes); -app.use('/users', users); -app.use('/todos', todos); +app.use("/", routes); +app.use("/users", users); +app.use("/todos", todos); // catch 404 and forward to error handler -app.use(function(req, res, next) { - var err = new Error('Not Found'); +app.use(function (req, res, next) { + const err = new Error("Not Found"); err.status = 404; next(err); }); @@ -49,25 +50,24 @@ app.use(function(req, res, next) { // development error handler // will print stacktrace -if (app.get('env') === 'development') { - app.use(function(err, req, res, next) { +if (app.get("env") === "development") { + app.use(function (err, req, res, next) { res.status(err.status || 500); - res.render('error', { + res.render("error", { message: err.message, - error: err + error: err, }); }); } // production error handler // no stacktraces leaked to user -app.use(function(err, req, res, next) { +app.use(function (err, req, res, next) { res.status(err.status || 500); - res.render('error', { + res.render("error", { message: err.message, - error: {} + error: {}, }); }); - module.exports = app; diff --git a/bin/www b/bin/www index 1a251df..90e3c67 100755 --- a/bin/www +++ b/bin/www @@ -4,37 +4,37 @@ * Module dependencies. */ -var app = require('../app'); -var debug = require('debug')('todo-api:server'); -var http = require('http'); +const app = require("../app"); +const debug = require("debug")("todo-api:server"); +const http = require("http"); /** * Get port from environment and store in Express. */ -var port = normalizePort(process.env.PORT || '3000'); -app.set('port', port); +const port = normalizePort(process.env.PORT || "3000"); +app.set("port", port); /** * Create HTTP server. */ -var server = http.createServer(app); +const server = http.createServer(app); /** * Listen on provided port, on all network interfaces. */ server.listen(port); -server.on('error', onError); -server.on('listening', onListening); +server.on("error", onError); +server.on("listening", onListening); /** * Normalize a port into a number, string, or false. */ function normalizePort(val) { - var port = parseInt(val, 10); + const port = parseInt(val, 10); if (isNaN(port)) { // named pipe @@ -54,22 +54,20 @@ function normalizePort(val) { */ function onError(error) { - if (error.syscall !== 'listen') { + if (error.syscall !== "listen") { throw error; } - var bind = typeof port === 'string' - ? 'Pipe ' + port - : 'Port ' + port; + const bind = typeof port === "string" ? "Pipe " + port : "Port " + port; // handle specific listen errors with friendly messages switch (error.code) { - case 'EACCES': - console.error(bind + ' requires elevated privileges'); + case "EACCES": + console.error(bind + " requires elevated privileges"); process.exit(1); break; - case 'EADDRINUSE': - console.error(bind + ' is already in use'); + case "EADDRINUSE": + console.error(bind + " is already in use"); process.exit(1); break; default: @@ -82,9 +80,7 @@ function onError(error) { */ function onListening() { - var addr = server.address(); - var bind = typeof addr === 'string' - ? 'pipe ' + addr - : 'port ' + addr.port; - debug('Listening on ' + bind); + const addr = server.address(); + const bind = typeof addr === "string" ? "pipe " + addr : "port " + addr.port; + debug("Listening on " + bind); } diff --git a/routes/index.js b/routes/index.js index 07c0a66..11d779e 100644 --- a/routes/index.js +++ b/routes/index.js @@ -1,9 +1,9 @@ -var express = require('express'); -var router = express.Router(); +const express = require("express"); +const router = express.Router(); /* GET home page. */ -router.get('/', function(req, res, next) { - res.render('index', { title: 'ngTodo' }); +router.get("/", function (req, res, next) { + res.render("index", { title: "ngTodo" }); }); module.exports = router; diff --git a/routes/todos.js b/routes/todos.js index 26741d9..dc7880a 100644 --- a/routes/todos.js +++ b/routes/todos.js @@ -1,10 +1,10 @@ -var express = require('express'); -var router = express.Router(); +const express = require("express"); +const router = express.Router(); -var Todo = require('../models/Todo.js'); +const Todo = require("../models/Todo.js"); /* GET /todos listing. */ -router.get('/', function(req, res, next) { +router.get("/", function (req, res, next) { Todo.find(function (err, todos) { if (err) return next(err); res.json(todos); @@ -12,7 +12,7 @@ router.get('/', function(req, res, next) { }); /* POST /todos */ -router.post('/', function(req, res, next) { +router.post("/", function (req, res, next) { Todo.create(req.body, function (err, post) { if (err) return next(err); res.json(post); @@ -20,7 +20,7 @@ router.post('/', function(req, res, next) { }); /* GET /todos/id */ -router.get('/:id', function(req, res, next) { +router.get("/:id", function (req, res, next) { Todo.findById(req.params.id, function (err, post) { if (err) return next(err); res.json(post); @@ -28,7 +28,7 @@ router.get('/:id', function(req, res, next) { }); /* PUT /todos/:id */ -router.put('/:id', function(req, res, next) { +router.put("/:id", function (req, res, next) { Todo.findByIdAndUpdate(req.params.id, req.body, function (err, post) { if (err) return next(err); res.json(post); @@ -36,7 +36,7 @@ router.put('/:id', function(req, res, next) { }); /* DELETE /todos/:id */ -router.delete('/:id', function(req, res, next) { +router.delete("/:id", function (req, res, next) { Todo.findByIdAndRemove(req.params.id, req.body, function (err, post) { if (err) return next(err); res.json(post); From 54e0422d513318e5e41184afa76487cf20781788 Mon Sep 17 00:00:00 2001 From: DefNotArham Date: Tue, 23 Jun 2026 07:50:58 -0400 Subject: [PATCH 3/4] var to const --- routes/users.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/routes/users.js b/routes/users.js index 623e430..30d30b5 100644 --- a/routes/users.js +++ b/routes/users.js @@ -1,9 +1,9 @@ -var express = require('express'); -var router = express.Router(); +const express = require("express"); +const router = express.Router(); /* GET users listing. */ -router.get('/', function(req, res, next) { - res.send('respond with a resource'); +router.get("/", function (req, res, next) { + res.send("respond with a resource"); }); module.exports = router; From 5fcb6ef8429267682395eddcd162dffdff59f746 Mon Sep 17 00:00:00 2001 From: DefNotArham Date: Tue, 23 Jun 2026 07:53:14 -0400 Subject: [PATCH 4/4] changed functions to arrow functions and added async --- routes/todos.js | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/routes/todos.js b/routes/todos.js index dc7880a..5991956 100644 --- a/routes/todos.js +++ b/routes/todos.js @@ -4,40 +4,40 @@ const router = express.Router(); const Todo = require("../models/Todo.js"); /* GET /todos listing. */ -router.get("/", function (req, res, next) { - Todo.find(function (err, todos) { +router.get("/", async (req, res, next) => { + Todo.find((err, todos) => { if (err) return next(err); res.json(todos); }); }); /* POST /todos */ -router.post("/", function (req, res, next) { - Todo.create(req.body, function (err, post) { +router.post("/", async (req, res, next) => { + Todo.create(req.body, (err, post) => { if (err) return next(err); res.json(post); }); }); /* GET /todos/id */ -router.get("/:id", function (req, res, next) { - Todo.findById(req.params.id, function (err, post) { +router.get("/:id", async (req, res, next) => { + Todo.findById(req.params.id, (err, post) => { if (err) return next(err); res.json(post); }); }); /* PUT /todos/:id */ -router.put("/:id", function (req, res, next) { - Todo.findByIdAndUpdate(req.params.id, req.body, function (err, post) { +router.put("/:id", async (req, res, next) => { + Todo.findByIdAndUpdate(req.params.id, req.body, (err, post) => { if (err) return next(err); res.json(post); }); }); /* DELETE /todos/:id */ -router.delete("/:id", function (req, res, next) { - Todo.findByIdAndRemove(req.params.id, req.body, function (err, post) { +router.delete("/:id", async (req, res, next) => { + Todo.findByIdAndRemove(req.params.id, req.body, (err, post) => { if (err) return next(err); res.json(post); });