Page MenuHomePhabricator
Differential D13213

Implement PhabricatorApplicationTransactionInterface on PhabricatorUser
ClosedPublic
Actions

Authored by epriestley on Jun 8 2015, 8:00 PM.
Tags
None
Referenced Files
F15402767: D13213.id.diff
Tue, Mar 18, 12:15 AM
F15390411: D13213.id31938.diff
Sat, Mar 15, 6:11 AM
F15390110: D13213.id.diff
Sat, Mar 15, 5:55 AM
F15385324: D13213.diff
Fri, Mar 14, 10:15 PM
F15384489: D13213.id31939.diff
Fri, Mar 14, 8:03 PM
F15301700: D13213.diff
Wed, Mar 5, 6:07 PM
Unknown Object (File)
Sat, Mar 1, 8:58 PM
Unknown Object (File)
Fri, Feb 28, 9:02 AM
View All Files
Subscribers
epriestley

Details

Reviewers
btrahan
Commits
Restricted Diffusion Commit
rP3bb7f3624fe9: Implement PhabricatorApplicationTransactionInterface on PhabricatorUser
Summary

Right now, "Publish" workers for user profile edits (title / blub) can get gummed up in the daemons. Implement the interfaces and provide a Query so they can go through.

Test Plan
  • Made a profile "Title" edit.
  • Used bin/worker execute --id <id> to see task fail.
  • Applied patch.
  • Saw task work.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 31938.Jun 8 2015, 8:00 PM
epriestley retitled this revision from to Implement PhabricatorApplicationTransactionInterface on PhabricatorUser.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
Herald added a subscriber: epriestley. · View Herald TranscriptJun 8 2015, 8:00 PM
btrahan accepted this revision.Jun 8 2015, 8:12 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Jun 8 2015, 8:12 PM
Closed by commit rP3bb7f3624fe9: Implement PhabricatorApplicationTransactionInterface on PhabricatorUser (authored by epriestley, committed by epriestley). · Explain WhyJun 8 2015, 8:36 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
3 lines
applications/
people/
query/
10 lines
storage/
25 lines

Diff 31939

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/people/query/PhabricatorPeopleTransactionQuery.php

Loading...
View Options

src/applications/people/storage/PhabricatorUser.php

Loading...
Phabricator · Built by Phacility