Page MenuHomePhabricator
Differential D21812

Do not extract file PHIDs from remarkup blocks for attachment
ClosedPublic
Actions

Authored by epriestley on May 10 2022, 6:54 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Jun 30, 9:25 PM
Unknown Object (File)
Sun, Jun 15, 10:41 PM
Unknown Object (File)
Sat, Jun 7, 2:23 AM
Unknown Object (File)
May 28 2025, 5:19 PM
Unknown Object (File)
May 25 2025, 5:15 AM
Unknown Object (File)
May 24 2025, 9:51 AM
Unknown Object (File)
May 5 2025, 7:46 PM
Unknown Object (File)
Apr 25 2025, 1:28 AM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
Restricted Maniphest Task
Commits
rP6fea5e5ce7d5: Do not extract file PHIDs from remarkup blocks for attachment
Summary

Ref T13603. When processing transactions that include remarkup blocks, stop extracting file references from them for attachment.

Test Plan

Submitted a remarkup block with a file reference, no longer saw automatic file attachment.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley requested review of this revision.May 10 2022, 6:54 PM
epriestley created this revision.
Harbormaster completed remote builds in B25709: Diff 51989.May 10 2022, 6:54 PM
This revision was not accepted when it landed; it landed in state Needs Review.May 19 2022, 8:21 PM
Closed by commit rP6fea5e5ce7d5: Do not extract file PHIDs from remarkup blocks for attachment (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rP6fea5e5ce7d5: Do not extract file PHIDs from remarkup blocks for attachment.

Revision Contents
Changeset List

Diff 52021

View Options

src/applications/transactions/editor/PhabricatorApplicationTransactionCommentEditor.php

Loading...
View Options

src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php

Loading...
Phabricator · Built by Phacility