Page MenuHomePhabricator

Support "Assign To" in Maniphest bulk editor
ClosedPublic

Authored by epriestley on Jan 19 2018, 1:18 PM.
Tags
None
Referenced Files
F15470940: D18874.id45285.diff
Sat, Apr 5, 2:16 AM
F15436876: D18874.id45285.diff
Tue, Mar 25, 4:42 PM
F15416304: D18874.id45285.diff
Thu, Mar 20, 10:27 AM
F15416193: D18874.id45266.diff
Thu, Mar 20, 9:48 AM
F15411718: D18874.id.diff
Wed, Mar 19, 9:37 AM
F15408746: D18874.diff
Wed, Mar 19, 1:04 AM
F15391598: D18874.id45285.diff
Sat, Mar 15, 10:42 AM
F15385446: D18874.id45285.diff
Fri, Mar 14, 10:37 PM
Subscribers
None

Details

Summary

Ref T13025. See PHI173. This supports the "Assign to" field in the new editor.

This is very slightly funky: to unassign tasks, you need to leave the field blank. I have half a diff to fix this, but the way the none() token works in the default datasource is odd so it needs a separate datasource. I'm punting on this for now since it works, at least, and isn't completely unreasonable.

This also simplifies some EditEngine stuff a little. Notably:

  • I reorganized EditType construction slightly so subclasses can copy/paste a little bit less.
  • EditType had field and editField properties which had the same values. I canonicalized on editField and made this value set a little more automatically.
Test Plan

Used bulk editor to reassign some tasks. By leaving the field blank, unassigned tasks.

Diff Detail

Repository
rP Phabricator
Branch
bulk9
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 19050
Build 25705: Run Core Tests
Build 25704: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Jan 19 2018, 7:37 PM
This revision was automatically updated to reflect the committed changes.