Feature #10282
Copy wiki attachments on project copy
| Status: | Closed | Start date: | ||
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assignee: |  Go MAEDA | % Done: | 0% | |
| Category: | Projects | |||
| Target version: | 4.0.0 | |||
| Resolution: | Fixed |
Description
When copying a project to a new project, the Wiki pages are copied, but the attachments are not. The attachments on the wiki pages are linked to the previous project, and if, for example, the users accessing the new project have no access to the original project, the images return a 403 error.
add_wiki_attachments_to_copy.patch 
(1.63 KB)
Related issues
Associated revisions
Copy wiki attachments on project copy (#10282).
Patch by Mizuki ISHIKAWA.
History
#1
Updated by Jean-Philippe Lang over 6 years ago
- Tracker changed from Defect to Feature
- Subject changed from Attachment permission when copying projects to Copy wiki attachments on project copy
- Category set to Projects
#2
Updated by Go MAEDA 5 months ago
- Related to Patch #26622: Copy version attachments (i.e. Files) along with the versions on project copy added
#4
Updated by Mizuki ISHIKAWA 5 months ago
- File add_wiki_attachments_to_copy.patch added
When you copy a project, Issue is copied together with the attached file.
I think that wiki attachments should be copied just like Issue.
By applying the following patch, you can copy the attached file of the wiki.
#5
Updated by Go MAEDA 5 months ago
Mizuki, thank you for submitting the patch. I tried out your patch and it works fine for me.
I think it is natural that attachments are copied along with wiki pages. In many cases, the content of Wiki page and its attachments are inseparable because attachments are often used to display inline images on the page. The page does not make sense without those images.
#7
Updated by Go MAEDA about 1 month ago
I think the test code should also check digest and disk_filename attributes to ensure that both attachments are identical and point to the same physical file.
@Mizuki, could you update the test code?
#8
Updated by Go MAEDA about 1 month ago
Go MAEDA wrote:
I think the test code should also check
digestanddisk_filenameattributes to ensure that both attachments are identical and point to the same physical file.
Sorry, you don't have to add such tests because no other tests which copy attachments in test/unit/project_copy_test.rb check those attributes.
#9
Updated by Go MAEDA about 1 month ago
- Status changed from New to Closed
- Assignee set to Go MAEDA
- Target version changed from 4.1.0 to 4.0.0
- Resolution set to Fixed
Committed. Thank you for improving Redmine.