Page MenuHomePhabricator
Differential D17610

Move Files to ModularTransactions
ClosedPublic
Actions

Authored by epriestley on Apr 4 2017, 2:47 PM.
Tags
None
Referenced Files
F19225939: D17610.diff
Thu, Dec 18, 5:33 AM
F19034347: D17610.id.diff
Tue, Nov 25, 2:10 PM
F19027033: D17610.diff
Mon, Nov 24, 1:07 PM
F18827705: D17610.id42362.diff
Oct 24 2025, 11:35 AM
F18824465: D17610.id.diff
Oct 23 2025, 3:51 PM
F18820190: D17610.diff
Oct 22 2025, 1:08 PM
F18750213: D17610.diff
Oct 4 2025, 6:57 AM
F18666680: D17610.diff
Sep 24 2025, 8:05 AM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T11357: Move Files to EditEngine and modern APIs
Commits
rP8500f78e451c: Move Files to ModularTransactions
Summary

Ref T11357. A lot of file creation doesn't go through transactions, so we only actually have one real transaction type: editing a file name.

Test Plan

Created and edited files.

Screen Shot 2017-04-04 at 7.45.24 AM.png (121×588 px, 14 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Apr 4 2017, 2:47 PM
Harbormaster completed remote builds in B16281: Diff 42354.Apr 4 2017, 2:48 PM
epriestley added inline comments.Apr 4 2017, 2:52 PM
src/applications/badges/xaction/PhabricatorBadgesBadgeNameTransaction.php
46

This is a minor fix: newRequiredError() is ignored if the Editor has setContinueOnMissingFields(true) set, but newInvalidError() is always raised.

chad accepted this revision.Apr 4 2017, 5:05 PM
This revision is now accepted and ready to land.Apr 4 2017, 5:05 PM
Closed by commit rP8500f78e451c: Move Files to ModularTransactions (authored by epriestley, committed by epriestley). · Explain WhyApr 4 2017, 5:25 PM
This revision was automatically updated to reflect the committed changes.
epriestley mentioned this in T12524: Modernize Countdown.Apr 9 2017, 10:42 AM

Revision Contents
Changeset List

PathSize
src/
6 lines
applications/
badges/
xaction/
2 lines
files/
controller/
4 lines
editor/
67 lines
storage/
69 lines
__tests__/
2 lines
xaction/
4 lines
files/
xaction/
badges/
xaction/
PhabricatorFileNameTransaction.php
PhabricatorBadgesBadgeNameTransaction.php
20 lines

Diff 42362

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/badges/xaction/PhabricatorBadgesBadgeNameTransaction.php

Loading...
View Options

src/applications/files/controller/PhabricatorFileEditController.php

Loading...
View Options

src/applications/files/editor/PhabricatorFileEditor.php

Loading...
View Options

src/applications/files/storage/PhabricatorFileTransaction.php

Loading...
View Options

src/applications/files/storage/__tests__/PhabricatorFileTestCase.php

Loading...
View Options

src/applications/files/xaction/PhabricatorFileNameTransaction.php

Loading...
View Options

src/applications/files/xaction/PhabricatorFileTransactionType.php

Loading...
Phabricator · Built by Phacility