Homec4science

Fix batch editor for edits which both add and remove projects

Authored by epriestley <git@epriestley.com> on Mar 21 2012, 12:37.

Description

Fix batch editor for edits which both add and remove projects

Summary:
Aggregate multiple add/remove transactions so we don't restore removed projects for a (remove + add) batch edit.

(Possibly we should do this in the TransactionEditor as well / instead, but it's fairly easy here and this is the only possible case currently.)

Test Plan: Performed a remove + add batch edit without issues.

Reviewers: btrahan

Reviewed By: btrahan

CC: cpiro, aran, epriestley

Maniphest Tasks: T985

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

Details

Committed
epriestley <git@epriestley.com>Mar 21 2012, 12:37
Pushed
aubortJan 31 2017, 17:16
Parents
rPH238b403509c2: Allow the users to view their unsubmitted inline comments in one view
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH81be1cf27b5a: Fix batch editor for edits which both add and remove projects (authored by epriestley <git@epriestley.com>).Mar 21 2012, 12:37