PKP Bugzilla – Bug 6408
Break ArticleFile > SuppFile inheritance hierarchy
Last modified: 2012-09-21 14:21:26 PDT
Supplementary files and SubmissionFile both implement get/setType with different semantic meanings.
Supplementary Files (a logical entity) should not inherit from ArticleFile (a concrete FS resource).
For reference: a few ideas in gliffy (to be ported to the Wiki, too): http://www.gliffy.com/gliffy/#d=2317453&t=SubmissionFilesGridHandler (scroll down to see the current file implementations across all apps and a few comments/ideas for potential improvements - all of them not very well thought out yet, just brainstorming).
@Alec: What do you mean by "logical entity"? Aren't supp files the files uploaded during submission as "attachments" to the main article? These would then also be concrete FS resources that go through editing and can be downloaded from the article page... I'm not sure whether I understand that distinction correctly, though. I like the OMP way in this case where all submission files are essentially the same in the editing process only distinguished by genre. Could that be a pattern to be applied to OJS submission files too? Then we could get rid of the distinction between ArticleFile and SuppFile...