From dcdde5aa015cd1a79de54b09f4cc60af5628d19a Mon Sep 17 00:00:00 2001
From: Pierre Narcisi <pierre.narcisi@mnhn.fr>
Date: Mon, 16 Sep 2024 16:36:38 +0200
Subject: [PATCH] Fix(mapping) fix mapping

---
 frontend/cypress/e2e/import/step3-field-mapping-spec.js        | 2 +-
 frontend/cypress/e2e/import/step4-content-mapping-spec.js      | 1 +
 frontend/src/app/GN2CommonModule/form/form.service.ts          | 2 +-
 .../components/import_report/import_report.component.html      | 2 +-
 .../modules/imports/services/mappings/field-mapping.service.ts | 3 ++-
 5 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/frontend/cypress/e2e/import/step3-field-mapping-spec.js b/frontend/cypress/e2e/import/step3-field-mapping-spec.js
index d096a0de7d..2598ecb528 100644
--- a/frontend/cypress/e2e/import/step3-field-mapping-spec.js
+++ b/frontend/cypress/e2e/import/step3-field-mapping-spec.js
@@ -80,7 +80,7 @@ function fillTheFormRaw() {
 
 function fillTheForm() {
   // Fill in the form with mandatory field
-  cy.get(SELECTOR_IMPORT_FIELDMAPPING_VALIDATE).should('exist').should('not.be.enabled');
+  cy.get(SELECTOR_IMPORT_FIELDMAPPING_VALIDATE).should('exist');
 
   selectField(SELECTOR_IMPORT_FIELDMAPPING_DATE_MIN, 'date_debut');
   cy.get(SELECTOR_IMPORT_FIELDMAPPING_VALIDATE).should('exist').should('not.be.enabled');
diff --git a/frontend/cypress/e2e/import/step4-content-mapping-spec.js b/frontend/cypress/e2e/import/step4-content-mapping-spec.js
index 9c6532428a..352dc0ae82 100644
--- a/frontend/cypress/e2e/import/step4-content-mapping-spec.js
+++ b/frontend/cypress/e2e/import/step4-content-mapping-spec.js
@@ -129,6 +129,7 @@ function runTheProcessForOcchab(user) {
   cy.configureImportFile();
   // cy.configureImportFieldMapping();
   selectFieldMappingField('import-fieldmapping-theme-date_min', 'error');
+  selectFieldMappingField('import-fieldmapping-theme-WKT', 'error');
   cy.get('#mat-tab-label-0-1').click();
   selectFieldMappingField('import-fieldmapping-theme-nom_cite', 'error');
   selectFieldMappingField('import-fieldmapping-theme-cd_hab', 'error');
diff --git a/frontend/src/app/GN2CommonModule/form/form.service.ts b/frontend/src/app/GN2CommonModule/form/form.service.ts
index 0b8739e629..6fe0312923 100644
--- a/frontend/src/app/GN2CommonModule/form/form.service.ts
+++ b/frontend/src/app/GN2CommonModule/form/form.service.ts
@@ -249,7 +249,7 @@ export class FormService {
       if (!this.areAnyRefControlsNotNull(entityControls, currentControl)) {
         return null;
       }
-      return this.areAllRefControlsNotNull(referenceControlNames, currentControl)
+      return this.areAnyRefControlsNotNull(referenceControlNames, currentControl)
         ? null
         : Validators.required(currentControl);
     };
diff --git a/frontend/src/app/modules/imports/components/import_report/import_report.component.html b/frontend/src/app/modules/imports/components/import_report/import_report.component.html
index ba54437e73..04147f8c75 100644
--- a/frontend/src/app/modules/imports/components/import_report/import_report.component.html
+++ b/frontend/src/app/modules/imports/components/import_report/import_report.component.html
@@ -264,7 +264,7 @@ <h6>
             <mat-expansion-panel>
               <mat-expansion-panel-header [collapsedHeight]="expansionPanelHeight">
                 <mat-panel-title>
-                  <h6>{{ importErrors.length }} erreur(s)</h6>
+                  <h6 data-qa="import-report-errors-title">{{ importErrors.length }} erreur(s)</h6>
                 </mat-panel-title>
               </mat-expansion-panel-header>
               <table class="table table-responsive table-striped table-bordered">
diff --git a/frontend/src/app/modules/imports/services/mappings/field-mapping.service.ts b/frontend/src/app/modules/imports/services/mappings/field-mapping.service.ts
index 839ebb6463..aa870ecc49 100644
--- a/frontend/src/app/modules/imports/services/mappings/field-mapping.service.ts
+++ b/frontend/src/app/modules/imports/services/mappings/field-mapping.service.ts
@@ -184,7 +184,8 @@ export class FieldMappingService {
         this.fieldsByEntity.set(entity.label, entityFields);
       });
     });
-    this.fieldsByEntity = this.removeCommonStrings(this.fieldsByEntity);
+    if (this.fieldsByEntity.size > 1)
+      this.fieldsByEntity = this.removeCommonStrings(this.fieldsByEntity);
     return flattened;
   }