HomePhabricator

Use ManiphestTaskQuery in nearly all interfaces

Description

Use ManiphestTaskQuery in nearly all interfaces

Summary:
Ref T603. Make almost every task read policy-aware. Notable exceptions are:

  • Edge editor -- this stuff is prescreened and should be moved to ApplicationTransactions eventually anyway.
  • Search/attach stuff -- this stuff needs some general work. The actual list should be fine since you can't pull handles. There may be a very indirect hole here where you could attach an object you can't see (but do know the ID of) to an object you can see. Pretty fluff.
  • The "Tasks" field in Differential will let you reference objects you can't see. Possibly this is desirable, in the case of commandeering revisions. Mostly, it was inconvenient to get a viewer (I think).

Test Plan:

  • Called maniphest.info.
  • Called maniphest.update.
  • Batch edited tasks.
  • Dragged and dropped tasks to change subpriority.
  • Subscribed and unsubscribed from a task.
  • Edited a task.
  • Created a task.
  • Created a task with a parent.
  • Created a task with a template.
  • Previewed a task update.
  • Commented on a task.
  • Added a dependency.
  • Searched for "T33" in object search dialog.
  • Created a branch "T33", ran arc diff, verified link.
  • Pushed a commit with "Fixes T33", verified close.

Reviewers: btrahan

Reviewed By: btrahan

CC: aran

Maniphest Tasks: T603

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

Details

Provenance
epriestleyAuthored on Sep 25 2013, 8:44 PM
Reviewer
btrahan
Differential Revision
Restricted Differential Revision
Parents
rP475afe4a5b51: Show full names for attached diffs and tasks in email in Maniphest
Branches
Unknown
Tags
Unknown
Tasks
Restricted Maniphest Task
T603: Support permissions/policies in all Phabricator applications

Event Timeline

epriestley added a task: Restricted Maniphest Task.Sep 15 2017, 5:26 PM