Homec4science

Maniphest - move subpriority edits to be transaction powered

Authored by Bob Trahan <bob.trahan@gmail.com> on Feb 27 2014, 18:39.

Description

Maniphest - move subpriority edits to be transaction powered

Summary:
...this was nice to do for boards, since this diff also starts calling this code in the board move case. The big trick is to use the new expandTransactions hook to expand the subpriority transaction into a priority transaction if pertinent. The other stuff is just about hiding these new subpriority extractions.

...also removes the "edit" UI from the default board since we can't actually edit anything and it thus is buggy.

Ref T4422. Next step is to move board edits into the editor with their own little transaction.

Test Plan: re-orded tasks on a maniphest query, reloaded, and noted re-order stuck.

Reviewers: epriestley

Reviewed By: epriestley

CC: Korvin, epriestley, aran

Maniphest Tasks: T4422

Differential Revision: https://secure.phabricator.com/D8358

Details

Committed
Bob Trahan <bob.trahan@gmail.com>Feb 27 2014, 18:39
Pushed
aubortJan 31 2017, 17:16
Parents
rPHcce67caa0fc8: Fix bad method call from CustomField migration
Branches
Unknown
Tags
Unknown

Event Timeline

Bob Trahan <bob.trahan@gmail.com> committed rPH37b1b3163852: Maniphest - move subpriority edits to be transaction powered (authored by Bob Trahan <bob.trahan@gmail.com>).Feb 27 2014, 18:39