@7365admin1/module-hygiene 4.29.1-staging.12 → 4.30.1

package/dist/index.mjs

@@ -3324,7 +3324,11 @@ function useAreaChecklistRepo() {
       const parentChecklistId = new ObjectId9(value.schedule);
       const areaId = new ObjectId9(value.area);
       const existingChecklist = await collection.findOne(
-        { schedule: parentChecklistId, area: areaId, serviceType: value.serviceType },
+        {
+          schedule: parentChecklistId,
+          area: areaId,
+          serviceType: value.serviceType
+        },
         { session }
       );
       if (existingChecklist) {
@@ -4080,6 +4084,32 @@ function useAreaChecklistRepo() {
       throw error;
     }
   }
+  async function readSetCompletionState(_id, set, unitId) {
+    try {
+      _id = new ObjectId9(_id);
+    } catch (error) {
+      throw new BadRequestError15("Invalid area checklist ID format.");
+    }
+    const doc = await collection.findOne(
+      { _id, "checklist.set": set },
+      { projection: { checklist: 1 } }
+    );
+    const group = (doc?.checklist ?? []).find(
+      (entry) => entry.set === set
+    );
+    const units = group?.units ?? [];
+    if (units.length === 0)
+      return { completesSet: false, hasEvidence: false };
+    const others = units.filter(
+      (unit) => String(unit.unit) !== String(unitId)
+    );
+    return {
+      completesSet: others.every((unit) => unit.status === "completed"),
+      hasEvidence: others.some(
+        (unit) => Array.isArray(unit.attachment) && unit.attachment.length > 0
+      )
+    };
+  }
   async function updateAreaChecklist(_id, value, session) {
     try {
       _id = new ObjectId9(_id);
@@ -4364,6 +4394,7 @@ function useAreaChecklistRepo() {
     updateAreaChecklist,
     updateAreaChecklistStatus,
     updateAreaChecklistUnits,
+    readSetCompletionState,
     getMaxSetNumberForArea,
     closeExpiredAreaChecklists,
     pushScheduleTaskSets,
@@ -4619,6 +4650,24 @@ import {
 } from "@7365admin1/node-server-utils";
 import { AppServiceType as AppServiceType10 } from "@7365admin1/core";
 
+// src/utils/completion-photo.util.ts
+var COMPLETION_PHOTO_MESSAGE = "A photo is required to complete this task. Take or upload at least one photo before marking it complete.";
+function requiresCompletionPhoto({
+  decision,
+  attachment,
+  completesSet,
+  hasEvidence
+}) {
+  if (decision !== "approve")
+    return false;
+  if (!completesSet)
+    return false;
+  if (hasEvidence)
+    return false;
+  const carriesEvidence = Array.isArray(attachment) && attachment.some((id) => typeof id === "string" && id.trim().length > 0);
+  return !carriesEvidence;
+}
+
 // src/services/hygiene-checklist-pdf.service.ts
 import { launch } from "puppeteer";
 import { InternalServerError as InternalServerError6, useAtlas as useAtlas8 } from "@7365admin1/node-server-utils";
@@ -4839,7 +4888,8 @@ function useAreaChecklistController() {
     getAllAreaChecklist: _getAllAreaChecklist,
     getAreaChecklistHistory: _getAreaChecklistHistory,
     getAreaChecklistHistoryDetails: _getAreaChecklistHistoryDetails,
-    getAreaChecklistUnits: _getAreaChecklistUnits
+    getAreaChecklistUnits: _getAreaChecklistUnits,
+    readSetCompletionState: _readSetCompletionState
   } = useAreaChecklistRepo();
   const {
     createAreaChecklist: _createAreaChecklist,
@@ -5044,36 +5094,11 @@ function useAreaChecklistController() {
       reject: Joi9.boolean().optional(),
       remarks: Joi9.string().optional().allow("", null),
       /*
-       * A PHOTO IS REQUIRED TO COMPLETE A TASK.
-       *
-       * Owner requirement, 2026-09-10: *"once task was completed they required
-       * to take photo before completing the task ... all the services that has
-       * this module must be the same process."*
-       *
-       * This endpoint is the one place every service and every client meets:
-       * the five field apps (M&E, Cleaning, Landscape, Pest, Pool) all PATCH
-       * `/api/hygiene-area-checklist/...` with a `serviceType`, and the web
-       * checklist (`layer-common ManageChecklistMain`) posts here too. Requiring
-       * it here is what makes "the same process" true rather than five client
-       * guards that can drift apart — and a client guard alone is a suggestion.
-       *
-       * ONLY ON APPROVE. Completing a unit is the act the evidence is for.
-       * A REJECT is the technician reporting a problem, and demanding a
-       * photograph before somebody can report one would stop them reporting it.
-       * `decision` carries which is which.
-       *
-       * At least one entry, and each must be a real id — an empty array or a
-       * blank string would satisfy `.optional()` while carrying no evidence at
-       * all, which is the shape a client sends when its uploader failed.
+       * Shape only. WHETHER a photo is required depends on the set's current
+       * state in the database, which Joi cannot see — see the check below.
+       * Each entry must still be a real id: a blank string carries no evidence.
        */
-      attachment: Joi9.array().items(Joi9.string().trim().min(1)).when("decision", {
-        is: "approve",
-        then: Joi9.array().min(1).required().messages({
-          "array.min": "A photo is required to complete this task. Take or upload at least one photo before marking it complete.",
-          "any.required": "A photo is required to complete this task. Take or upload at least one photo before marking it complete."
-        }),
-        otherwise: Joi9.array().optional()
-      }),
+      attachment: Joi9.array().items(Joi9.string().trim().min(1)).optional(),
       completedBy: Joi9.string().hex().required()
     });
     const { error } = validation.validate(payload);
@@ -5084,7 +5109,24 @@ function useAreaChecklistController() {
     }
     try {
       const { id, set, unit, decision: decision2, ...value } = payload;
-      await _updateAreaChecklistUnits(id, parseInt(set), unit, value);
+      const setNumber = parseInt(set);
+      if (decision2 === "approve") {
+        const { completesSet, hasEvidence } = await _readSetCompletionState(
+          id,
+          setNumber,
+          unit
+        );
+        if (requiresCompletionPhoto({
+          decision: decision2,
+          attachment: value.attachment,
+          completesSet,
+          hasEvidence
+        })) {
+          next(new BadRequestError16(COMPLETION_PHOTO_MESSAGE));
+          return;
+        }
+      }
+      await _updateAreaChecklistUnits(id, setNumber, unit, value);
       res.json({ message: "Area checklist updated successfully." });
       return;
     } catch (error2) {