Page MenuHomePhabricator

Reuse more transaction construction code in bulk editor
ClosedPublic

Authored by epriestley on Jan 19 2018, 1:53 PM.
Tags
None
Referenced Files
F19034738: D18875.diff
Tue, Nov 25, 2:48 PM
F18906839: D18875.diff
Sat, Nov 8, 8:36 PM
F18876568: D18875.diff
Thu, Nov 6, 1:50 AM
F18876567: D18875.diff
Thu, Nov 6, 1:50 AM
F18814634: D18875.diff
Oct 20 2025, 11:38 PM
F18785760: D18875.diff
Oct 14 2025, 11:15 AM
F18767669: D18875.id.diff
Oct 7 2025, 11:42 PM
F18758065: D18875.diff
Oct 5 2025, 8:39 PM
Subscribers
None

Details

Summary

Ref T13025. Currently, the bulk editor takes an HTTP request and emits a list of "raw" transactions (simple dictionaries). This goes into the job queue, and the background job builds a real transaction.

However, the logic to turn an HTTP request into a raw transaction is ending up with some duplication, since we generally already have logic to turn an HTTP request into a full object.

Instead: build real objects first, then serialize them to dictionaries. Send those to the job queue, rebuild them into objects again, and we end up in the same spot with a little less code duplication.

Finally, delete the mostly-copied code.

Test Plan

Used bulk editor to add comments, projects, and rename tasks.

Diff Detail

Repository
rP Phabricator
Branch
bulk10
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 19051
Build 25707: Run Core Tests
Build 25706: arc lint + arc unit