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

	F18804266: D11336.id27234.diff
	Sat, Oct 18, 7:07 AM

	F18798820: D11336.id27235.diff
	Fri, Oct 17, 9:00 AM

	F18771701: D11336.diff
	Wed, Oct 8, 9:10 PM

	F18764974: D11336.id27234.diff
	Tue, Oct 7, 8:51 AM

	F18682193: D11336.diff
	Sep 26 2025, 5:21 AM

	F18680925: D11336.diff
	Sep 26 2025, 3:31 AM

	F18460451: D11336.id27233.diff
	Sep 1 2025, 7:06 PM

	F18388461: D11336.id27234.diff
	Aug 29 2025, 4:27 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