Page MenuHomePhabricator

In Maniphest tasks, only move old owner to CC if owner changed
ClosedPublic

Authored by epriestley on Feb 23 2016, 6:01 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Mar 2, 9:49 PM
Unknown Object (File)
Tue, Feb 11, 6:48 AM
Unknown Object (File)
Sun, Feb 9, 12:50 AM
Unknown Object (File)
Jan 28 2025, 11:29 PM
Unknown Object (File)
Jan 28 2025, 1:01 AM
Unknown Object (File)
Jan 24 2025, 11:58 AM
Unknown Object (File)
Jan 8 2025, 2:33 AM
Unknown Object (File)
Dec 6 2024, 2:17 PM
Subscribers
None

Details

Summary

Fixes T10426. When the owner of a task changes, we try to add the old owner to CC so they're kept in the loop.

Currently, we do this unconditionally. This can add the owner as a subscriber when someone didn't change anything, which is confusing.

Instead, only do this if the owner actually changed.

Test Plan
  • With "A" as owner, edited task and saved.
    • Before patch, A was added as subscriber.
    • After patch, A not added.
  • With "A" as owner, changed owner to "B" and saved.
    • Both before and after patch, "A" is added as a subscriber.

Diff Detail

Repository
rP Phabricator
Branch
ownerself
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 10821
Build 13334: Run Core Tests
Build 13333: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to In Maniphest tasks, only move old owner to CC if owner changed.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Feb 23 2016, 6:09 PM
This revision was automatically updated to reflect the committed changes.