use case when for computing the corresponding application status #242

hpi-swt2 · Jan 24, 2017 · e82d809 · e82d809
1 parent 65d1d57
commit e82d809
Showing 1 changed file with 8 additions and 4 deletions.
diff --git a/app/controllers/emails_controller.rb b/app/controllers/emails_controller.rb
@@ -64,10 +64,14 @@ def get_email_template_status
   end
 
   def get_corresponding_application_letter_status
-    return :accepted if params[:status] == "acceptance"
-    return :rejected if params[:status] == "rejection"
-    # default value
-    return :accepted
+    case params[:status]
+    when "acceptance"
+      return :accepted
+    when "rejection"
+      return :rejected
+    else
+      return :accepted
+    end
   end
 
   # Only allow a trusted parameter "white list" through.