@7365admin1/module-hygiene 4.29.1-staging.17 → 4.29.1-staging.18

package/dist/index.mjs

@@ -2968,6 +2968,18 @@ function useParentChecklistRepo() {
       throw error;
     }
   }
+  async function reopenParentChecklist(_id) {
+    const res = await collection.updateOne(
+      { _id: new ObjectId7(_id), status: "completed" },
+      { $set: { status: "ongoing", updatedAt: /* @__PURE__ */ new Date() }, $unset: { completedAt: "" } }
+    );
+    if (res.modifiedCount > 0) {
+      delNamespace().catch((err) => {
+        logger14.error(`Failed to clear cache for namespace: ${namespace_collection}`, err);
+      });
+    }
+    return res.modifiedCount;
+  }
   async function getHygieneSiteIds() {
     const query = {
       status: { $ne: "deleted" }
@@ -3074,6 +3086,7 @@ function useParentChecklistRepo() {
     getAllParentChecklist,
     getParentChecklistById,
     updateParentChecklistStatuses,
+    reopenParentChecklist,
     getHygieneSiteIds,
     closeExpiredParentChecklists,
     getTodayParentChecklists,
@@ -4302,6 +4315,10 @@ function useAreaChecklistRepo() {
         }
       );
       if (result.modifiedCount > 0) {
+        await collection.updateOne(
+          { schedule, serviceType, area, status: "completed" },
+          { $set: { status: "ongoing", updatedAt: /* @__PURE__ */ new Date() }, $unset: { completedAt: "" } }
+        );
         delNamespace().catch((err) => {
           logger17.error(
             `Failed to clear cache for namespace: ${namespace_collection}`,
@@ -4598,6 +4615,9 @@ function validateScheduleRules(task, now = /* @__PURE__ */ new Date()) {
   }
   if (task.endDate && task.endDate < today)
     return "The end date has already passed.";
+  if (!repeating && !(task.dates ?? []).length) {
+    return "Pick at least one date, or choose a repeat.";
+  }
   if (!repeating) {
     const dates = (task.dates ?? []).filter((d) => !task.endDate || d <= task.endDate);
     if (dates.length && dates.every((d) => d < today)) {
@@ -7058,7 +7078,11 @@ var SCHEDULE_FIELDS = {
   startDate: 1,
   endDate: 1,
   paused: 1,
-  lastRunDate: 1
+  lastRunDate: 1,
+  // when it was saved: a task saved after its time today waits for its next
+  // day, so Next visit must know it (else it says "today" for a visit that never comes)
+  createdAt: 1,
+  updatedAt: 1
 };
 function useScheduleTaskRepository() {
   const db = useAtlas14.getDb();
@@ -7303,7 +7327,6 @@ function useScheduleTaskRepository() {
             description: 1,
             areas: 1,
             status: 1,
-            createdAt: 1,
             site: 1,
             serviceType: 1,
             ...SCHEDULE_FIELDS
@@ -7429,7 +7452,7 @@ function useScheduleTaskRepository() {
 }
 
 // src/utils/schedule-task-set.util.ts
-function buildScheduleTaskSets(task, units, startSet, count) {
+function buildScheduleTaskSets(task, units, startSet, count = 1) {
   return Array.from({ length: count }, (_, i) => ({
     set: startSet + i,
     units,
@@ -7444,7 +7467,7 @@ function buildScheduleTaskSets(task, units, startSet, count) {
 import { ObjectId as ObjectId20 } from "mongodb";
 import { logger as logger32 } from "@7365admin1/node-server-utils";
 function useScheduleTaskService() {
-  const { createParentChecklist } = useParentChecklistRepo();
+  const { createParentChecklist, reopenParentChecklist } = useParentChecklistRepo();
   const { getAllScheduleTask, markScheduleTaskRan } = useScheduleTaskRepository();
   const {
     createAreaChecklist,
@@ -7515,7 +7538,7 @@ function useScheduleTaskService() {
           const parentChecklistIds = await createParentChecklist({
             site: scheduleTask.site.toString(),
             serviceType,
-            createdAt: /* @__PURE__ */ new Date()
+            createdAt: currentDate ?? /* @__PURE__ */ new Date()
           });
           const parentChecklistId = Array.isArray(parentChecklistIds) ? parentChecklistIds[0] : parentChecklistIds;
           logger32.info(
@@ -7578,14 +7601,13 @@ function useScheduleTaskService() {
                 );
               }
               if (existingAreaChecklist) {
-                const areaSetConfig = Number(areaDetails.set) || 1;
                 const maxSetInChecklist = existingAreaChecklist.checklist?.length ? Math.max(
                   ...existingAreaChecklist.checklist.map(
                     (c) => c.set
                   )
                 ) : 0;
                 const startSetNumber = maxSetInChecklist + 1;
-                const newSets = buildScheduleTaskSets(scheduleTask, units, startSetNumber, areaSetConfig);
+                const newSets = buildScheduleTaskSets(scheduleTask, units, startSetNumber);
                 const modified = await pushScheduleTaskSets(
                   parentChecklistId.toString(),
                   serviceType,
@@ -7596,7 +7618,7 @@ function useScheduleTaskService() {
                 if (modified > 0) {
                   addedAreas.push(area.name);
                   logger32.info(
-                    `Appended ${areaSetConfig} set(s) starting from ${startSetNumber} to area checklist for area ${area.name}`
+                    `Appended extra-visit set ${startSetNumber} to area checklist for area ${area.name}`
                   );
                 } else {
                   logger32.info(
@@ -7604,8 +7626,7 @@ function useScheduleTaskService() {
                   );
                 }
               } else {
-                const areaSetConfig = Number(areaDetails.set) || 1;
-                const schedTaskSets = buildScheduleTaskSets(scheduleTask, units, 1, areaSetConfig);
+                const schedTaskSets = buildScheduleTaskSets(scheduleTask, units, 1);
                 const checklistData = {
                   schedule: parentChecklistId.toString(),
                   serviceType,
@@ -7627,6 +7648,13 @@ function useScheduleTaskService() {
               continue;
             }
           }
+          if (addedAreas.length > 0) {
+            try {
+              await reopenParentChecklist(parentChecklistId);
+            } catch (error) {
+              logger32.error(`Could not reopen day list ${parentChecklistId}`, error);
+            }
+          }
           const attemptKey = `${scheduleTask._id}:${today}`;
           const tried = (attempts.get(attemptKey) ?? 0) + 1;
           attempts.set(attemptKey, tried);
@@ -7776,8 +7804,8 @@ function useScheduleTaskController() {
     const optional = Object.fromEntries(
       Object.entries(scheduleTaskFields).map(([key, rule]) => [
         key,
-        key === "dates" ? Joi19.array().min(1).items(scheduleDate).optional().messages({
-          "array.min": "Pick at least one date, or choose a repeat."
+        key === "dates" ? Joi19.array().items(scheduleDate).optional().messages({
+          "array.base": "Dates must be a list of dates."
         }) : rule.optional()
       ])
     );