fix: executeHyperlink api is missing some attributes #1282

Author: Hufe921

src/editor/core/command/CommandAdapt.ts

@@ -1058,36 +1058,26 @@ export class CommandAdapt {
     this.tableOperate.tableSelectAll()
   }
 
-  public hyperlink(payload: IElement) {
+  public hyperlink(
+    payload: Pick<IElement, 'valueList' | 'hyperlinkId' | 'url'>
+  ) {
+    const { valueList, url, hyperlinkId } = payload
+    if (!url || !valueList?.length) return
     const isDisabled = this.draw.isReadonly() || this.draw.isDisabled()
     if (isDisabled) return
     const activeControl = this.control.getActiveControl()
     if (activeControl) return
     const { startIndex, endIndex } = this.range.getRange()
     if (!~startIndex && !~endIndex) return
-    const elementList = this.draw.getElementList()
-    const { valueList, url } = payload
-    const hyperlinkId = getUUID()
-    const newElementList = valueList?.map<IElement>(v => ({
-      url,
-      hyperlinkId,
-      value: v.value,
-      type: ElementType.HYPERLINK
-    }))
-    if (!newElementList) return
-    const start = startIndex + 1
-    formatElementContext(elementList, newElementList, startIndex, {
-      editorOptions: this.options
-    })
-    this.draw.spliceElementList(
-      elementList,
-      start,
-      startIndex === endIndex ? 0 : endIndex - startIndex,
-      newElementList
-    )
-    const curIndex = start + newElementList.length - 1
-    this.range.setRange(curIndex, curIndex)
-    this.draw.render({ curIndex })
+    this.insertElementList([
+      {
+        type: ElementType.HYPERLINK,
+        value: '',
+        valueList,
+        url,
+        hyperlinkId: hyperlinkId || getUUID()
+      }
+    ])
   }
 
   public getHyperlinkRange(): [number, number] | null {

src/main.ts

@@ -483,8 +483,6 @@ window.onload = function () {
         const url = payload.find(p => p.name === 'url')?.value
         if (!url) return
         instance.command.executeHyperlink({
-          type: ElementType.HYPERLINK,
-          value: '',
           url,
           valueList: splitText(name).map(n => ({
             value: n,