Page MenuHomePhabricator

Only attach files that are both referenced in Remarkup and attached by explicit metadata
ClosedPublic

Authored by epriestley on May 20 2022, 5:48 PM.
Tags
None
Referenced Files
F13321311: D21832.diff
Fri, Jun 14, 6:43 AM
F13302056: D21832.id52042.diff
Sat, Jun 8, 4:17 AM
F13286250: D21832.diff
Tue, Jun 4, 6:53 AM
F13278983: D21832.diff
Sat, Jun 1, 8:45 AM
F13273146: D21832.diff
Fri, May 31, 1:37 AM
F13270401: D21832.id52042.diff
Wed, May 29, 9:37 AM
F13270399: D21832.id52033.diff
Wed, May 29, 9:37 AM
F13270398: D21832.id.diff
Wed, May 29, 9:37 AM
Subscribers
None

Details

Summary

Ref T13682. When a user uploads a file, then changes their mind and deletes the reference to the file, we don't actually want to attach the file.

When choosing which files to attach, only attach files which are both referenced in Remarkup and explicitly attached in remarkup metadata.

Test Plan
  • Dropped a file into a comment, submitted it, saw it attach normally.
  • Dropped a file into a comment, deleted the reference, submitted it, saw no attachment.

Diff Detail

Repository
rP Phabricator
Branch
file2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 25734
Build 35562: arc lint + arc unit

Event Timeline

epriestley created this revision.
This revision was not accepted when it landed; it landed in state Needs Review.May 23 2022, 9:08 PM
This revision was automatically updated to reflect the committed changes.