Make conditional a guard clause

77ca4a67 · Alfredo Sumaran · Jacob Schatz · bc38bfad · 77ca4a67
Commit 77ca4a67 authored May 18, 2016 by Alfredo Sumaran Committed by Jacob Schatz May 25, 2016
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 5 deletions

app/helpers/todos_helper.rb app/helpers/todos_helper.rb +5 -5

No files found.
--- a/app/helpers/todos_helper.rb
+++ b/app/helpers/todos_helper.rb
@@ -38,11 +38,11 @@ module TodosHelper
  end

  def todo_target_state_pill(todo)
-    if show_todo_state?(todo)
-      content_tag(:span, nil, class: 'target-status') do
-        content_tag(:span, nil, class: "status-box status-box-#{todo.target.state.dasherize}") do
-          todo.target.state.capitalize
-        end
+    return unless show_todo_state?(todo)
+
+    content_tag(:span, nil, class: 'target-status') do
+      content_tag(:span, nil, class: "status-box status-box-#{todo.target.state.dasherize}") do
+        todo.target.state.capitalize
      end
    end
  end