Redmine

This could easily work for file directly attached to the project.

For files attached to versions, we can only copy them if the versions are also copied to have a valid reference. Besides this not fitting well into the existing code, we would then have to find a way to handle people opting to copy files but not versions.

The simplest solution could be do not copy version-attached files in that case (which would probably be surprising for people not being accustomed to the details of Redmine's data model). Alternatively, we could also try to enforce the selection on the UI with javascript (but would still have to handle this somehow on the server anyway). In any case, I'm afraid this gets out of hand pretty quickly implementation-wise...

Maybe, we can just improve the wording so show

[ ] Versions (2) and Files

instead of only

[ ] Versions (2)

today.

What do you think?

I think it is confusing to always copy files along with versions for ordinary users because many users don't know the data structure of Redmine.

Besides, it is very problematic when there are dozens of files in the Files module and the user don't want to copy those files. The user have to click the trash icon dozens of times to delete all files after copying a project.

In my opinion, we should implement a feature to delete multiple files at once in Files module when this patch is merged.

Go MAEDA wrote:

In my opinion, we should implement a feature to delete multiple files at once in Files module when this patch is merged.

Absolutely, that in itself is a good idea, regardless of whether this patch is merged or not.

Still, right now there is no way to copy the files along with a project. To maintain consistency with all the other objects being copied, we should make sure to also copy attachments in all cases along with their parent objects.

Given that the most common use-case for copying projects is to create a new copy of a template project (rather than random copies of existing heavily-used projects), I think it's not that suprising that files are also copied along. Yes, the data structure is non-obvious to most users (along with the general distinction between "Files" and "Documents"), hence my proposal to change the wording to make it at least obvious what is happening when the checkbox is enabled.