Homec4science

Fix an issue with To/CC placeholders

Authored by epriestley <git@epriestley.com> on Aug 20 2012, 23:08.

Description

Fix an issue with To/CC placeholders

Summary: Currently, if no placeholder is configured we always move "Cc" up to "To", even if we have a valid "To". Instead, move "Cc" to "To" only if there's no "To" and no placeholder.

Test Plan: Sent email with "to" and "cc", email with "cc" only with a placeholder, and email with "cc" only without a placeholder. Verified recipients ended up in the right location in all cases.

Reviewers: nh, btrahan

Reviewed By: btrahan

CC: klimek, aran

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

Details

Committed
epriestley <git@epriestley.com>Aug 20 2012, 23:08
Pushed
aubortJan 31 2017, 17:16
Parents
rPHe7796caa788c: Don't reverse downloaded raw diff
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH21ebd1a609e2: Fix an issue with To/CC placeholders (authored by epriestley <git@epriestley.com>).Aug 20 2012, 23:08