From d2c9e860cb7f3bedd32c73395c735cbad92dd4a8 Mon Sep 17 00:00:00 2001
From: djmaze <djmaze@djmaze.lan>
Date: Tue, 22 Sep 2020 14:17:48 +0200
Subject: [PATCH] Bugfix: mail?to= should be encodeURIComponent()

---
 dev/Model/Email.js | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/dev/Model/Email.js b/dev/Model/Email.js
index ac1adb1b8..115cbe353 100644
--- a/dev/Model/Email.js
+++ b/dev/Model/Email.js
@@ -365,13 +365,11 @@ class EmailModel {
 					? '<a href="mailto:' +
 					  encodeHtml(this.email) +
 					  '?to=' +
-					  encodeHtml('"' + this.name + '" <' + this.email + '>') +
+					  encodeURIComponent('"' + this.name + '" <' + this.email + '>') +
 					  '" target="_blank" tabindex="-1">' +
 					  encodeHtml(this.name) +
 					  '</a>'
-					: useEncodeHtml
-					? encodeHtml(this.name)
-					: this.name;
+					: (useEncodeHtml ? encodeHtml(this.name) : this.name);
 				// result = wrapWithLink ? '<a href="mailto:' + encodeHtml('"' + this.name + '" <' + this.email + '>') +
 				// 	'" target="_blank" tabindex="-1">' + encodeHtml(this.name) + '</a>' : (useEncodeHtml ? encodeHtml(this.name) : this.name);
 			} else {
@@ -383,7 +381,7 @@ class EmailModel {
 							'<a href="mailto:' +
 							encodeHtml(this.email) +
 							'?to=' +
-							encodeHtml('"' + this.name + '" <' + this.email + '>') +
+							encodeURIComponent('"' + this.name + '" <' + this.email + '>') +
 							'" target="_blank" tabindex="-1">' +
 							encodeHtml(result) +
 							'</a>' +