Feature #18555
Show warning when attempting to attach more than the allowed number of attachments
| Status: | Closed | Start date: | ||
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assignee: |  Go MAEDA | % Done: | 0% | |
| Category: | Attachments | |||
| Target version: | 4.2.0 | |||
| Resolution: | Fixed |
Description
I was just notified by an employee of mine that my ticket was missing some of the attached files that should have been there. I thought I had merely forgot to attach them; however, upon repeating my edit due to needing to remake all the files I learned that redmine has a hard limit of 10 files attached per edit.
Although I'm not sure why this limit exists (I'll be adding another issue to make this a setting or remove it) redmine should definitely alert the end user to the limit both beforehand such as like the the text for max file size is displayed as well as after an attempt to add more than 10 files is added (javascript alert box or something similar).
This is related to issue #8579
This is the code in question: source:/trunk/public/javascripts/attachments.js#L6
remine number of files limit beforehand.png (229 KB)
defect-18555-v1.patch 
(1.97 KB)
ScreenShot.png (200 KB)
defect-18555-v2.patch
(2.04 KB)
defect-18555-v3.patch
(2.08 KB)
Related issues
Associated revisions
Show warning when attempting to attach more than the allowed number of attachments (#18555).
Patch by Mizuki ISHIKAWA.
Update locales (#18555).
History
#1
Updated by Go MAEDA over 5 years ago
- Related to Feature #18556: Add ability to configure limit of attachments at once added
#2
Updated by Go MAEDA over 5 years ago
- Duplicated by Defect #18642: Limited number of simultanous file uploads added
#3
Updated by Mizuki ISHIKAWA 4 months ago
- File defect-18555-v1.patch added
- File ScreenShot.png added
After adding the eleventh file, I attached a patch that gives a javascript alert like the image below.
max-number-of-files-message:
This file cannot be uploaded because the maximum number of files that can be attached at the same time has been reached.
I'm not a native English speaker, so I'd like someone to review the "max-number-of-files-message".
#6
Updated by Mizuki ISHIKAWA 4 months ago
- File defect-18555-v3.patch added
@Marius BALTEANU and @Bernhard Rohloff corrected the English sentence, so I updated the patch based on that.
#8
Updated by Go MAEDA 3 months ago
- Tracker changed from Defect to Feature
- Subject changed from Notify user when attempting to attach more than the allowed number of attachments to Show warning when attempting to attach more than the allowed number of attachments
- Assignee set to Go MAEDA
- Resolution set to Fixed
Committed the patch. Thank you for improving Redmine.