Patch #22401

Notify the user of missing attachments

Added by Jens Krämer over 1 year ago. Updated over 1 year ago.

Status:ClosedStart date:
Priority:NormalDue date:
Assignee:Jean-Philippe Lang% Done:

0%

Category:Attachments
Target version:3.3.0

Description

Under certain (rare) circumstances it may happen that, when an issue or other
container is saved, added attachments have already been removed (i.e. by the
attachments:prune rake task).

Until now these tokens are silently ignored and the user is left in the dark about the missing file. This patch adds a validation error to the container in this case.

0001-notify-the-user-of-missing-attachments.patch Magnifier (4.14 KB) Jens Krämer, 2016-04-05 15:58

Associated revisions

Revision 15378
Added by Jean-Philippe Lang over 1 year ago

Notify the user of missing attachments (#22401).

under certain (rare) circumstances it may happen that, when an issue or other
container is saved, added attachments have already been removed (i.e. by the
attachments:prune rake task). This patch adds a validation error to the
container in this case.

Patch by Jens Kraemer.

History

#1 Updated by Jan from Planio www.plan.io over 1 year ago

  • Target version set to Candidate for next minor release

#2 Updated by Toshi MARUYAMA over 1 year ago

  • Target version changed from Candidate for next minor release to 3.3.0

#3 Updated by Jean-Philippe Lang over 1 year ago

  • Subject changed from notify the user of missing attachments to Notify the user of missing attachments
  • Category set to Attachments
  • Status changed from New to Closed
  • Assignee set to Jean-Philippe Lang

Committed, thanks. I used the existing warning_attachments_not_saved i18n string instead of adding a new one.

Also available in: Atom PDF