Use switch case in a helper, feedback:

https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5142#note_13988401

1 files changed, 3 insertions, 7 deletions

diff --git a/app/controllers/projects/artifacts_controller.rb b/app/controllers/projects/artifacts_controller.rb
index 5cc6d643b64..8261a73c642 100644
--- a/app/controllers/projects/artifacts_controller.rb
+++ b/app/controllers/projects/artifacts_controller.rb
@@ -35,14 +35,10 @@ class Projects::ArtifactsController < Projects::ApplicationController
   end
 
   def latest_succeeded
-    path = params[:path]
+    target_url = artifacts_action_url(params[:path], project, build)
 
-    if %w[download browse file].include?(path)
-      redirect_to send(
-        "#{path}_namespace_project_build_artifacts_url",
-        project.namespace,
-        project,
-        build)
+    if target_url
+      redirect_to(target_url)
     else
       render_404
     end