Paths

Table of Contentst

Set from name when $can_send_as_user is true
ClosedPublic
Actions

Authored by sophiebits on Apr 4 2014, 12:43 AM.

Tags

None

Referenced Files

	F19506288: D8694.id20621.diff
	Fri, Jan 9, 4:43 PM

	F19506206: D8694.id20623.diff
	Fri, Jan 9, 4:42 PM

	F19042186: D8694.id.diff
	Nov 26 2025, 6:30 PM

	F19035037: D8694.diff
	Nov 25 2025, 3:55 PM

	F18972483: D8694.id20623.diff
	Nov 15 2025, 2:05 PM

	F18972481: D8694.id20621.diff
	Nov 15 2025, 2:05 PM

	F18775206: D8694.diff
	Oct 10 2025, 6:16 PM

	F18758847: D8694.id20621.diff
	Oct 6 2025, 12:03 AM

View All Files

Subscribers

epriestley

Korvin

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

Branch

from-name

Lint

Lint Passed

Unit

Tests Passed

Event Timeline

sophiebits updated this revision to Diff 20621.Apr 4 2014, 12:43 AM

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.

Herald added a reviewer: Blessed Reviewers.Apr 4 2014, 12:43 AM

Herald added subscribers: Korvin, epriestley.

whatcouldgowrong

This revision is now accepted and ready to land.Apr 4 2014, 12:44 AM

epriestley added inline comments.Apr 4 2014, 12:45 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))`.

sophiebits added inline comments.Apr 4 2014, 12:46 AM

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?

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

Revision Contents
Changeset List

Path

Size

src/

applications/

metamta/

storage/

PhabricatorMetaMTAMail.php

2 lines

Diff 20621

View Options

src/applications/metamta/storage/PhabricatorMetaMTAMail.php

Set from name when $can_send_as_user is trueClosedPublicActions