Ne pas crasher quand une personne a un code d'accès, mais pas de cap …

…d'écriture d'annotations (#86)
betagouv · Sep 18, 2024 · 5b0370d · 5b0370d
1 parent 7dba47b
commit 5b0370d
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/scripts/server/database.js b/scripts/server/database.js
@@ -632,7 +632,7 @@ export async function getInstructeurIdByÉcritureAnnotationCap(cap, databaseConn
  * 
  * @param {NonNullable<Personne['code_accès']>} code_accès 
  * @param {knex.Knex.Transaction | knex.Knex} [databaseConnection]
- * @returns {Promise<{écritureAnnotationCap: CapÉcritureAnnotation['cap'], listerDossiers: string, modifierDossier: string}>}
+ * @returns {Promise<Partial<{écritureAnnotationCap: CapÉcritureAnnotation['cap'], listerDossiers: string, modifierDossier: string}>>}
  */
 export async function getInstructeurCapBundleByPersonneCodeAccès(code_accès, databaseConnection = directDatabaseConnection){
 
@@ -651,8 +651,14 @@ export async function getInstructeurCapBundleByPersonneCodeAccès(code_accès, d
     const modifierDossierP = Promise.resolve(code_accès)
 
     return Promise.all([écritureAnnotationCapP, listerDossiersP, modifierDossierP])
-        .then(([{cap}, listerDossiers, modifierDossier]) => 
-            ({écritureAnnotationCap: cap, listerDossiers, modifierDossier}))
+        .then(([écritureAnnotationCap, listerDossiers, modifierDossier]) => {
+            if(écritureAnnotationCap && écritureAnnotationCap.cap)
+                return {écritureAnnotationCap: écritureAnnotationCap.cap, listerDossiers, modifierDossier}
+            else{
+                return {listerDossiers, modifierDossier}
+            }
+        })
+
 
 
 }