summaryrefslogtreecommitdiff
path: root/app
diff options
context:
space:
mode:
authorJames Lopez <james@jameslopez.es>2018-09-07 13:42:49 +0200
committerJames Lopez <james@jameslopez.es>2018-09-07 13:42:49 +0200
commit27442862cf81881c81552e64d0b4d3354535c45e (patch)
treea05e89cdad1f455971748b74f4092ca3992de8b8 /app
parenta2aa505805478540b3150b2f9093c8658e18597d (diff)
downloadgitlab-ce-27442862cf81881c81552e64d0b4d3354535c45e.tar.gz
refactor code based on feedback
Diffstat (limited to 'app')
-rw-r--r--app/controllers/projects_controller.rb4
-rw-r--r--app/models/project.rb12
2 files changed, 10 insertions, 6 deletions
diff --git a/app/controllers/projects_controller.rb b/app/controllers/projects_controller.rb
index 646d8329aff..f4049bbfa46 100644
--- a/app/controllers/projects_controller.rb
+++ b/app/controllers/projects_controller.rb
@@ -191,8 +191,8 @@ class ProjectsController < Projects::ApplicationController
end
def download_export
- if @project.export_project_object_exists?
- send_upload(@project.import_export_upload.export_file)
+ if @project.export_project_exists?
+ send_upload(@project.export_file)
else
redirect_to(
edit_project_path(@project, anchor: 'js-export-project'),
diff --git a/app/models/project.rb b/app/models/project.rb
index 35c93aec20a..60d3e99252f 100644
--- a/app/models/project.rb
+++ b/app/models/project.rb
@@ -1738,7 +1738,7 @@ class Project < ActiveRecord::Base
:started
elsif after_export_in_progress?
:after_export_action
- elsif export_project_object_exists?
+ elsif export_project_exists?
:finished
else
:none
@@ -1754,14 +1754,18 @@ class Project < ActiveRecord::Base
end
def remove_exports
- return unless export_project_object_exists?
+ return unless export_project_exists?
import_export_upload.remove_export_file!
import_export_upload.save
end
- def export_project_object_exists?
- import_export_upload&.export_file&.file
+ def export_project_exists?
+ export_file&.file
+ end
+
+ def export_file
+ import_export_upload&.export_file
end
def full_path_slug