Remove the unnecessary async handler (in Expr5)

Author: hamishwillee

controllers/genreController.js

@@ -2,19 +2,18 @@ const Genre = require("../models/genre");
 const Book = require("../models/book");
 
 const { body, validationResult } = require("express-validator");
-const asyncHandler = require("express-async-handler");
 
 // Display list of all Genre.
-exports.genre_list = asyncHandler(async (req, res, next) => {
+exports.genre_list = async (req, res, next) => {
   const allGenres = await Genre.find().sort({ name: 1 }).exec();
   res.render("genre_list", {
     title: "Genre List",
     list_genres: allGenres,
   });
-});
+};
 
 // Display detail page for a specific Genre.
-exports.genre_detail = asyncHandler(async (req, res, next) => {
+exports.genre_detail = async (req, res, next) => {
   // Get details of genre and all associated books (in parallel)
   const [genre, booksInGenre] = await Promise.all([
     Genre.findById(req.params.id).exec(),
@@ -32,7 +31,7 @@ exports.genre_detail = asyncHandler(async (req, res, next) => {
     genre,
     genre_books: booksInGenre,
   });
-});
+};
 
 // Display Genre create form on GET.
 exports.genre_create_get = (req, res, next) => {
@@ -48,7 +47,7 @@ exports.genre_create_post = [
     .escape(),
 
   // Process request after validation and sanitization.
-  asyncHandler(async (req, res, next) => {
+  async (req, res, next) => {
     // Extract the validation errors from a request.
     const errors = validationResult(req);
 
@@ -79,11 +78,11 @@ exports.genre_create_post = [
     // New genre. Save and redirect to its detail page.
     await genre.save();
     res.redirect(genre.url);
-  }),
+  },
 ];
 
 // Display Genre delete form on GET.
-exports.genre_delete_get = asyncHandler(async (req, res, next) => {
+exports.genre_delete_get = async (req, res, next) => {
   // Get details of genre and all associated books (in parallel)
   const [genre, booksInGenre] = await Promise.all([
     Genre.findById(req.params.id).exec(),
@@ -99,10 +98,10 @@ exports.genre_delete_get = asyncHandler(async (req, res, next) => {
     genre,
     genre_books: booksInGenre,
   });
-});
+};
 
 // Handle Genre delete on POST.
-exports.genre_delete_post = asyncHandler(async (req, res, next) => {
+exports.genre_delete_post = async (req, res, next) => {
   // Get details of genre and all associated books (in parallel)
   const [genre, booksInGenre] = await Promise.all([
     Genre.findById(req.params.id).exec(),
@@ -122,10 +121,10 @@ exports.genre_delete_post = asyncHandler(async (req, res, next) => {
   // Genre has no books. Delete object and redirect to the list of genres.
   await Genre.findByIdAndDelete(req.body.id);
   res.redirect("/catalog/genres");
-});
+};
 
 // Display Genre update form on GET.
-exports.genre_update_get = asyncHandler(async (req, res, next) => {
+exports.genre_update_get = async (req, res, next) => {
   const genre = await Genre.findById(req.params.id).exec();
 
   if (genre === null) {
@@ -136,7 +135,7 @@ exports.genre_update_get = asyncHandler(async (req, res, next) => {
   }
 
   res.render("genre_form", { title: "Update Genre", genre });
-});
+};
 
 // Handle Genre update on POST.
 exports.genre_update_post = [
@@ -147,7 +146,7 @@ exports.genre_update_post = [
     .escape(),
 
   // Process request after validation and sanitization.
-  asyncHandler(async (req, res, next) => {
+  async (req, res, next) => {
     // Extract the validation errors from a request .
     const errors = validationResult(req);
 
@@ -170,5 +169,5 @@ exports.genre_update_post = [
     // Data from form is valid. Update the record.
     await Genre.findByIdAndUpdate(req.params.id, genre);
     res.redirect(genre.url);
-  }),
+  },
 ];