From a720efdbd0f2d0dcab3f4df93718f5ba52c56239 Mon Sep 17 00:00:00 2001
From: barredterra <14891507+barredterra@users.noreply.github.com>
Date: Mon, 8 Feb 2021 12:51:47 +0100
Subject: [PATCH] fix: signature should be an empty string by default

(would become undefined if the server message was empty)
---
 frappe/public/js/frappe/views/communication.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/frappe/public/js/frappe/views/communication.js b/frappe/public/js/frappe/views/communication.js
index 0c294d5869..6e70284619 100755
--- a/frappe/public/js/frappe/views/communication.js
+++ b/frappe/public/js/frappe/views/communication.js
@@ -726,7 +726,7 @@ frappe.views.CommunicationComposer = Class.extend({
 
 		if (!signature) {
 			const res = await this.get_default_outgoing_email_account_signature();
-			signature = "<!-- signature-included -->" + res.message.signature;
+			signature = res.message.signature ? "<!-- signature-included -->" + res.message.signature : "";
 		}
 
 		if (signature && !frappe.utils.is_html(signature)) {