swisstopo · MiraGeowerkstatt · Dec 16, 2024 · Dec 12, 2024 · Dec 12, 2024 · Dec 12, 2024
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -28,6 +28,7 @@
 - Removed deduplication check when adding and detaching attachments.
 - When copying a borehole, attachments won't be copied.
 - Removed csv lithology import.
+- Removed import id from csv import.
 
 ### Fixed
 

diff --git a/src/api/Models/BoreholeImport.cs b/src/api/Models/BoreholeImport.cs
@@ -23,9 +23,4 @@ public class BoreholeImport : Borehole
     /// e.g. "borehole_1.pdf,borehole_2.pdf".
     /// </summary>
     public string Attachments { get; set; }
-
-    /// <summary>
-    /// Gets or sets the <see cref="BoreholeImport"/>'s import id.
-    /// </summary>
-    public int ImportId { get; set; }
 }
diff --git a/src/client/cypress/fixtures/import/boreholes-missing-fields-and-duplicates.csv b/src/client/cypress/fixtures/import/boreholes-missing-fields-and-duplicates.csv
@@ -1,6 +1,6 @@
-import_id;alternate_name;original_name;location_x;location_y
-1;BH-1001;Wellington 1;
-2;BH-1002;Wellington 2;
-3;BH-1003;Wellington 3;2189456;1334567
-4;BH-1004;Wellington 4;2312345;1200987
-5;BH-1005;Wellington 5;2312345;1200987
+alternate_name;original_name;location_x;location_y
+BH-1001;Wellington 1;
+BH-1002;Wellington 2;
+BH-1003;Wellington 3;2189456;1334567
+BH-1004;Wellington 4;2312345;1200987
+BH-1005;Wellington 5;2312345;1200987
diff --git a/src/client/cypress/fixtures/import/boreholes-multiple-valid.csv b/src/client/cypress/fixtures/import/boreholes-multiple-valid.csv
@@ -1,5 +1,5 @@
-import_id;alternate_name;original_name;location_x;location_y;attachments;
-344;BH-1001;Wellington 1;2156784;1154321;borehole_attachment_1.pdf,borehole_attachment_2.zip;
-33;BH-1002;Wellington 2;2367999;1276543;
-2;BH-1003;Wellington 3;2189456;1334567;
-55;BH-1004;Wellington 4;2312345;1200987;
+alternate_name;original_name;location_x;location_y;attachments;
+BH-1001;Wellington 1;2156784;1154321;borehole_attachment_1.pdf,borehole_attachment_2.zip;
+BH-1002;Wellington 2;2367999;1276543;
+BH-1003;Wellington 3;2189456;1334567;
+BH-1004;Wellington 4;2312345;1200987;
diff --git a/...client/cypress/fixtures/import/data-sets/invalid-lithology/lithology-single-not-valid.csv b/...client/cypress/fixtures/import/data-sets/invalid-lithology/lithology-single-not-valid.csv
diff --git a/src/client/cypress/fixtures/import/lithology-single-valid.csv b/src/client/cypress/fixtures/import/lithology-single-valid.csv
diff --git a/src/client/docs/import.md b/src/client/docs/import.md
@@ -50,7 +50,6 @@ Die zu importierenden Daten müssen gemäss obigen Anforderungen im CSV-Format v
 
 | Feldname                    | Datentyp       | Pflichtfeld | Beschreibung                                                                          |
 | --------------------------- | -------------- | ----------- | ------------------------------------------------------------------------------------- |
-| import_id                   | Zahl           | Ja          | Zufällig gewählte Zahl. Wird nicht gepeichert. Muss in der Datei einzigartig sein     |
 | id_geodin_shortname         | Zahl           | Nein        | ID GeODin-Shortname                                                                   |
 | id_info_geol                | Zahl           | Nein        | ID InfoGeol                                                                           |
 | id_original                 | Zahl           | Nein        | ID Original                                                                           |

