Replace case with the guard clause

174847b4 · Vitali Tatarintev · d60243ed · 174847b4
Commit 174847b4 authored Sep 13, 2019 by Vitali Tatarintev
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 8 deletions

ee/app/controllers/projects/alerting/notifications_controller.rb ...controllers/projects/alerting/notifications_controller.rb +3 -8

No files found.
--- a/ee/app/controllers/projects/alerting/notifications_controller.rb
+++ b/ee/app/controllers/projects/alerting/notifications_controller.rb
@@ -40,14 +40,9 @@ module Projects
      end

      def response_status(result)
-        case result.http_status
-        when 401
-          :unauthorized
-        when 403
-          :forbidden
-        else
-          :ok
-        end
+        return :ok if result.success?
+
+        result.http_status
      end

      def permitted_params