fix problems related to conll dialog

Arborator · Oct 16, 2024 · 1118689 · 1118689
1 parent e85de2a
commit 1118689
Showing 1 changed file with 11 additions and 16 deletions.
diff --git a/src/components/sentence/ConlluDialog.vue b/src/components/sentence/ConlluDialog.vue
@@ -253,15 +253,6 @@ export default defineComponent({
     ...mapState(useProjectStore, ['annotationFeatures']),
     ...mapState(useUserStore, ['isLoggedIn']),
   },
-  watch: {
-    conllTable: {
-      handler() {
-        this.generateConllFromTable();
-        this.currentConllContent = this.sentenceBus.sentenceSVGs[this.userId].exportConll();
-      },
-      deep: true,
-    },
-  },
   mounted() {
     this.sentenceBus.on('open:conlluDialog', ({ userId }) => {
       this.userId = userId;
@@ -278,8 +269,8 @@ export default defineComponent({
     getConllTable() {
       this.conllTable = [];
       this.conllContent = this.sentenceBus.sentenceSVGs[this.userId].exportConll();
-      this.nodesJson = this.sentenceBus.sentenceSVGs[this.userId].treeJson.nodesJson;
-      this.groupsJson = this.sentenceBus.sentenceSVGs[this.userId].treeJson.groupsJson;
+      this.nodesJson = { ...this.sentenceBus.sentenceSVGs[this.userId].treeJson.nodesJson };
+      this.groupsJson = { ...this.sentenceBus.sentenceSVGs[this.userId].treeJson.groupsJson };
       for (const node of Object.values(this.nodesJson)) {
         const multiWordId = `${node.ID}-${parseInt(node.ID) +1}`;
         const multiWordNode = this.groupsJson[multiWordId]
@@ -318,6 +309,7 @@ export default defineComponent({
       });
     },
     onConllDialogOk() {
+      this.generateConllFromTable();
       if (this.currentConllContent !== this.conllContent) {
         this.sentenceBus.emit('tree-update:sentence', {
           sentenceJson: {
@@ -334,25 +326,28 @@ export default defineComponent({
       }
     },
     generateConllFromTable() {
-      this.conllTable.forEach((row, index) => {
+      const newNodesJson: nodesJson_T = {};
+      const newGroupsJson: groupsJson_T = {};
+      this.conllTable.forEach((row) => {
         if (!row.ID.includes('-')) {
-          this.nodesJson[`${index + 1}`] = {
+          newNodesJson[row.ID] = {
             ...row,
             FEATS: row.FEATS ? _featuresConllToJson(row.FEATS) : {},
             MISC: row.MISC ? _featuresConllToJson(row.MISC) : {},
             DEPS: row.DEPS ? _depsConllToJson(row.DEPS) : {},
           };
         }
         else {
-          this.groupsJson[row.ID] = {
+          newGroupsJson[row.ID] = {
             ...row,
             FEATS: row.FEATS ? _featuresConllToJson(row.FEATS) : {},
             MISC: row.MISC ? _featuresConllToJson(row.MISC) : {},
           }
         }
       });
-      this.sentenceBus.sentenceSVGs[this.userId].treeJson.nodesJson = { ...this.nodesJson };
-      this.sentenceBus.sentenceSVGs[this.userId].treeJson.groupsJson = { ...this.groupsJson };
+      this.sentenceBus.sentenceSVGs[this.userId].treeJson.nodesJson = { ...newNodesJson};
+      this.sentenceBus.sentenceSVGs[this.userId].treeJson.groupsJson = { ...newGroupsJson };
+      this.currentConllContent = this.sentenceBus.sentenceSVGs[this.userId].exportConll();
     },
     copyConll() {
       copyToClipboard(this.currentConllContent).then(() => {