Paths

Table of Contentst

Differential D11336

Fix some CC handling in Maniphest
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F14071391: D11336.id27234.diff
	Wed, Nov 20, 5:21 PM

	Unknown Object (File)
	Fri, Oct 25, 4:16 AM

	Unknown Object (File)
	Tue, Oct 22, 11:25 PM

	Unknown Object (File)
	Oct 19 2024, 4:18 PM

	Unknown Object (File)
	Oct 17 2024, 5:43 PM

	Unknown Object (File)
	Oct 17 2024, 7:47 AM

	Unknown Object (File)
	Oct 9 2024, 1:26 PM

	Unknown Object (File)
	Oct 9 2024, 1:26 PM

Subscribers

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

Branch

taskxaction

Lint

Lint Errors

Severity	Location	Code	Message
Error	src/applications/auth/provider/PhabricatorPhabricatorAuthProvider.php:50	PHL1	Unknown Symbol
Advice	src/applications/maniphest/controller/ManiphestTransactionSaveController.php:135	XHP16	TODO Comment

Unit

No Test Coverage

Build Status

Buildable 3769
Build 3780: [Placeholder Plan] Wait for 30 Seconds

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 27233

src/applications/maniphest/controller/ManiphestTransactionSaveController.php

Loading...