Don't use form field title for geopackage column name, as geopackage …

…column name is required to be unique
ngageoint · Mar 11, 2024 · 5bdc810 · 5bdc810
1 parent f0b853c
commit 5bdc810
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 2 deletions.
diff --git a/service/functionalTests/exports/fixture.ts b/service/functionalTests/exports/fixture.ts
@@ -123,6 +123,19 @@ export const formsSeed: MageFormCreateRequest[] = [
           { id: 3, value: 3, title: 'gold' },
         ]
       },
+      {
+        id: 5,
+        archived: true,
+        name: 'form1/dropdown3',
+        required: false,
+        title: 'Choice 2',
+        type: FormFieldType.Dropdown,
+        choices: [
+          { id: 1, value: 1, title: 'red' },
+          { id: 2, value: 2, title: 'green' },
+          { id: 3, value: 3, title: 'gold' },
+        ]
+      }
     ]
   },
   {

diff --git a/service/src/export/geopackage.ts b/service/src/export/geopackage.ts
@@ -304,7 +304,7 @@ export class GeoPackage extends Exporter {
           dataColumn: {
             column_name: field.name,
             table_name: 'Form_' + form.id,
-            name: field.title,
+            name: field.name,
             title: field.title
           },
           name: field.name,

diff --git a/service/test/export/geopackageTest.js b/service/test/export/geopackageTest.js
@@ -74,7 +74,7 @@ describe("geopackage export tests", function () {
         const writable = new TestWritableStream();
         writable.on('finish', async () => {
             const zip = await JSZip.loadAsync(writable.byteArray);
-            const jsobj = zip.files['mage-' + event.name + '.gpkg'];
+            const jsobj = zip.files['MAGE ' + event.name + '.gpkg'];
             expect(jsobj).to.be.not.undefined;
 
             const gpkg = await jsobj.async('nodebuffer');