diff --git a/src/client/src/pages/overview/sidePanelContent/importer/importModalContent.tsx b/src/client/src/pages/overview/sidePanelContent/importer/importModalContent.tsx
@@ -66,7 +66,7 @@ const ImportModalContent = ({ setSelectedBoreholeAttachments, setSelectedFile, s
         <StackHalfWidth direction="column">
           {t("csvFormatExplanation")}
           {ExampleHeadings(
-            "import_id;id_geodin_shortname;id_info_geol;id_original;" +
+            "id_geodin_shortname;id_info_geol;id_original;" +
               "id_canton;id_geo_quat;id_geo_mol;id_geo_therm;id_top_fels;" +
               "id_geodin;id_kernlager;original_name;project_name;name;" +
               "restriction_id;restriction_until;national_interest;location_x;location_y;" +

diff --git a/tests/api/Controllers/UploadControllerTest.cs b/tests/api/Controllers/UploadControllerTest.cs
@@ -838,7 +838,7 @@ public async Task UploadBoreholeCsvFileWithoutAttachmentsButWithProvidedFilesSho
            .Returns(() => new HttpClient())
            .Verifiable();
 
-        var boreholeCsvFile = GetFormFileByContent(fileContent: "import_id;original_name;location_x;location_y\r\n123;Frank Place;2000000;1000000", fileName: "boreholes.csv");
+        var boreholeCsvFile = GetFormFileByContent(fileContent: "original_name;location_x;location_y\r\nFrank Place;2000000;1000000", fileName: "boreholes.csv");
         var firstPdfFormFile = GetFormFileByExistingFile("borehole_attachment_1.pdf");
         var secondPdfFormFile = GetFormFileByExistingFile("borehole_attachment_2.pdf");
 
@@ -860,8 +860,8 @@ public async Task UploadBoreholeCsvFileWithAttachmentsLinkedPdfsShouldCreateBore
         var firstAttachmentFileName = "borehole_attachment_1.pdf";
         var secondAttachmentFileName = "borehole_attachment_2.pdf";
 
-        var pdfContent = @"import_id;original_name;location_x;location_y;attachments
-123;Frank Place;2000000;1000000;borehole_attachment_1.pdf,borehole_attachment_2.pdf";
+        var pdfContent = @"original_name;location_x;location_y;attachments
+Frank Place;2000000;1000000;borehole_attachment_1.pdf,borehole_attachment_2.pdf";
         var boreholeCsvFile = GetFormFileByContent(fileContent: pdfContent, fileName: "boreholes.csv");
         var firstPdfFormFile = GetFormFileByExistingFile(firstAttachmentFileName);
         var secondPdfFormFile = GetFormFileByExistingFile(secondAttachmentFileName);
@@ -990,11 +990,10 @@ public async Task UploadRequiredHeadersMissingShouldReturnError()
         ActionResultAssert.IsBadRequest(result);
 
         ProblemDetails problemDetails = (ProblemDetails)result.Value!;
-        Assert.AreEqual(4, Regex.Matches(problemDetails.Detail!, "Header with name ").Count);
+        Assert.AreEqual(3, Regex.Matches(problemDetails.Detail!, "Header with name ").Count);
         StringAssert.Contains(problemDetails.Detail, "Header with name 'Location_x'[0] was not found.");
         StringAssert.Contains(problemDetails.Detail, "Header with name 'Location_y'[0] was not found.");
         StringAssert.Contains(problemDetails.Detail, "Header with name 'OriginalName'[0] was not found.");
-        StringAssert.Contains(problemDetails.Detail, "Header with name 'ImportId'[0] was not found.");
     }
 
     [TestMethod]

diff --git a/tests/api/TestData/borehole_and_location_data.csv b/tests/api/TestData/borehole_and_location_data.csv
@@ -1,2 +1,2 @@
-import_id;original_name;municipality;canton;country;location_x;location_y
-1;ACORNFLEA;OASIS;DOGLEAK;GOAT-XVI;0;0
+original_name;municipality;canton;country;location_x;location_y
+ACORNFLEA;OASIS;DOGLEAK;GOAT-XVI;0;0
diff --git a/tests/api/TestData/borehole_with_attachments.csv b/tests/api/TestData/borehole_with_attachments.csv
@@ -1,2 +1,2 @@
-import_id;original_name;location_x;location_y;attachments
-123456;ACORNFLEA;0;0;attachment_1.pdf,attachment_2.txt,attachment_3.zip,attachment_4.jpg,attachment_5.csv,borehole_attachment_1.pdf,borehole_attachment_2.pdf,borehole_attachment_3.csv,borehole_attachment_4.zip,borehole_attachment_5.png
+original_name;location_x;location_y;attachments
+ACORNFLEA;0;0;attachment_1.pdf,attachment_2.txt,attachment_3.zip,attachment_4.jpg,attachment_5.csv,borehole_attachment_1.pdf,borehole_attachment_2.pdf,borehole_attachment_3.csv,borehole_attachment_4.zip,borehole_attachment_5.png
diff --git a/tests/api/TestData/borehole_with_mixed_case_in_attachments_filenames.csv b/tests/api/TestData/borehole_with_mixed_case_in_attachments_filenames.csv
@@ -1,2 +1,2 @@
-import_id;original_name;location_x;location_y;attachments
-123456;ACORNFLEA;0;0;Borehole_Attachment_1.pdf,borehole_attachment_2.pdf
+original_name;location_x;location_y;attachments
+ACORNFLEA;0;0;Borehole_Attachment_1.pdf,borehole_attachment_2.pdf
diff --git a/tests/api/TestData/borehole_with_not_present_attachments.csv b/tests/api/TestData/borehole_with_not_present_attachments.csv
@@ -1,2 +1,2 @@
-import_id;original_name;location_x;location_y;attachments
-123456;ACORNFLEA;0;0;borehole_attachment_1.pdf,is_not_present_in_upload_files.pdf
+original_name;location_x;location_y;attachments
+ACORNFLEA;0;0;borehole_attachment_1.pdf,is_not_present_in_upload_files.pdf
diff --git a/tests/api/TestData/boreholes_not_all_have_attachments.csv b/tests/api/TestData/boreholes_not_all_have_attachments.csv
@@ -1,4 +1,4 @@
-import_id;original_name;location_x;location_y;attachments
-123;ACORNFLEA;2000000;1000000;borehole_attachment_1.pdf,borehole_attachment_2.pdf
-234;BERRYSNAIL;2000010;1000010;
-456;BLUEBIRDY;2000020;1000020;;
+original_name;location_x;location_y;attachments
+ACORNFLEA;2000000;1000000;borehole_attachment_1.pdf,borehole_attachment_2.pdf
+BERRYSNAIL;2000010;1000010;
+BLUEBIRDY;2000020;1000020;;
diff --git a/tests/api/TestData/duplicateBoreholesInDb.csv b/tests/api/TestData/duplicateBoreholesInDb.csv
@@ -1,4 +1,4 @@
-import_id;original_name;location_x;location_y;total_depth
-54564;Madrid Perez;2100000;1100000;855;
-787;Seoul Park;2500000;1500000;
-123;Secret Zack;676700;185081;1000
+original_name;location_x;location_y;total_depth
+Madrid Perez;2100000;1100000;855;
+Seoul Park;2500000;1500000;
+Secret Zack;676700;185081;1000
diff --git a/tests/api/TestData/duplicateBoreholesInDbButDifferentWorkgroup.csv b/tests/api/TestData/duplicateBoreholesInDbButDifferentWorkgroup.csv
@@ -1,3 +1,3 @@
-import_id;original_name;location_x;location_y;total_depth
-1;Sophia Lee;2100000;1100000;855;
-2;Liam Kim;2500000;1500000;
+original_name;location_x;location_y;total_depth
+Sophia Lee;2100000;1100000;855;
+Liam Kim;2500000;1500000;
diff --git a/tests/api/TestData/duplicateBoreholesInFile.csv b/tests/api/TestData/duplicateBoreholesInFile.csv
@@ -1,3 +1,3 @@
-import_id;original_name;location_x;location_y;total_depth
-1;Isaac Finn;2100000;1100000;855;
-2;Chloe David;2100000;1100000;855;
+original_name;location_x;location_y;total_depth
+Isaac Finn;2100000;1100000;855;
+Chloe David;2100000;1100000;855;
diff --git a/tests/api/TestData/lv03_coordinates_provided_testdata.csv b/tests/api/TestData/lv03_coordinates_provided_testdata.csv
@@ -1,2 +1,2 @@
-import_id;original_name;location_x;location_y
-123;Unit_Test_ LV03 - All coordinates set;649258.3612564525;131551.85893587855
+original_name;location_x;location_y
+Unit_Test_ LV03 - All coordinates set;649258.3612564525;131551.85893587855
diff --git a/tests/api/TestData/lv03_out_of_range_coordinates_provided_testdata.csv b/tests/api/TestData/lv03_out_of_range_coordinates_provided_testdata.csv
@@ -1,2 +1,2 @@
-import_id;original_name;location_x;location_y
-456;Unit_Test_ LV03 - LV03 x out of range;999999;-999999
+original_name;location_x;location_y
+Unit_Test_ LV03 - LV03 x out of range;999999;-999999
diff --git a/tests/api/TestData/lv95_coordinates_provided_testdata.csv b/tests/api/TestData/lv95_coordinates_provided_testdata.csv
@@ -1,2 +1,2 @@
-import_id;original_name;location_x;location_y
-33;Unit_Test_ LV95 - All coordinates set;2631690;1170516
+original_name;location_x;location_y
+Unit_Test_ LV95 - All coordinates set;2631690;1170516
diff --git a/tests/api/TestData/lv95_only_one_coordinate_provided_testdata.csv b/tests/api/TestData/lv95_only_one_coordinate_provided_testdata.csv
@@ -1,2 +1,2 @@
-import_id;original_name;location_x;location_y
-1;Unit_Test_ LV95 - LV95 x missing;;1178661
+original_name;location_x;location_y
+Unit_Test_ LV95 - LV95 x missing;;1178661