Paths

Table of Contentst

Fix some CC handling in Maniphest
ClosedPublic
Actions

Authored by epriestley on Jan 11 2015, 11:56 PM.

Tags

None

Referenced Files

	F18352695: D11336.diff
	Tue, Aug 26, 11:52 PM

	F18219518: D11336.id27234.diff
	Tue, Aug 19, 11:48 AM

	F18217300: D11336.id27233.diff
	Tue, Aug 19, 8:46 AM

	F18214026: D11336.id27235.diff
	Tue, Aug 19, 5:07 AM

	F18196411: D11336.diff
	Sun, Aug 17, 6:02 PM

	F17899852: D11336.id27235.diff
	Tue, Jul 29, 8:47 AM

	F17713539: D11336.id27233.diff
	Jul 17 2025, 2:12 PM

	F17705413: D11336.diff
	Jul 16 2025, 7:53 AM

View All Files

Subscribers

epriestley

Tokens

"Grey Medal" token, awarded by btrahan.

Details

Reviewers

joshuaspence
btrahan

Maniphest Tasks

T6932: Unable to claim a Maniphest task

Commits

Restricted Diffusion Commit
rP5b3b9b7182ae: 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.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 27233.Jan 11 2015, 11:56 PM

epriestley retitled this revision from to Fix some CC handling in Maniphest.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added reviewers: joshuaspence, btrahan.

Herald added a subscriber: epriestley. · View Herald TranscriptJan 11 2015, 11:56 PM

Clean lint.

joshuaspence accepted this revision.Jan 12 2015, 12:17 AM

joshuaspence updated this object.

joshuaspence added a task: T6932: Unable to claim a Maniphest task.

joshuaspence edited edge metadata.

This revision is now accepted and ready to land.Jan 12 2015, 12:18 AM

btrahan awarded a token.Jan 12 2015, 1:50 AM

Closed by commit rP5b3b9b7182ae: Fix some CC handling in Maniphest (authored by epriestley, committed by epriestley). · Explain WhyJan 12 2015, 1:57 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

maniphest/

controller/

ManiphestTransactionSaveController.php

34 lines

Diff 27235

View Options

src/applications/maniphest/controller/ManiphestTransactionSaveController.php

Fix some CC handling in ManiphestClosedPublicActions