refactor(hyperlink): use generic helper functions only

Author: singerla

__tests__/modify-hyperlink-remove-internal.test.ts

@@ -0,0 +1,24 @@
+import Automizer, { modify } from '../src/index';
+import { vd } from '../src/helper/general-helper';
+
+test('delete internal hyperlink - using removeHyperlink helper', async () => {
+  const automizer = new Automizer({
+    templateDir: `${__dirname}/pptx-templates`,
+    outputDir: `${__dirname}/pptx-output`,
+    verbosity: 0
+  });
+
+  const pres = automizer
+    .loadRoot(`RootTemplate.pptx`)
+    .load(`SlideWithLink.pptx`, 'link');
+
+  const outputFile = `delete-hyperlink-internal.test.pptx`;
+
+  const result = await pres
+    .addSlide('link', 1, (slide) => {
+      slide.modifyElement('LinkToSlide', [modify.removeHyperlink()]);
+    })
+    .write(outputFile);
+
+  console.log(result);
+});

src/classes/shape.ts

@@ -89,21 +89,6 @@ export class Shape {
   }
 
   async setTargetElement(): Promise<void> {
-    if (!this.sourceElement) {
-      // If we don't have a source element, we might be trying to remove a hyperlink
-      console.log(
-        `Warning: No source element for shape ${this.name}. Creating empty element for operations.`,
-      );
-      if (this.shape && this.shape.mode === 'remove' && this.targetArchive) {
-        // For remove operations, we don't need a source element
-        // Just continue without setting targetElement
-        return;
-      }
-
-      // For non-remove operations or if other conditions aren't met, throw the error
-      console.log(this.shape);
-      throw `No source element for shape ${this.name}`;
-    }
     this.targetElement = this.sourceElement.cloneNode(true) as XmlElement;
   }
 

src/helper/modify-hyperlink-element.ts

@@ -0,0 +1,48 @@
+export default class HyperlinkElement {
+  private doc: Document;
+  private relId: string;
+  private isInternal: boolean;
+
+  constructor(doc: Document, relId: string, isInternal: boolean) {
+    this.doc = doc;
+    this.relId = relId;
+    this.isInternal = isInternal;
+  }
+
+  createHlinkClick(): Element {
+    const hlinkClick = this.doc.createElement('a:hlinkClick');
+    hlinkClick.setAttribute('r:id', this.relId);
+    hlinkClick.setAttribute(
+      'xmlns:r',
+      'http://schemas.openxmlformats.org/officeDocument/2006/relationships',
+    );
+
+    if (this.isInternal) {
+      hlinkClick.setAttribute('action', 'ppaction://hlinksldjump');
+      hlinkClick.setAttribute(
+        'xmlns:a',
+        'http://schemas.openxmlformats.org/drawingml/2006/main',
+      );
+      hlinkClick.setAttribute(
+        'xmlns:p14',
+        'http://schemas.microsoft.com/office/powerpoint/2010/main',
+      );
+    }
+
+    return hlinkClick;
+  }
+
+  createTextRun(text: string): Element {
+    const run = this.doc.createElement('a:r');
+    const rPr = this.doc.createElement('a:rPr');
+    const t = this.doc.createElement('a:t');
+
+    rPr.appendChild(this.createHlinkClick());
+    t.textContent = text;
+
+    run.appendChild(rPr);
+    run.appendChild(t);
+
+    return run;
+  }
+}