Display bsvhu plate errors

MTES-MCT · Jan 12, 2025 · b3e0101 · b3e0101
1 parent 0908262
commit b3e0101
Show file tree

Hide file tree

Showing 6 changed files with 82 additions and 35 deletions.
diff --git a/back/src/bsvhu/validation/__tests__/validation.integration.ts b/back/src/bsvhu/validation/__tests__/validation.integration.ts
@@ -737,7 +737,6 @@ describe("BSVHU validation", () => {
           currentSignatureType: "TRANSPORT"
         });
       } catch (err) {
-        console.log(err);
         expect((err as ZodError).issues).toEqual([
           expect.objectContaining({
             message:
@@ -764,7 +763,6 @@ describe("BSVHU validation", () => {
           currentSignatureType: "TRANSPORT"
         });
       } catch (err) {
-        console.log(err);
         expect((err as ZodError).issues).toEqual([
           expect.objectContaining({
             message:
@@ -791,7 +789,6 @@ describe("BSVHU validation", () => {
           currentSignatureType: "TRANSPORT"
         });
       } catch (err) {
-        console.log(err);
         expect((err as ZodError).issues).toEqual([
           expect.objectContaining({
             message: "Le numéro de plaque fourni est incorrect"

diff --git a/back/src/bsvhu/validation/index.ts b/back/src/bsvhu/validation/index.ts
@@ -21,7 +21,7 @@ import { BsvhuValidationContext, PrismaBsvhuForParsing } from "./types";
  * par signature.
  */
 export async function mergeInputAndParseBsvhuAsync(
-  // BSFF déjà stockée en base de données.
+  // Bsvhu déjà stocké en base de données.
   persisted: PrismaBsvhuForParsing,
   // Données entrantes provenant de la couche GraphQL.
   input: BsvhuInput,
@@ -65,7 +65,7 @@ export async function mergeInputAndParseBsvhuAsync(
   }
 
   // Calcule la signature courante à partir des données si elle n'est
-  // pas fourni via le contexte
+  // pas fournie via le contexte
   const currentSignatureType =
     context.currentSignatureType ?? getCurrentSignatureType(zodPersisted);
 
@@ -75,7 +75,7 @@ export async function mergeInputAndParseBsvhuAsync(
   };
 
   // Vérifie que l'on n'est pas en train de modifier des données
-  // vérrouillées par signature.
+  // verrouillées par signature.
   const updatedFields = await checkBsvhuSealedFields(
     zodPersisted,
     bsvhu,

diff --git a/back/src/bsvhu/validation/refinements.ts b/back/src/bsvhu/validation/refinements.ts
@@ -307,6 +307,46 @@ export const checkTransportModeAndReceptionWeight: Refinement<
   );
 };
 
+const onlyWhiteSpace = (str: string) => !str.trim().length; // check whitespaces, tabs, newlines and invisible chars
+
+export const checkTransportPlates: Refinement<ParsedZodBsvhu> = (
+  bsvhu,
+  ctx
+) => {
+  const { transporterTransportPlates } = bsvhu;
+  const path = ["transporter", "transport", "plates"];
+  if (transporterTransportPlates.length > 2) {
+    ctx.addIssue({
+      code: z.ZodIssueCode.custom,
+      path,
+      message: "Un maximum de 2 plaques d'immatriculation est accepté"
+    });
+  }
+
+  if (transporterTransportPlates.some(plate => plate.length > 12)) {
+    ctx.addIssue({
+      code: z.ZodIssueCode.custom,
+      path,
+      message: "Un numéro d'immatriculation doit faire 12 caractères au maximum"
+    });
+  }
+
+  if (transporterTransportPlates.some(plate => plate.length < 4)) {
+    ctx.addIssue({
+      code: z.ZodIssueCode.custom,
+      path,
+      message: "Un numéro d'immatriculation doit faire 4 caractères au minimum"
+    });
+  }
+  if (transporterTransportPlates.some(plate => onlyWhiteSpace(plate))) {
+    ctx.addIssue({
+      code: z.ZodIssueCode.custom,
+      path,
+      message: "Le numéro de plaque fourni est incorrect"
+    });
+  }
+};
+
 export const checkRequiredFields: (
   validationContext: BsvhuValidationContext
 ) => Refinement<ParsedZodBsvhu> = validationContext => {

diff --git a/back/src/bsvhu/validation/schema.ts b/back/src/bsvhu/validation/schema.ts
@@ -10,7 +10,8 @@ import {
   checkEmitterSituation,
   checkPackagingAndIdentificationType,
   checkTransportModeAndWeight,
-  checkTransportModeAndReceptionWeight
+  checkTransportModeAndReceptionWeight,
+  checkTransportPlates
 } from "./refinements";
 import { BsvhuValidationContext } from "./types";
 import { weightSchema } from "../../common/validation/weight";
@@ -71,8 +72,6 @@ export const ZodOperationEnum = z
 
 export type ZodOperationEnum = z.infer<typeof ZodOperationEnum>;
 
-const notOnlyWhiteSpace = (str: string) => str.trim(); // check whitespaces, tabs, newlines and invisible chars
-
 const rawBsvhuSchema = z.object({
   id: z.string().default(() => getReadableId(ReadableIdPrefix.VHU)),
   // on ajoute `createdAt` au schéma de validation pour appliquer certaines
@@ -192,24 +191,25 @@ const rawBsvhuSchema = z.object({
   transporterTransportTakenOverAt: z.coerce.date().nullish(),
   transporterCustomInfo: z.string().nullish(),
   transporterTransportMode: z.nativeEnum(TransportMode).nullish(),
-  transporterTransportPlates: z
-    .array(
-      z
-        .string()
-        .min(4, {
-          message:
-            "Un numéro d'immatriculation doit faire 4 caractères au minimum"
-        })
-        .max(12, {
-          message:
-            "Un numéro d'immatriculation doit faire 12 caractères au maximum"
-        })
-        .refine(notOnlyWhiteSpace, {
-          message: "Le numéro de plaque fourni est incorrect"
-        })
-    )
-    .max(2, "Un maximum de 2 plaques d'immatriculation est accepté")
-    .default([]),
+  // transporterTransportPlates: z
+  //   .array(
+  //     z
+  //       .string()
+  //       .min(4, {
+  //         message:
+  //           "Un numéro d'immatriculation doit faire 4 caractères au minimum"
+  //       })
+  //       .max(12, {
+  //         message:
+  //           "Un numéro d'immatriculation doit faire 12 caractères au maximum"
+  //       })
+  //   )
+  //   .max(2, "Un maximum de 2 plaques d'immatriculation est accepté")
+
+  //   .default([]),
+
+  transporterTransportPlates: z.array(z.string()).default([]),
+
   ecoOrganismeName: z.string().nullish(),
   ecoOrganismeSiret: siretSchema(CompanyRole.EcoOrganisme).nullish(),
   brokerCompanyName: z.string().nullish(),
@@ -255,7 +255,8 @@ const refinedBsvhuSchema = rawBsvhuSchema
   .superRefine(checkEmitterSituation)
   .superRefine(checkPackagingAndIdentificationType)
   .superRefine(checkTransportModeAndWeight)
-  .superRefine(checkTransportModeAndReceptionWeight);
+  .superRefine(checkTransportModeAndReceptionWeight)
+  .superRefine(checkTransportPlates);
 
 // Transformations synchrones qui sont toujours
 // joués même si `enableCompletionTransformers=false`
@@ -284,7 +285,7 @@ export const contextualBsvhuSchemaAsync = (context: BsvhuValidationContext) => {
     .transform((bsvhu: ParsedZodBsvhu) => runTransformers(bsvhu, context))
     .superRefine(
       // run le check sur les champs requis après les transformations
-      // au cas où des transformations auto-complète certains champs
+      // au cas où des transformations auto-complètent certains champs
       checkRequiredFields(context)
     );
 };
diff --git a/front/src/Apps/Dashboard/Creation/bsvhu/steps/Transporter.tsx b/front/src/Apps/Dashboard/Creation/bsvhu/steps/Transporter.tsx
@@ -92,6 +92,14 @@ const TransporterBsvhu = ({ errors }) => {
           setError
         );
       }
+      if (transporter?.transport?.plates) {
+        setFieldError(
+          errors,
+          `${actor}.transport.plates`,
+          formState.errors?.[actor]?.["transport"]?.plates,
+          setError
+        );
+      }
     }
   }, [
     errors,
@@ -104,7 +112,8 @@ const TransporterBsvhu = ({ errors }) => {
     transporter?.company?.mail,
     transporter?.company?.phone,
     transporter?.company?.siret,
-    transporter?.company?.vatNumber
+    transporter?.company?.vatNumber,
+    transporter?.transport?.plates
   ]);
 
   const orgId = useMemo(
@@ -220,11 +229,6 @@ const TransporterBsvhu = ({ errors }) => {
             fieldName={`transporter.transport.plates`}
             hintText="2 max : Véhicule, remorque"
           />
-          {formState.errors?.transporter?.["transport"]?.plates && (
-            <p className="fr-text--sm fr-error-text fr-mb-4v">
-              {formState.errors?.transporter?.["transport"]?.plates?.message}
-            </p>
-          )}
         </div>
       </div>
     </>

diff --git a/front/src/Apps/Dashboard/Creation/utils.ts b/front/src/Apps/Dashboard/Creation/utils.ts
@@ -110,6 +110,10 @@ const pathPrefixToTab = {
     if (pathPrefix === "ecoOrganisme") {
       return TabId.other;
     }
+    if (pathPrefix.startsWith("transporter")) {
+      // dirty solution to handle TransporterFooBar paths
+      return TabId.transporter;
+    }
     if (Object.values(TabId).includes(pathPrefix as TabId)) {
       return TabId[pathPrefix];
     }
@@ -147,6 +151,7 @@ export const getPublishErrorMessages = (
   apiErrors?: NormalizedError[]
 ): TabError[] => {
   // return an array of messages with tabId, name (path) and the related message
+
   const publishErrorMessages =
     apiErrors
       ?.map(apiError => {