Merge pull request #103 from mission-apprentissage/role_emails

corrections divers
mission-apprentissage · Oct 2, 2023 · 17ee7e2 · 17ee7e2
2 parents 86770c3 + 7f39e59
commit 17ee7e2
Show file tree

Hide file tree

Showing 4 changed files with 55 additions and 42 deletions.
diff --git a/ui/app/(wrapped)/intentions/intentionForm/capaciteSection/CapaciteSection.tsx b/ui/app/(wrapped)/intentions/intentionForm/capaciteSection/CapaciteSection.tsx
@@ -19,7 +19,6 @@ import { IntentionForms } from "@/app/(wrapped)/intentions/intentionForm/default
 
 import {
   isTypeAugmentation,
-  isTypeDiminution,
   isTypeFermeture,
   isTypeOuverture,
 } from "../../utils/typeDemandeUtils";
@@ -42,41 +41,27 @@ const ConstanteField = ({ value }: { value: string | number | undefined }) => (
 );
 
 const differenceCapacité = (
-  capaciteActuelle: number | undefined = 0,
-  capacite: number | undefined
+  valueA: number | undefined,
+  valueB: number | undefined = 0
 ) => {
-  if (capaciteActuelle === undefined || capacite === undefined) return "-";
-  if (capacite >= capaciteActuelle) return capacite - capaciteActuelle;
-  return capacite - capaciteActuelle;
+  if (valueB === undefined || valueA === undefined) return "-";
+  if (valueA < valueB) return "-";
+  return valueA - valueB;
 };
 
 const ConstanteSection = ({
-  typeDemande,
-  capaciteActuelle,
-  capacite,
+  label,
+  value,
 }: {
-  typeDemande: string;
-  capaciteActuelle?: number;
-  capacite?: number;
+  label: string;
+  value: number | string;
 }) => {
   return (
     <>
-      {(isTypeOuverture(typeDemande) || isTypeAugmentation(typeDemande)) && (
-        <FormControl mb="8">
-          <FormLabel>Nombre de nouvelles places</FormLabel>
-          <ConstanteField
-            value={differenceCapacité(capaciteActuelle, capacite)}
-          />
-        </FormControl>
-      )}
-      {(isTypeDiminution(typeDemande) || isTypeFermeture(typeDemande)) && (
-        <FormControl mb="8">
-          <FormLabel>Nombre de places fermées</FormLabel>
-          <ConstanteField
-            value={differenceCapacité(capacite, capaciteActuelle)}
-          />
-        </FormControl>
-      )}
+      <FormControl mb="8">
+        <FormLabel>{label}</FormLabel>
+        <ConstanteField value={value} />
+      </FormControl>
     </>
   );
 };
