From 7e01fba2c39fd680c334f6443fa6d4539a4cc13f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?T=E1=BB=91ng=20Th=C3=A0nh=20=C4=90=E1=BA=A1t?= <84076965+tongthanhdat009@users.noreply.github.com> Date: Thu, 23 Apr 2026 13:43:53 +0700 Subject: [PATCH] chore(controllers): disable json file writing on page content updates Disable the automatic writing of data to JSON files in `createPageContentController`, `admissionsController`, and `policiesController` to prevent redundant file system operations during updates. --- controllers/_createPageContentController.js | 10 +++++----- controllers/admissionsController.js | 10 +++++----- controllers/policiesController.js | 12 ++++++------ 3 files changed, 16 insertions(+), 16 deletions(-) diff --git a/controllers/_createPageContentController.js b/controllers/_createPageContentController.js index 56a1ace..16478a8 100644 --- a/controllers/_createPageContentController.js +++ b/controllers/_createPageContentController.js @@ -75,11 +75,11 @@ function createPageContentController({ }); } - const finalData = await model - .findOne() - .select("-_id -__v -createdAt -updatedAt") - .lean(); - jsonHelper.writeJsonFile(editorConfig.dataFile, finalData); + // const finalData = await model + // .findOne() + // .select("-_id -__v -createdAt -updatedAt") + // .lean(); + // jsonHelper.writeJsonFile(editorConfig.dataFile, finalData); req.flash("success_msg", `${editorConfig.title} updated successfully`); return req.session.save(() => diff --git a/controllers/admissionsController.js b/controllers/admissionsController.js index 2a082aa..3fe030d 100644 --- a/controllers/admissionsController.js +++ b/controllers/admissionsController.js @@ -314,11 +314,11 @@ controller.updateCalculatorOption = async function updateCalculatorOption(req, r }); } - const finalData = await AdmissionsPage - .findOne() - .select("-_id -__v -createdAt -updatedAt") - .lean(); - jsonHelper.writeJsonFile(admissionsConfig.dataFile, normalizeAdmissionsPayload(finalData)); + // const finalData = await AdmissionsPage + // .findOne() + // .select("-_id -__v -createdAt -updatedAt") + // .lean(); + // jsonHelper.writeJsonFile(admissionsConfig.dataFile, normalizeAdmissionsPayload(finalData)); req.flash("success_msg", "Calculator option updated successfully"); return req.session.save(() => res.redirect(`/admin/admissions/calculator/${optionId}`)); diff --git a/controllers/policiesController.js b/controllers/policiesController.js index 65dc7c9..39ae4da 100644 --- a/controllers/policiesController.js +++ b/controllers/policiesController.js @@ -193,12 +193,12 @@ module.exports = { }); } - const finalData = normalizePoliciesDocument( - (await PoliciesPage.findOne() - .select("-_id -__v -createdAt -updatedAt") - .lean()) || {}, - ); - jsonHelper.writeJsonFile(policiesConfig.dataFile, finalData); + // const finalData = normalizePoliciesDocument( + // (await PoliciesPage.findOne() + // .select("-_id -__v -createdAt -updatedAt") + // .lean()) || {}, + // ); + // jsonHelper.writeJsonFile(policiesConfig.dataFile, finalData); const successMessage = validation.warnings.length ? `Policy content updated with warnings: ${validation.warnings.join(" ")}`