Merge branch 'rails-logger-cop-18' into 'master'

Use applogger in file mover See merge request gitlab-org/gitlab!32206

Merge branch 'rails-logger-cop-18' into 'master'
Use applogger in file mover See merge request gitlab-org/gitlab!32206
bf774317 · Peter Leitzen · 959c297c · 4b0cc888 · bf774317 · bf774317
Commit bf774317 authored May 21, 2020 by Peter Leitzen
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 1 deletion

app/uploaders/file_mover.rb app/uploaders/file_mover.rb +1 -1

changelogs/unreleased/rails-logger-cop-18.yml changelogs/unreleased/rails-logger-cop-18.yml +5 -0

No files found.
--- a/app/uploaders/file_mover.rb
+++ b/app/uploaders/file_mover.rb
@@ -98,7 +98,7 @@ class FileMover
  end

  def revert
-    Rails.logger.warn("Markdown not updated, file move reverted for #{to_model}") # rubocop:disable Gitlab/RailsLogger
+    Gitlab::AppLogger.warn("Markdown not updated, file move reverted for #{to_model}")

    if temp_file_uploader.file_storage?
      FileUtils.move(file_path, temp_file_path)

--- a/changelogs/unreleased/rails-logger-cop-18.yml
+++ b/changelogs/unreleased/rails-logger-cop-18.yml
+---
+title: Use applogger in file mover file
+merge_request: 32206
+author: Rajendra Kadam
+type: fixed