# Conflicts: # app/models/issue.rb # app/models/journal.rbpull/342/head
| @@ -224,6 +224,9 @@ class AttachmentsController < ApplicationController | |||||
| elsif @file.container.is_a?(Journal) | elsif @file.container.is_a?(Journal) | ||||
| project = @file.container.issue.project | project = @file.container.issue.project | ||||
| candown = project.is_public || (current_user.logged? && project.member?(current_user)) | 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 | else | ||||
| project = nil | project = nil | ||||
| end | end | ||||
| @@ -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\/.+\"/).map{|s|s.match(/\d+/)[0]} | ||||
| att_ids += self.description.to_s.scan(/\/api\/attachments\/\d+/).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? | 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 | ||||
| end | end | ||||
| def to_builder | |||||
| def to_builder | |||||
| Jbuilder.new do |issue| | Jbuilder.new do |issue| | ||||
| issue.(self, :id, :project_issues_index, :subject, :description, :branch_name, :start_date, :due_date) | 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") | issue.created_at self.created_on.strftime("%Y-%m-%d %H:%M") | ||||
| @@ -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\/.+\"/).map{|s|s.match(/\d+/)[0]} | ||||
| att_ids += self.notes.to_s.scan(/\/api\/attachments\/\d+/).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? | 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 | ||||
| end | end | ||||
| def operate_content | |||||
| def operate_content | |||||
| content = "" | content = "" | ||||
| detail = self.journal_details.take | detail = self.journal_details.take | ||||
| case detail.property | case detail.property | ||||