fix: #7469 missing email logos, occasionally

NodeBB · Mar 25, 2019 · 8d62ead · 8d62ead
1 parent 81306ff
commit 8d62ead
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/src/emailer.js b/src/emailer.js
@@ -162,9 +162,15 @@ Emailer.registerApp = function (expressApp) {
 	// Update default payload if new logo is uploaded
 	pubsub.on('config:update', function (config) {
 		if (config) {
-			Emailer._defaultPayload.logo.src = config['brand:emailLogo'];
-			Emailer._defaultPayload.logo.height = config['brand:emailLogo:height'];
-			Emailer._defaultPayload.logo.width = config['brand:emailLogo:width'];
+			if (config['brand:emailLogo']) {
+				Emailer._defaultPayload.logo.src = config['brand:emailLogo'];
+			}
+			if (config['brand:emailLogo:height']) {
+				Emailer._defaultPayload.logo.height = config['brand:emailLogo:height'];
+			}
+			if (config['brand:emailLogo:width']) {
+				Emailer._defaultPayload.logo.width = config['brand:emailLogo:width'];
+			}
 
 			if (smtpSettingsChanged(config)) {
 				Emailer.setupFallbackTransport(config);