Skip to content

Commit 705f1a4

Browse files
committed
Controller - return/if fixes
1 parent dad2fe0 commit 705f1a4

File tree

2 files changed

+10
-8
lines changed

2 files changed

+10
-8
lines changed

controllers/bookController.js

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -142,11 +142,12 @@ exports.book_create_post = [
142142
book,
143143
errors: errors.array(),
144144
});
145-
} else {
146-
// Data from form is valid. Save book.
147-
await book.save();
148-
res.redirect(book.url);
145+
return;
149146
}
147+
148+
// Data from form is valid. Save book.
149+
await book.save();
150+
res.redirect(book.url);
150151
}),
151152
];
152153

controllers/genreController.js

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -73,11 +73,12 @@ exports.genre_create_post = [
7373
if (genreExists) {
7474
// Genre exists, redirect to its detail page.
7575
res.redirect(genreExists.url);
76-
} else {
77-
await genre.save();
78-
// New genre saved. Redirect to genre detail page.
79-
res.redirect(genre.url);
76+
return;
8077
}
78+
79+
// New genre. Save and redirect to its detail page.
80+
await genre.save();
81+
res.redirect(genre.url);
8182
}),
8283
];
8384

0 commit comments

Comments
 (0)