From c300b7cfea5a9f7ecf0ec41eaad6377a0029952d Mon Sep 17 00:00:00 2001
From: garrettmills <shout@garrettmills.dev>
Date: Tue, 2 Feb 2021 09:11:59 -0600
Subject: [PATCH] WYSYWIG - only replace links with <a> if they are NOT already
 in a tag

---
 src/app/components/wysiwyg/wysiwyg.component.ts | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/src/app/components/wysiwyg/wysiwyg.component.ts b/src/app/components/wysiwyg/wysiwyg.component.ts
index e9a6d0a..c030939 100644
--- a/src/app/components/wysiwyg/wysiwyg.component.ts
+++ b/src/app/components/wysiwyg/wysiwyg.component.ts
@@ -84,8 +84,16 @@ export class WysiwygComponent implements OnInit {
     }
 
     public get displayContents() {
-        return (this.contents || 'Double-click to edit...').replace(/</g, '\n<')
-            .replace(/(https?:\/\/[^\s]+)/g, (val) => `<a href="${val}" target="_blank">${val}</a>`);
+        return this.replaceLinks((this.contents || 'Double-click to edit...').replace(/</g, '\n<'));
+    }
+
+    private replaceLinks(text) {
+        const exp = /((href|src)=["']|)(\b(https?|ftp|file):\/\/[-A-Z0-9+&@#\/%?=~_|!:,.;]*[-A-Z0-9+&@#\/%=~_|])/ig;
+        return text.replace(exp, (...args) => {
+            return  args[2] ?
+                args[0] :
+                '<a href="' + args[3] + '" target="_blank">' + args[3] + '</a>';
+        });
     }
 
     constructor(