PKP Bugzilla – Bug 6613
Add option to delete supplementary file
Last modified: 2012-08-27 21:37:59 PDT
Currently, supplementary files can be deleted from submission Editing -> Layout Editing. There should also be an option to delete individual supplementary files from the submission summary page as well, as there is supplementary file information there (both add and edit functions) and it is not immediately clear that the delete function exists, but on a separate page.
The more I'm looking at where exactly the template for the submission summary is (it is remarkably well-hidden), the more I get the feeling that the problem may not so much be that there isn't a "delete" option in the summary page, but that there are "edit" and "add" options that are better left off - usually a summary is associated with "this is the information as it is known", and then for editing the actual properties you'd use whichever option points to editing. Of course I can still add in the "delete" option if that's the preferred solution.
Created attachment 3537 [details]
adds a "delete" option to a submission summary
This is a patch for the templates/sectionEditor/submission/management.tpl file (which may need renaming at some point, since it's the summary page. The summary.tpl file is actually the template for the summary section on the "EDITING" page), adding a "delete" operation to the already existing "edit" and "add" for supplementary files.
Looks like there's still some indentation weirdness in the patch.
Created attachment 3542 [details]
removed superfluous tabs
Hi Mike, I tested the patch and it looks good. I do still think that adding a "delete" option on the summary page makes sense. Eventually, it would be worth looking at allowing supplementary files to exist in the system without necessarily being automatically made available via the RT on an individual basis (it is currently possible to disable links supplementary file across the board in the RT), but that's another bug report.
*** This bug has been marked as a duplicate of bug 6740 ***