@7365admin1/module-hygiene 4.29.0 → 4.30.0

package/dist/index.mjs

@@ -5043,7 +5043,37 @@ function useAreaChecklistController() {
       approve: Joi9.boolean().optional(),
       reject: Joi9.boolean().optional(),
       remarks: Joi9.string().optional().allow("", null),
-      attachment: Joi9.array().items(Joi9.string()).optional(),
+      /*
+       * 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.
+       */
+      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()
+      }),
       completedBy: Joi9.string().hex().required()
     });
     const { error } = validation.validate(payload);