From 7b90680c2dd78d88d747fff545d69b1908ced7ae Mon Sep 17 00:00:00 2001
From: Airat Shigapov <contact@airatshigapov.com>
Date: Wed, 12 Oct 2016 09:51:02 +0300
Subject: [PATCH] Use guard clause instead of if-else statement

---
 .../project_services/hipchat_service.rb       | 25 +++++++++----------
 1 file changed, 12 insertions(+), 13 deletions(-)

diff --git a/app/models/project_services/hipchat_service.rb b/app/models/project_services/hipchat_service.rb
index 8988a7b905e..e7a77070b9f 100644
--- a/app/models/project_services/hipchat_service.rb
+++ b/app/models/project_services/hipchat_service.rb
@@ -128,22 +128,21 @@ class HipchatService < Service
   end
 
   def markdown(text, options = {})
-    if text
-      context = {
-        project: project,
-        pipeline: :email
-      }
+    return "" unless text
 
-      Banzai.render(text, context)
+    context = {
+      project: project,
+      pipeline: :email
+    }
 
-      context.merge!(options)
+    Banzai.render(text, context)
 
-      html = Banzai.render(text, context)
-      html = Banzai.post_process(html, context)
-      sanitize html, attributes: %w(href title alt)
-    else
-      ""
-    end
+    context.merge!(options)
+
+    html = Banzai.render(text, context)
+    html = Banzai.post_process(html, context)
+
+    sanitize html, attributes: %w(href title alt)
   end
 
   def create_issue_message(data)