Fixed: Sending Slack notifications without an icon

8 years ago · 197febe9db
parent e10717f6bb
commit 197febe9db
1 changed files with 10 additions and 7 deletions
--- a/src/NzbDrone.Core/Notifications/Slack/Slack.cs
+++ b/src/NzbDrone.Core/Notifications/Slack/Slack.cs
@ -115,14 +115,17 @@ namespace NzbDrone.Core.Notifications.Slack
                Attachments = attachments
            };

-            // Set the correct icon based on the value
-            if (icon.StartsWith(":") && icon.EndsWith(":"))
+            if (icon.IsNotNullOrWhiteSpace())
            {
-                payload.IconEmoji = icon; 
-            }
-            else if (icon.IsNotNullOrWhiteSpace())
-            {
-                payload.IconUrl = icon;
+                // Set the correct icon based on the value
+                if (icon.StartsWith(":") && icon.EndsWith(":"))
+                {
+                    payload.IconEmoji = icon;
+                }
+                else
+                {
+                    payload.IconUrl = icon;
+                }
            }

            return payload;