Merge pull request #414 from Harbour-Enterprises/har-9211_comments-export

HAR-9211 Comments export fixes

Author: harbournick

packages/super-editor/src/core/super-converter/SuperConverter.js

@@ -358,7 +358,7 @@ class SuperConverter {
       });
       updatedXml = { ...documentXml };
       commentsRels = relationships;
-    };
+    }
 
     this.convertedXml = { ...this.convertedXml, ...updatedXml };
 
@@ -390,8 +390,10 @@ class SuperConverter {
     const relsData = this.convertedXml['word/_rels/document.xml.rels'];
     const relationships = relsData.elements.find((x) => x.name === 'Relationships');
     const newRels = [];
-
-    let largestId = Math.max(...relationships.elements.map((el) => Number(el.attributes.Id.replace('rId', ''))));
+    
+    const regex = /rId|mi/g;
+    let largestId = Math.max(...relationships.elements.map((el) => Number(el.attributes.Id.replace(regex, ''))));
+    
     rels.forEach((rel) => {
       const existingId = rel.attributes.Id;
       const existingTarget = relationships.elements.find((el) => el.attributes.Target === rel.attributes.Target);
@@ -400,7 +402,7 @@ class SuperConverter {
       if (existingTarget && !isNewMedia) {
         return;
       }
-
+      
       // Update the target to escape ampersands
       rel.attributes.Target = rel.attributes?.Target?.replace(/&/g, '&');
 

packages/super-editor/src/core/super-converter/exporter.js

@@ -1604,11 +1604,14 @@ function translateFieldAnnotation(params) {
 
 export function translateHardBreak() {
   return {
-    name: 'w:br',
-    type: 'element',
-    attributes: { 'w:type': 'page' }
+    name: 'w:r',
+    elements: [{
+      name: 'w:br',
+      type: 'element',
+      attributes: { 'w:type': 'page' }
+    }]
   };
-};
+}
 
 export class DocxExporter {
   constructor(converter) {

packages/super-editor/src/core/super-converter/v2/exporter/commentsExporter.js

@@ -38,11 +38,14 @@ export function translateCommentNode(params, type) {
 
   let commentSchema = getCommentSchema(type, commentIndex);
   if (type === 'End') {
-    const commentReference = { name: 'w:commentReference', attributes: { 'w:id': String(commentIndex) } };
+    const commentReference = {
+      name: 'w:r',
+      elements: [{ name: 'w:commentReference', attributes: { 'w:id': String(commentIndex) } }]
+    };
     commentSchema = [commentSchema, commentReference];
-  };
+  }
   return commentSchema;
-};
+}
 
 
 /**
@@ -185,15 +188,14 @@ export const updateCommentsExtendedXml = (comments = [], commentsExtendedXml) =>
     const attributes = {
       'w15:paraId': comment.commentParaId,
       'w15:done': comment.resolvedTime ? '1' : '0',
-      'w:rsid': comment.commentId || comment.importedId,
     };
 
     const parentId = comment.parentCommentId;
     if (parentId) {
       const parentComment = comments.find((c) => c.commentId === parentId);
       const parentParaId = parentComment.commentParaId;
       attributes['w15:paraIdParent'] = parentParaId;
-    };
+    }
 
     return {
       type: 'element',
@@ -240,7 +242,7 @@ export const updateCommentsIdsAndExtensible = (comments = [], commentsIds, exten
       "name": "w16cex:commentExtensible",
       "attributes": {
           "w16cex:durableId": newDurableId,
-          "w16cex:dateUtc": "2025-03-06T23:32:00Z"
+          "w16cex:dateUtc": toIsoNoFractional()
       }
     };
     extensibleUpdated.elements[0].elements.push(newExtensible);
@@ -353,8 +355,8 @@ export const prepareCommentsXmlFilesForExport = ({
   if (exportType === 'clean') {
     const documentXml = removeCommentsFilesFromConvertedXml(convertedXml);
     return { documentXml, relationships };
-  };
-
+  }
+  
   // Initialize comments files with empty content
   const updatedXml = generateConvertedXmlWithCommentFiles(convertedXml);
 

packages/superdoc/src/core/SuperDoc.js

@@ -523,7 +523,7 @@ export class SuperDoc extends EventEmitter {
     const comments = [];
     if (commentsType !== 'clean') {
       comments.push(...this.commentsStore?.translateCommentsForExport());
-    };
+    }
 
     const docxPromises = [];
     this.superdocStore.documents.forEach((doc) => {