Page MenuHomePhabricator
Differential D12940

Transactions - fix bug adding people to projects
ClosedPublic
Actions

Authored by btrahan on May 19 2015, 10:21 PM.
Tags
None
Referenced Files
F18615543: D12940.diff
Sun, Sep 14, 3:00 PM
F18600171: D12940.diff
Sat, Sep 13, 9:07 AM
F18600073: D12940.diff
Sat, Sep 13, 8:57 AM
F18589017: D12940.diff
Fri, Sep 12, 2:10 AM
F18498024: D12940.diff
Sep 4 2025, 6:23 PM
F17831103: D12940.id.diff
Jul 26 2025, 11:41 AM
Unknown Object (File)
Jun 16 2025, 7:47 AM
Unknown Object (File)
Jun 16 2025, 12:37 AM
View All Files
Subscribers
epriestley
Korvin
Tokens
"Like" token, awarded by epriestley.

Details

Reviewers
epriestley
chad
Maniphest Tasks
T6403: Make application customization of builtin transaction types optional
T8264: Oddness adding people to the Dropbox project
Commits
Restricted Diffusion Commit
rP2154f17b3d7b: Transactions - fix bug adding people to projects
Summary

Fixes T8264. Broken in D12929. Sweep all the applyBuiltin implementations and always break; rather than return

Test Plan

added myself to a project successfully (showed up as a member)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

btrahan updated this revision to Diff 31158.May 19 2015, 10:21 PM
btrahan added a task: T8264: Oddness adding people to the Dropbox project.
btrahan retitled this revision from to Transactions - fix bug adding people to projects.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added reviewers: epriestley, chad.
Herald added subscribers: epriestley, Korvin. · View Herald TranscriptMay 19 2015, 10:21 PM
btrahan added a task: T6403: Make application customization of builtin transaction types optional.May 19 2015, 10:22 PM
Closed by commit rP2154f17b3d7b: Transactions - fix bug adding people to projects (authored by btrahan, committed by btrahan). · Explain WhyMay 19 2015, 10:22 PM
This revision was automatically updated to reflect the committed changes.
epriestley awarded a token.May 19 2015, 11:38 PM

Revision Contents
Changeset List

PathSize
src/
applications/
audit/
editor/
2 lines
differential/
editor/
2 lines
project/
editor/
6 lines

Diff 31159

View Options

src/applications/audit/editor/PhabricatorAuditEditor.php

Loading...
View Options

src/applications/differential/editor/DifferentialTransactionEditor.php

Loading...
View Options

src/applications/project/editor/PhabricatorProjectTransactionEditor.php

Loading...
Phabricator · Built by Phacility