@@ -95,6 +80,9 @@ export const CapaciteSection = () => {
 
   const typeDemande = watch("typeDemande");
   const mixte = watch("mixte");
+  const isTransfertApprentissage = watch("motif").includes(
+    "transfert_apprentissage"
+  );
 
   return (
     <>
@@ -122,9 +110,16 @@ export const CapaciteSection = () => {
         <CapaciteScolaireColoreeField maxW={240} flex={1} />
       </Flex>
       <ConstanteSection
-        typeDemande={typeDemande}
-        capaciteActuelle={capaciteScolaireActuelle}
-        capacite={capaciteScolaire}
+        label={
+          isTypeOuverture(typeDemande) || isTypeAugmentation(typeDemande)
+            ? "Nombre de nouvelles places"
+            : "Nombre de places fermées"
+        }
+        value={
+          isTypeOuverture(typeDemande) || isTypeAugmentation(typeDemande)
+            ? differenceCapacité(capaciteScolaire, capaciteScolaireActuelle)
+            : differenceCapacité(capaciteScolaireActuelle, capaciteScolaire)
+        }
       />
       {mixte && (
         <>
@@ -137,9 +132,27 @@ export const CapaciteSection = () => {
             <CapaciteApprentissageColoreeField maxW={240} flex={1} />
           </Flex>
           <ConstanteSection
-            typeDemande={typeDemande}
-            capaciteActuelle={capaciteApprentissageActuelle}
-            capacite={capaciteApprentissage}
+            label={
+              isTransfertApprentissage
+                ? "Nombre de places transférées"
+                : isTypeOuverture(typeDemande) ||
+                  isTypeAugmentation(typeDemande)
+                ? "Nombre de nouvelles places"
+                : "Nombre de places fermées"
+            }
+            value={
+              isTransfertApprentissage ||
+              isTypeOuverture(typeDemande) ||
+              isTypeAugmentation(typeDemande)
+                ? differenceCapacité(
+                    capaciteApprentissage,
+                    capaciteApprentissageActuelle
+                  )
+                : differenceCapacité(
+                    capaciteApprentissageActuelle,
+                    capaciteApprentissage
+                  )
+            }
           />
         </>
       )}

diff --git a/ui/app/(wrapped)/intentions/intentionForm/typeDemandeSection/TypeDemandeSection.tsx b/ui/app/(wrapped)/intentions/intentionForm/typeDemandeSection/TypeDemandeSection.tsx
@@ -56,12 +56,12 @@ export const TypeDemandeSection = ({
           <InfoBox flex="1" mt="8" ml="6">
             Dans le cadre de votre
             {` ${typeDemandesOptions[typeDemande].label.toLowerCase()}, `}
-            veuillez saisir le code diplôme et l’établissement si il est
+            veuillez saisir le code diplôme et l’établissement s'il est
             différent. <br />
             Nous ferons le lien automatiquement entre la demande
             {` d'${typeDemandesOptions[typeDemande].label.toLowerCase()} `}
             et la demande de fermeture / diminution pour le code diplôme et
-            l’UAI renseigné
+            l’UAI renseignés
           </InfoBox>
         </Flex>
       )}

diff --git a/ui/app/(wrapped)/intentions/menuIntention/MenuIntention.tsx b/ui/app/(wrapped)/intentions/menuIntention/MenuIntention.tsx
@@ -40,12 +40,12 @@ export const MenuIntention = ({
   return (
     <Flex direction="column" pr={4} minW={250}>
       <Button
-        disabled={!hasPermissionEnvoi}
+        isDisabled={!hasPermissionEnvoi}
         mb="4"
         variant="createButton"
         size={"md"}
         width={"100%"}
-        as={NextLink}
+        as={hasPermissionEnvoi ? NextLink : undefined}
         href="/intentions/new"
       >
         Nouvelle demande

diff --git a/ui/app/(wrapped)/intentions/utils/typeDemandeUtils.tsx b/ui/app/(wrapped)/intentions/utils/typeDemandeUtils.tsx
@@ -50,7 +50,7 @@ export const typeDemandesOptions: Record<
   augmentation_nette: {
     value: "augmentation_nette",
     label: "Augmentation nette",
-    desc: "Utiliser ce formulaire pour toute augmentation de capacité d'accueil sur une formation existant. Ne pas utiliser pour des places déjà ouvertes sur l'établissement",
+    desc: "Utiliser ce formulaire pour toute augmentation de capacité d'accueil sur une formation existant. Ne pas utiliser pour des places déjà ouvertes sur l'établissement.",
     exemple: (
       <>
         <Text mb="3" fontWeight="bold">
@@ -97,7 +97,7 @@ export const typeDemandesOptions: Record<
   ouverture_compensation: {
     value: "ouverture_compensation",
     label: "Ouverture avec compensation",
-    desc: "Utiliser ce formulaire pour tout cas de transfert de capacité d'une formation vers une autres (voir exemple ci-contre).",
+    desc: "Utiliser ce formulaire pour tout cas de transfert de capacité d'une formation vers une autre (voir exemple ci-contre).",
     exemple: (
       <>
         <Text mb="3" fontWeight="bold">
@@ -110,7 +110,7 @@ export const typeDemandesOptions: Record<
         <OrderedList>
           <ListItem mb="2">
             Dans la demande d’ouverture avec compensation j’indique la formation
-            et l’établissement sur laquelle la fermeture va intervenir.
+            et l’établissement sur lequel la fermeture va intervenir.
           </ListItem>
           <ListItem>
             Une fois cette saisie terminée, je saisis la fermeture en lien sur
@@ -136,7 +136,7 @@ export const typeDemandesOptions: Record<
         <OrderedList>
           <ListItem mb="2">
             Dans la demande d’augmentation avec compensation, j’indique la
-            formation et l’établissement sur laquelle je vais augmenter la
+            formation et l’établissement sur lequel je vais augmenter la
             capacité.
           </ListItem>
           <ListItem>