Page MenuHomePhabricator
Differential D21834

Provide missing default attachment list for Files transactions
ClosedPublic
Actions

Authored by epriestley on May 23 2022, 4:24 PM.
Tags
None
Referenced Files
F13292436: D21834.diff
Wed, Jun 5, 3:20 AM
F13270408: D21834.id52036.diff
Wed, May 29, 9:38 AM
F13270407: D21834.id52035.diff
Wed, May 29, 9:38 AM
F13270405: D21834.id.diff
Wed, May 29, 9:38 AM
F13266782: D21834.diff
Tue, May 28, 7:28 PM
F13262533: D21834.diff
Mon, May 27, 2:13 AM
F13245819: D21834.diff
Thu, May 23, 6:16 AM
F13198320: D21834.diff
Mon, May 13, 5:45 AM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13682: Improve Remarkup file reference attachment behaviors after removal of "attach by default"
Commits
rP8cd02e6727e3: Provide missing default attachment list for Files transactions
Summary

Ref T13682. This property may not exist, and should defualt to an empty array if not present.

Test Plan

Created a new object.

Diff Detail

Repository
rP Phabricator
Branch
filex1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 25736
Build 35564: arc lint + arc unit

Event Timeline

epriestley requested review of this revision.May 23 2022, 4:24 PM
epriestley created this revision.
Harbormaster completed remote builds in B25736: Diff 52035.May 23 2022, 4:24 PM
This revision was not accepted when it landed; it landed in state Needs Review.May 23 2022, 4:24 PM
Closed by commit rP8cd02e6727e3: Provide missing default attachment list for Files transactions (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rP8cd02e6727e3: Provide missing default attachment list for Files transactions.

Revision Contents
Changeset List

PathSize
src/
applications/
transactions/
editor/
2 lines

Diff 52035

View Options

src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php

Loading...
Phabricator · Built by Phacility