diff --git a/app/models/attachment.rb b/app/models/attachment.rb
index c641f3006..5af82a2e3 100644
--- a/app/models/attachment.rb
+++ b/app/models/attachment.rb
@@ -44,7 +44,7 @@ class Attachment < ActiveRecord::Base
                                                         "LEFT JOIN #{Project.table_name} ON #{Document.table_name}.project_id = #{Project.table_name}.id"}
 
   cattr_accessor :storage_path
-  @@storage_path = Redmine::Configuration['attachments_storage_path'] || "#{Rails.root}/files"
+  @@storage_path = Redmine::Configuration['attachments_storage_path'] || File.join(Rails.root, "files")
 
   before_save :files_to_final_location
   after_destroy :delete_from_disk
@@ -125,7 +125,7 @@ class Attachment < ActiveRecord::Base
 
   # Returns file's location on disk
   def diskfile
-    "#{@@storage_path}/#{self.disk_filename}"
+    File.join(self.class.storage_path, disk_filename.to_s)
   end
 
   def increment_download