Project

General

Profile

Actions

Feature #18555

closed

Show warning when attempting to attach more than the allowed number of attachments

Added by Stephen Ostrow about 10 years ago. Updated almost 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Attachments
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
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


Files

remine number of files limit beforehand.png (229 KB) remine number of files limit beforehand.png Stephen Ostrow, 2014-12-03 21:36
defect-18555-v1.patch (1.97 KB) defect-18555-v1.patch Mizuki ISHIKAWA, 2020-03-16 05:36
ScreenShot.png (200 KB) ScreenShot.png Mizuki ISHIKAWA, 2020-03-16 05:36
defect-18555-v2.patch (2.04 KB) defect-18555-v2.patch Mizuki ISHIKAWA, 2020-03-24 09:34
defect-18555-v3.patch (2.08 KB) defect-18555-v3.patch Mizuki ISHIKAWA, 2020-03-25 08:57
alerts-appear-multiple-times.gif (507 KB) alerts-appear-multiple-times.gif Mizuki ISHIKAWA, 2021-01-12 05:35
fix-the-number-of-alerts.patch (1.21 KB) fix-the-number-of-alerts.patch Mizuki ISHIKAWA, 2021-01-12 05:37

Related issues

Related to Redmine - Feature #18556: Add ability to configure limit of attachments at onceNew

Actions
Has duplicate Redmine - Defect #18642: Limited number of simultanous file uploadsClosed

Actions
Actions #1

Updated by Go MAEDA about 10 years ago

  • Related to Feature #18556: Add ability to configure limit of attachments at once added
Actions #2

Updated by Go MAEDA about 10 years ago

  • Has duplicate Defect #18642: Limited number of simultanous file uploads added
Actions #3

Updated by Mizuki ISHIKAWA almost 5 years ago

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".

Actions #4

Updated by Mizuki ISHIKAWA almost 5 years ago

I changed the message.

Actions #5

Updated by Go MAEDA almost 5 years ago

  • Target version set to Candidate for next major release
Actions #6

Updated by Mizuki ISHIKAWA almost 5 years ago

Marius Ionescu BALTEANU and Bernhard Ganslmeier Rohloff corrected the English sentence, so I updated the patch based on that.

Actions #7

Updated by Go MAEDA almost 5 years ago

  • Target version changed from Candidate for next major release to 4.2.0

Setting the target version to 4.2.0.

Actions #8

Updated by Go MAEDA almost 5 years 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.

Actions #9

Updated by Go MAEDA almost 5 years ago

  • Status changed from New to Closed
Actions #10

Updated by Mizuki ISHIKAWA almost 4 years ago

With this change, you will now be alerted three times when you attach 13 files at the same time. The alert will be repeated as many times as the limit of 10 is exceeded. ( Related comment: https://www.redmine.org/issues/30776#note-7 )

I think one alerts is enough, so I'll attach a patch to fix it.

Actions #11

Updated by Go MAEDA almost 4 years ago

  • Status changed from Closed to Reopened
Actions #12

Updated by Go MAEDA almost 4 years ago

  • Status changed from Reopened to Closed

Mizuki ISHIKAWA wrote:

With this change, you will now be alerted three times when you attach 13 files at the same time. The alert will be repeated as many times as the limit of 10 is exceeded. ( Related comment: https://www.redmine.org/issues/30776#note-7 )

Fixed in r20711. Thank you.

Actions

Also available in: Atom PDF