Details
- Reviewers
amckinley - Maniphest Tasks
- T4900: Workboards updating in real time
- Commits
- rPec352b1b31bb: Move workboard "Bulk Edit Tasks" workflow to a separate controller
- Used "Bulk Edit Tasks" on a column with some tasks, got an appropraite edit operation.
- Used "Bulk Edit Tasks" on an empty column, got an error.
Diff Detail
- Repository
- rP Phabricator
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
| src/applications/project/controller/PhabricatorProjectBoardViewController.php | ||
|---|---|---|
| 6 | This removes some "edit all" stuff, I'll discuss that separately. | |
In D12475, the "bulk edit" (then, "batch edit") stuff was added originally. This included a "Batch Edit Visible Tasks..." action that brought you into an edit workflow for every visible task on the board.
This was removed in D17879, probably by accident, since none of the support code was removed and the revision doesn't mention intending to remove it.
However, no one has noticed since then or requested this feature, so just get rid of it completely. You can generally perform this operation via queries in Maniphest with relative ease, anyway.
Later, in D18806, I made the UI consistently say "Bulk Edit" (previously, it sometimes said "Batch" and sometimes said "Bulk") but didn't change all the variable names. Now that this is getting some standalone testing, update the variable names to match the UI.