ref #356, delete banners and flags

3 years ago · b4415ab940
parent db01644f14
commit b4415ab940
2 changed files with 6 additions and 10 deletions
--- a/controllers/forms/deletecustompage.js
+++ b/controllers/forms/deletecustompage.js
@ -14,11 +14,9 @@ module.exports = {

 	controller: async (req, res, next) => {

-		const errors = [];
-
-		if (!req.body.checkedcustompages || req.body.checkedcustompages.length === 0) {
-			errors.push('Must select at least one custom page to delete');
-		}
+		const errors = await checkSchema([
+			{ result: lengthBody(req.body.checkedcustompages, 1), expected: false, error: 'Must select at least one custom page to delete' },
+		]);

 		if (errors.length > 0) {
 			return dynamicResponse(req, res, 400, 'message', {
--- a/controllers/forms/deleteflags.js
+++ b/controllers/forms/deleteflags.js
@ -14,11 +14,9 @@ module.exports = {

 	controller: async (req, res, next) => {

-		const errors = [];
-
-		if (!req.body.checkedflags || req.body.checkedflags.length === 0) {
-			errors.push('Must select at least one flag to delete');
-		}
+		const errors = await checkSchema([
+			{ result: lengthBody(req.body.checkedflags, 1), expected: false, error: 'Must select at least one flag to delete' },
+		]);

 		if (errors.length > 0) {
 			return dynamicResponse(req, res, 400, 'message', {