Page MenuHomePhabricator

Set from name when $can_send_as_user is true
ClosedPublic

Authored by sophiebits on Apr 4 2014, 12:43 AM.
Tags
None
Referenced Files
F18775206: D8694.diff
Fri, Oct 10, 6:16 PM
F18758847: D8694.id20621.diff
Mon, Oct 6, 12:03 AM
F18700352: D8694.diff
Sat, Sep 27, 3:04 PM
F18700327: D8694.diff
Sat, Sep 27, 3:00 PM
F18700235: D8694.diff
Sat, Sep 27, 2:49 PM
F18658794: D8694.diff
Sep 23 2025, 8:27 AM
F18618617: D8694.diff
Sep 15 2025, 12:34 AM
F18207357: D8694.id20621.diff
Aug 18 2025, 5:52 PM
Subscribers

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
Restricted Diffusion Commit
rP133772d7eb1f: Set from name when $can_send_as_user is true
Test Plan

Crossed fingers.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

sophiebits retitled this revision from to Set from name when $can_send_as_user is true.
sophiebits updated this object.
sophiebits edited the test plan for this revision. (Show Details)
sophiebits added a reviewer: epriestley.
This revision is now accepted and ready to land.Apr 4 2014, 12:44 AM
src/applications/metamta/storage/PhabricatorMetaMTAMail.php
365

Also I wonder if this clause should maybe be like if ($can_send_as_user && ($actor_email !== null)).

src/applications/metamta/storage/PhabricatorMetaMTAMail.php
365

Well, we have

$can_send_as_user = $actor_email &&
  PhabricatorEnv::getEnvConfig('metamta.can-send-as-user');

which I assumed was good enough.

365

Or did you mean just store the config option in that var?

epriestley updated this revision to Diff 20623.

Closed by commit rP133772d7eb1f (authored by @spicyj, committed by @epriestley).