diff options
author | Rémy Coutable <remy@rymai.me> | 2018-06-19 11:36:57 +0300 |
---|---|---|
committer | Rémy Coutable <remy@rymai.me> | 2018-06-19 11:36:57 +0300 |
commit | 719e0dc39e8b333c51a7cf504ef736eb31c085cb (patch) | |
tree | 6dae6ed2576ded8d106f0e0ef28f30fb514c953f /app/controllers | |
parent | cc4bb691af07d134e80d20282dd0f07b7ed4192e (diff) | |
parent | 4f76d953ddb44babe87c74be7e88018bd2e9aaa3 (diff) |
Merge branch 'rails5-fix-46276' into 'master'
Rails5 fix format in uploads actions
Closes #46276
See merge request gitlab-org/gitlab-ce!19907
Diffstat (limited to 'app/controllers')
-rw-r--r-- | app/controllers/concerns/uploads_actions.rb | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/app/controllers/concerns/uploads_actions.rb b/app/controllers/concerns/uploads_actions.rb index 170bca8b56f..16374146ae4 100644 --- a/app/controllers/concerns/uploads_actions.rb +++ b/app/controllers/concerns/uploads_actions.rb @@ -1,9 +1,15 @@ module UploadsActions + extend ActiveSupport::Concern + include Gitlab::Utils::StrongMemoize include SendFileUpload UPLOAD_MOUNTS = %w(avatar attachment file logo header_logo favicon).freeze + included do + prepend_before_action :set_html_format, only: :show + end + def create link_to_file = UploadService.new(model, params[:file], uploader_class).execute @@ -41,6 +47,13 @@ module UploadsActions private + # Explicitly set the format. + # Otherwise rails 5 will set it from a file extension. + # See https://github.com/rails/rails/commit/84e8accd6fb83031e4c27e44925d7596655285f7#diff-2b8f2fbb113b55ca8e16001c393da8f1 + def set_html_format + request.format = :html + end + def uploader_class raise NotImplementedError end |