HomePhabricator

Fix some CC handling in Maniphest

Description

Fix some CC handling in Maniphest

Summary:
Fixes T6932. Fixes some issues from D11303.

  • When claiming a task, if it was previously unassigned, we would try to CC null.
  • When claiming a task, if the current owner was already CC'd, the viewer would incorrectly be warned about all subscribers being CC'd.

Test Plan:

  • Claimed an unclaimed task.
  • Claimed a task with owner CC'd.

Reviewers: btrahan, joshuaspence

Reviewed By: joshuaspence

Subscribers: epriestley

Maniphest Tasks: T6932

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

Details

Provenance
epriestleyAuthored on
epriestleyPushed on Jan 12 2015, 1:57 AM
Reviewer
joshuaspence
Differential Revision
D11336: Fix some CC handling in Maniphest
Parents
rP698b7f9ea39c: Explicitly declare method/property visibility
Branches
Unknown
Tags
Unknown
Tasks
T6932: Unable to claim a Maniphest task