Fix RuboCop offense Lint/NonAtomicFileOperation (#36919).

git-svn-id: https://svn.redmine.org/redmine/trunk@21694 e93f8b46-1217-0410-a6f0-8f06a7374b81

diff --git a/app/models/attachment.rb b/app/models/attachment.rb
index 7c60c7164422e0b24620b2ef2469bec6bc8860c0..dd4c6d86f8a18871308cad2edb356222efd317d5 100644 (file)
--- a/app/models/attachment.rb
+++ b/app/models/attachment.rb
@@ -525,9 +525,7 @@ class Attachment < ActiveRecord::Base
 
   # Physically deletes the file from the file system
   def delete_from_disk!
-    if disk_filename.present? && File.exist?(diskfile)
-      File.delete(diskfile)
-    end
+    FileUtils.rm_f(diskfile) if disk_filename.present?
     Dir[thumbnail_path("*")].each do |thumb|
       File.delete(thumb)
     end

diff --git a/lib/redmine/thumbnail.rb b/lib/redmine/thumbnail.rb
index 3cd978c8e66af290ed759c7f951c78054fcb495b..2f9b41ce88f54250f2635b9aabdbd60161e11ea3 100644 (file)
--- a/lib/redmine/thumbnail.rb
+++ b/lib/redmine/thumbnail.rb
@@ -43,9 +43,7 @@ module Redmine
         return nil if is_pdf && mime_type != "application/pdf"
 
         directory = File.dirname(target)
-        unless File.exist?(directory)
-          FileUtils.mkdir_p directory
-        end
+        FileUtils.mkdir_p directory
         size_option = "#{size}x#{size}>"
 
         if is_pdf

diff --git a/lib/redmine/utils.rb b/lib/redmine/utils.rb
index c5431123928d798eb241876099b1e34841d93703..55fbbbfb3405db6cc897e106802e5fb01088585d 100644 (file)
--- a/lib/redmine/utils.rb
+++ b/lib/redmine/utils.rb
@@ -51,9 +51,7 @@ module Redmine
 
       def save_upload(upload, path)
         directory = File.dirname(path)
-        unless File.exist?(directory)
-          FileUtils.mkdir_p directory
-        end
+        FileUtils.mkdir_p directory
         File.open(path, "wb") do |f|
           if upload.respond_to?(:read)
             buffer = ""