diff --git a/app/controllers/attachments_controller.rb b/app/controllers/attachments_controller.rb
index cfce8b7a7..2bbccb495 100644
--- a/app/controllers/attachments_controller.rb
+++ b/app/controllers/attachments_controller.rb
@@ -224,6 +224,9 @@ class AttachmentsController < ApplicationController
         elsif @file.container.is_a?(Journal)
           project = @file.container.issue.project
           candown = project.is_public || (current_user.logged? && project.member?(current_user))
+        elsif @file.container.is_a?(Project)
+          project = @file.container
+          candown = project.is_public || (current_user.logged? && project.member?(current_user))
         else
           project = nil
         end
diff --git a/app/models/issue.rb b/app/models/issue.rb
index fff3e55d5..c6f1a74bb 100644
--- a/app/models/issue.rb
+++ b/app/models/issue.rb
@@ -228,11 +228,11 @@ class Issue < ApplicationRecord
     att_ids += self.description.to_s.scan(/\/api\/attachments\/.+\"/).map{|s|s.match(/\d+/)[0]}
     att_ids += self.description.to_s.scan(/\/api\/attachments\/\d+/).map{|s|s.match(/\d+/)[0]}
     if att_ids.present?
-      Attachment.where(id: att_ids).where(container_type: nil).update_all(container_id: self.id, container_type: self.class.name)
+      Attachment.where(id: att_ids).where("container_type IS NULL OR container_type = 'Issue'").update_all(container_id: self.project_id, container_type: "Project")
     end
   end
 
-  def to_builder 
+  def to_builder
     Jbuilder.new do |issue|
       issue.(self, :id, :project_issues_index, :subject, :description, :branch_name, :start_date, :due_date)
       issue.created_at self.created_on.strftime("%Y-%m-%d %H:%M")
diff --git a/app/models/journal.rb b/app/models/journal.rb
index 3229ae886..e4a7dcf9a 100644
--- a/app/models/journal.rb
+++ b/app/models/journal.rb
@@ -67,11 +67,11 @@ class Journal < ApplicationRecord
     att_ids += self.notes.to_s.scan(/\/api\/attachments\/.+\"/).map{|s|s.match(/\d+/)[0]}
     att_ids += self.notes.to_s.scan(/\/api\/attachments\/\d+/).map{|s|s.match(/\d+/)[0]}
     if att_ids.present?
-      Attachment.where(id: att_ids).where(container_type: nil).update_all(container_id: self.id, container_type: self.class.name)
+      Attachment.where(id: att_ids).where("container_type IS NULL OR container_type = 'Journal'").update_all(container_id: self.issue.project_id, container_type: "Project")
     end
   end
 
-  def operate_content 
+  def operate_content
     content = ""
     detail = self.journal_details.take
     case detail.property