fix(tabular-editor): disable nested-field parsing (#1164)

Replace the watch([tableRef, schemaFields]) + initializeTable() combo
with a self-stopping watchEffect that initializes the Tabulator table
exactly once.

PapaParse warnings (e.g. ragged rows) are now logged as warning: the
parsed data is still loaded into the table. Only the PapaParse error
callback or a synchronous exception falls back to an empty table.

Disable Tabulator's nestedFieldSeparator so schema field names
containing dots (e.g. note_c2.1) are treated as flat keys.

Without this, Tabulator tries to traverse dotted field names as object
paths and can throw "Cannot assign to read only property" when the
parent value is a primitive string.

Author: nicolaskempf57

components/Datasets/TabularEditor.client.vue

@@ -327,47 +327,17 @@ async function validateData() {
   }
 }
 
-async function initializeTable() {
-  await nextTick()
-
-  if (!tableRef.value) {
-    console.error('tableRef.value est null')
-    return
-  }
-  let tableData: RowData[] = []
-  if (uploadedFile.value) {
-    try {
-      paparse.parse<RowData, File>(uploadedFile.value, {
-        header: true,
-        complete: function (results) {
-          tableData = results.data.map((r, i) => {
-            return { id: i, ...r }
-          })
-          makeTable(tableData, true)
-          uploadedFile.value = null
-        },
-      })
-    }
-    catch (error) {
-      console.error('Erreur lors du chargement du fichier:', error)
-      customErrors.value = [t('Erreur lors du chargement du fichier CSV')]
-      tableData = createEmptyRows(1)
-      makeTable(tableData)
-    }
-  }
-  else {
-    tableData = createEmptyRows(1)
-    makeTable(tableData)
-  }
-}
-
 async function makeTable(data: Array<RowData>, shouldValidate = false) {
   if (!tableRef.value) {
     return
   }
+
+  table?.destroy()
+
   table = new Tabulator(tableRef.value, {
     data: data,
     layout: 'fitData',
+    nestedFieldSeparator: false,
     columns: getColumns(),
     rowContextMenu: [
       {
@@ -518,11 +488,45 @@ function generateFile() {
 
 defineExpose({ generateFile })
 
-watch([tableRef, schemaFields], () => {
-  if (tableRef.value && schemaFields.value.length > 0) {
-    initializeTable()
+const stopInit = watchEffect(() => {
+  if (!tableRef.value || schemaFields.value.length === 0) return
+
+  stopInit()
+
+  if (uploadedFile.value) {
+    let handled = false
+    try {
+      paparse.parse<RowData, File>(uploadedFile.value, {
+        header: true,
+        complete: (results) => {
+          if (handled) return
+          if (results.errors.length > 0) {
+            console.warn('PapaParse a rencontré des avertissements:', results.errors)
+          }
+          const tableData = results.data.map((r, i) => ({ id: i, ...r }))
+          makeTable(tableData, true)
+        },
+        error: (error) => {
+          if (handled) return
+          handled = true
+          console.error('Erreur lors du chargement du fichier:', error)
+          customErrors.value = [t('Erreur lors du chargement du fichier CSV')]
+          makeTable(createEmptyRows(1))
+        },
+      })
+    }
+    catch (error) {
+      if (handled) return
+      handled = true
+      console.error('Erreur lors du chargement du fichier:', error)
+      customErrors.value = [t('Erreur lors du chargement du fichier CSV')]
+      makeTable(createEmptyRows(1))
+    }
+  }
+  else {
+    makeTable(createEmptyRows(1))
   }
-}, { immediate: true })
+}, { flush: 'post' })
 
 watch(validationReport, () => {
   if (validationReport.value && !validationReport.value.report?.valid && validationReport.value.report?.errors) {