migration: postgres mes-adresses-api (#945)

* update api-bal open-api * update front * change open id class * insert/update positions * correct create/update numero/toponyme * correct build * correct wording * correct batch numero -> toponyme null * refacto openapi * test ci * test ci * test ci * test ci * correct souncloud alert * delete package mongoose
BaseAdresseNationale · Sep 10, 2024 · 3c7b339 · 3c7b339
1 parent f332be0
commit 3c7b339
Show file tree

Hide file tree

Showing 89 changed files with 509 additions and 648 deletions.
diff --git a/components/bal/address-editor.tsx b/components/bal/address-editor.tsx
@@ -43,7 +43,7 @@ function AddressEditor({ commune, closeForm }: AddressEditorProps) {
           <ToponymeEditor commune={commune} closeForm={closeForm} />
         ) : (
           <NumeroEditor
-            initialVoieId={voie?._id}
+            initialVoieId={voie?.id}
             commune={commune}
             hasPreview
             closeForm={closeForm}

diff --git a/components/bal/commune-tab.tsx b/components/bal/commune-tab.tsx
@@ -26,7 +26,7 @@ function CommuneTab({ commune, openRecoveryDialog }: CommuneTabProps) {
     <Pane overflowY="auto">
       {!token && <ReadOnlyInfos openRecoveryDialog={openRecoveryDialog} />}
       {signalements.length > 0 && (
-        <SignalementsInfos balId={baseLocale._id} signalements={signalements} />
+        <SignalementsInfos balId={baseLocale.id} signalements={signalements} />
       )}
       {token && baseLocale.status !== BaseLocale.status.DEMO && (
         <HabilitationInfos commune={commune} />

diff --git a/components/bal/numero-editor.tsx b/components/bal/numero-editor.tsx
@@ -28,9 +28,7 @@ import DisabledFormInput from "@/components/disabled-form-input";
 import {
   BasesLocalesService,
   Numero,
-  NumeroPopulate,
   NumerosService,
-  Voie,
   VoiesService,
 } from "@/lib/openapi";
 import { CommuneType } from "@/types/commune";
@@ -40,7 +38,7 @@ const REMOVE_TOPONYME_LABEL = "Aucun toponyme";
 
 interface NumeroEditorProps {
   initialVoieId?: string;
-  initialValue?: Numero | NumeroPopulate;
+  initialValue?: Numero;
   commune: CommuneType;
   hasPreview?: boolean;
   closeForm: () => void;
@@ -61,11 +59,9 @@ function NumeroEditor({
   certificationBtnProps,
   onVoieChanged,
 }: NumeroEditorProps) {
-  const [voieId, setVoieId] = useState(
-    initialVoieId || (initialValue?.voie as unknown as Voie)?._id
-  );
+  const [voieId, setVoieId] = useState(initialVoieId || initialValue?.voieId);
   const [selectedNomToponyme, setSelectedNomToponyme] = useState("");
-  const [toponymeId, setToponymeId] = useState(initialValue?.toponyme);
+  const [toponymeId, setToponymeId] = useState(initialValue?.toponymeId);
   const [isLoading, setIsLoading] = useState(false);
   const [certifie, setCertifie] = useState(initialValue?.certifie || false);
   const [numero, onNumeroChange] = useInput(
@@ -99,7 +95,7 @@ function NumeroEditor({
   const getEditedVoie = useCallback(async () => {
     if (nomVoie) {
       try {
-        const newVoie = await BasesLocalesService.createVoie(baseLocale._id, {
+        const newVoie = await BasesLocalesService.createVoie(baseLocale.id, {
           nom: nomVoie,
         });
         return newVoie;
@@ -111,12 +107,12 @@ function NumeroEditor({
       }
     }
 
-    return { _id: voieId };
-  }, [baseLocale._id, nomVoie, voieId, setValidationMessages]);
+    return { id: voieId };
+  }, [baseLocale.id, nomVoie, voieId, setValidationMessages]);
 
   const getNumeroBody = useCallback(() => {
     const body = {
-      toponyme: toponymeId,
+      toponymeId,
       numero: Number(numero),
       suffixe: suffixe?.length > 0 ? suffixe.toLowerCase().trim() : null,
       comment: comment.length > 0 ? comment : null,
@@ -128,6 +124,7 @@ function NumeroEditor({
       const positions = [];
       markers.forEach((marker) => {
         positions.push({
+          id: marker.id,
           point: {
             type: "Point",
             coordinates: [marker.longitude, marker.latitude],
@@ -159,11 +156,11 @@ function NumeroEditor({
 
         const voie = await getEditedVoie();
 
-        if (initialValue?._id) {
+        if (initialValue?.id) {
           const updateNumero = toaster(
             () =>
-              NumerosService.updateNumero(initialValue._id, {
-                voie: voie._id,
+              NumerosService.updateNumero(initialValue.id, {
+                voieId: voie.id,
                 ...body,
               }),
             "Le numéro a bien été modifié",
@@ -175,7 +172,7 @@ function NumeroEditor({
           await updateNumero();
         } else {
           const createNumero = toaster(
-            () => VoiesService.createNumero(voie._id, body),
+            () => VoiesService.createNumero(voie.id, body),
             "Le numéro a bien été ajouté",
             "Le numéro n’a pas pu être ajouté",
             (err) => {
@@ -192,7 +189,7 @@ function NumeroEditor({
           await reloadParcelles();
         }
 
-        if (initialVoieId !== voie._id) {
+        if (initialVoieId !== voie.id) {
           reloadVoies();
         }
 
@@ -277,7 +274,7 @@ function NumeroEditor({
   useEffect(() => {
     let nom = null;
     if (voieId) {
-      nom = voies.find((voie) => voie._id === voieId).nom;
+      nom = voies.find((voie) => voie.id === voieId).nom;
     }
 
     setSelectedNomVoie(nom);
@@ -286,15 +283,15 @@ function NumeroEditor({
   useEffect(() => {
     let nom = null;
     if (toponymeId) {
-      nom = toponymes.find((toponyme) => toponyme._id === toponymeId).nom;
+      nom = toponymes.find((toponyme) => toponyme.id === toponymeId).nom;
     }
 
     setSelectedNomToponyme(nom);
   }, [toponymeId, toponymes]);
 
   return (
     <Form
-      editingId={initialValue?._id}
+      editingId={initialValue?.id}
       closeForm={closeForm}
       onFormSubmit={onFormSubmit}
     >
@@ -338,13 +335,13 @@ function NumeroEditor({
               }}
             >
               <option value={null}>
-                {initialValue?.toponyme
+                {initialValue?.toponymeId
                   ? REMOVE_TOPONYME_LABEL
                   : "- Choisir un toponyme -"}
               </option>
               {sortBy(toponymes, (t) => normalizeSort(t.nom)).map(
-                ({ _id, nom }) => (
-                  <option key={_id} value={_id}>
+                ({ id, nom }) => (
+                  <option key={id} value={id}>
                     {nom}
                   </option>
                 )

diff --git a/components/bal/numero-editor/numero-voie-selector.tsx b/components/bal/numero-editor/numero-voie-selector.tsx
@@ -81,8 +81,8 @@ function NumeroVoieSelector({
             onChange={handleVoieChange}
           >
             {!voieId && <option value="">- Choisir une voie-</option>}
-            {sortBy(voies, (v) => normalizeSort(v.nom)).map(({ _id, nom }) => (
-              <option key={_id} value={_id}>
+            {sortBy(voies, (v) => normalizeSort(v.nom)).map(({ id, nom }) => (
+              <option key={id} value={id}>
                 {nom}
               </option>
             ))}

diff --git a/components/bal/position-editor.tsx b/components/bal/position-editor.tsx
@@ -42,11 +42,11 @@ function PositionEditor({
   useEffect(() => {
     if (initialPositions) {
       const positions = initialPositions.map((position) => ({
+        id: position.id,
         longitude: position.point.coordinates[0],
         latitude: position.point.coordinates[1],
         type: position.type,
       }));
-
       positions.forEach((position) => addMarker(position));
     } else {
       handleAddMarker();
@@ -90,7 +90,7 @@ function PositionEditor({
             .filter(({ isDisabled }) => !isDisabled)
             .map((marker) => (
               <PositionItem
-                key={marker._id}
+                key={marker.id}
                 marker={marker}
                 isRemovable={isToponyme ? false : markers.length === 1}
                 handleChange={updateMarker}

diff --git a/components/bal/position-item.tsx b/components/bal/position-item.tsx
@@ -31,17 +31,17 @@ function PositionItem({
   const onSelect = useCallback(
     (e) => {
       const type = e.target.value;
-      handleChange && handleChange(marker._id, { ...marker, type });
+      handleChange && handleChange(marker.id, { ...marker, type });
     },
     [marker, handleChange]
   );
 
   const removeMarker = useCallback(
     (e) => {
       e.preventDefault();
-      onRemove(marker._id);
+      onRemove(marker.id);
     },
-    [marker._id, onRemove]
+    [marker.id, onRemove]
   );
 
   return (

diff --git a/components/bal/toponyme-editor.tsx b/components/bal/toponyme-editor.tsx
@@ -72,6 +72,7 @@ function ToponymeEditor({
       if (markers) {
         markers.forEach((marker) => {
           body.positions.push({
+            id: marker.id,
             point: {
               type: "Point",
               coordinates: [marker.longitude, marker.latitude],
@@ -85,15 +86,15 @@ function ToponymeEditor({
         // Add or edit a toponyme
         const submit = initialValue
           ? toaster(
-              () => ToponymesService.updateToponyme(initialValue._id, body),
+              () => ToponymesService.updateToponyme(initialValue.id, body),
               "Le toponyme a bien été modifé",
               "Le toponyme n’a pas pu être modifié",
               (error) => {
                 setValidationMessages(error.body.message);
               }
             )
           : toaster(
-              () => BasesLocalesService.createToponyme(baseLocale._id, body),
+              () => BasesLocalesService.createToponyme(baseLocale.id, body),
               "Le toponyme a bien été ajouté",
               "Le toponyme n’a pas pu être ajouté",
               (error) => {
@@ -105,7 +106,7 @@ function ToponymeEditor({
 
         refreshBALSync();
 
-        if (initialValue?._id === toponyme._id && router.query.idToponyme) {
+        if (initialValue?.id === toponyme.id && router.query.idToponyme) {
           setToponyme(toponyme);
         }
 
@@ -127,7 +128,7 @@ function ToponymeEditor({
       }
     },
     [
-      baseLocale._id,
+      baseLocale.id,
       initialValue,
       nom,
       nomAlt,
@@ -168,7 +169,7 @@ function ToponymeEditor({
 
   return (
     <Form
-      editingId={initialValue?._id}
+      editingId={initialValue?.id}
       closeForm={closeForm}
       onFormSubmit={onFormSubmit}
     >

diff --git a/components/bal/toponymes-list.tsx b/components/bal/toponymes-list.tsx
@@ -141,9 +141,9 @@ function ToponymesList({
 
         <InfiniteScrollList items={scrollableItems}>
           {(toponyme: ExtentedToponymeDTO & { commentedNumeros: Numero[] }) => (
-            <Table.Row key={toponyme._id} paddingRight={8} minHeight={48}>
+            <Table.Row key={toponyme.id} paddingRight={8} minHeight={48}>
               <Table.Cell
-                onClick={() => onSelect(toponyme._id)}
+                onClick={() => onSelect(toponyme.id)}
                 cursor="pointer"
                 className="main-table-cell"
               >
@@ -181,13 +181,13 @@ function ToponymesList({
               {isEditingEnabled && (
                 <TableRowActions
                   onSelect={() => {
-                    onSelect(toponyme._id);
+                    onSelect(toponyme.id);
                   }}
                   onEdit={() => {
-                    onEnableEditing(toponyme._id);
+                    onEnableEditing(toponyme.id);
                   }}
                   onRemove={() => {
-                    setToRemove(toponyme._id);
+                    setToRemove(toponyme.id);
                   }}
                 />
               )}

diff --git a/components/bal/voie-editor.tsx b/components/bal/voie-editor.tsx
@@ -72,10 +72,7 @@ function VoieEditor({
         const submit = initialValue
           ? toaster(
               async () =>
-                VoiesService.updateVoie(
-                  initialValue._id,
-                  body as UpdateVoieDTO
-                ),
+                VoiesService.updateVoie(initialValue.id, body as UpdateVoieDTO),
               "La voie a bien été modifiée",
               "La voie n’a pas pu être modifiée",
               (err) => {
@@ -85,7 +82,7 @@ function VoieEditor({
           : toaster(
               async () =>
                 BasesLocalesService.createVoie(
-                  baseLocale._id,
+                  baseLocale.id,
                   body as CreateVoieDTO
                 ),
               "La voie a bien été ajoutée",
@@ -99,7 +96,7 @@ function VoieEditor({
 
         refreshBALSync();
 
-        if (initialValue?._id === voie._id && router.query.idVoie) {
+        if (initialValue?.id === voie.id && router.query.idVoie) {
           setVoie(voie);
           // Reload voie trace
           if (
@@ -126,7 +123,7 @@ function VoieEditor({
       }
     },
     [
-      baseLocale._id,
+      baseLocale.id,
       initialValue,
       nom,
       isMetric,
@@ -175,7 +172,7 @@ function VoieEditor({
 
   return (
     <Form
-      editingId={initialValue?._id}
+      editingId={initialValue?.id}
       unmountForm={onUnmount}
       closeForm={closeForm}
       onFormSubmit={onFormSubmit}

diff --git a/components/bal/voies-list.tsx b/components/bal/voies-list.tsx
@@ -167,9 +167,9 @@ function VoiesList({
 
         <InfiniteScrollList items={scrollableItems}>
           {(voie: ExtendedVoieDTO & { commentedNumeros: Numero[] }) => (
-            <Table.Row key={voie._id} paddingRight={8} minHeight={48}>
+            <Table.Row key={voie.id} paddingRight={8} minHeight={48}>
               <Table.Cell
-                onClick={() => onSelect(voie._id)}
+                onClick={() => onSelect(voie.id)}
                 cursor="pointer"
                 className="main-table-cell"
               >
@@ -207,19 +207,19 @@ function VoiesList({
               {isEditingEnabled && (
                 <TableRowActions
                   onSelect={() => {
-                    onSelect(voie._id);
+                    onSelect(voie.id);
                   }}
                   onEdit={() => {
-                    onEnableEditing(voie._id);
+                    onEnableEditing(voie.id);
                   }}
                   onRemove={() => {
-                    setToRemove(voie._id);
+                    setToRemove(voie.id);
                   }}
                   extra={
                     voie.nbNumeros === 0
                       ? {
                           callback: () => {
-                            setToConvert(voie._id);
+                            setToConvert(voie.id);
                           },
                           icon: KeyTabIcon,
                           text: "Convertir en toponyme",