Differential D20001

Fix "Welcome Mail" check for a message when no message exists
ClosedPublic
Actions

Authored by epriestley on Jan 20 2019, 2:57 PM.

Tags

None

Referenced Files

	F19795766: D20001.diff
	Fri, Feb 27, 4:07 AM

	F19113371: D20001.id47756.diff
	Dec 9 2025, 6:04 PM

	F19056643: D20001.id47756.diff
	Nov 28 2025, 8:12 PM

	F19020137: D20001.diff
	Nov 23 2025, 5:05 PM

	F19018584: D20001.diff
	Nov 23 2025, 11:48 AM

	F18837995: D20001.diff
	Oct 27 2025, 8:39 AM

	F18814364: D20001.id.diff
	Oct 20 2025, 11:04 PM

	F18809391: D20001.diff
	Oct 19 2025, 1:36 PM

View All Files

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13239: null pointer exception in PhabricatorPeopleWelcomeController
Commits: rP491ba6d77e19: (stable) Fix "Welcome Mail" check for a message when no message exists
rPbe8b7c9ebac7: Fix "Welcome Mail" check for a message when no message exists

Summary

Fixes T13239. See that task for discussion.

Test Plan

Tried to send welcome mail with no "Welcome" message.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Jan 20 2019, 2:57 PM

Harbormaster completed remote builds in B21611: Diff 47756.Jan 20 2019, 2:59 PM

epriestley requested review of this revision.Jan 20 2019, 2:59 PM

epriestley mentioned this in T13239: null pointer exception in PhabricatorPeopleWelcomeController.Jan 20 2019, 3:00 PM

This revision was not accepted when it landed; it landed in state Needs Review.Jan 20 2019, 3:00 PM

Closed by commit rPbe8b7c9ebac7: Fix "Welcome Mail" check for a message when no message exists (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

amckinley mentioned this in D19995: Use the customizable "Welcome Mail" message in welcome mail.Jan 21 2019, 7:49 PM

Revision Contents
Changeset List

Path

Size

src/

applications/

people/

controller/

PhabricatorPeopleWelcomeController.php

2 lines

Diff 47757

View Options

src/applications/people/controller/PhabricatorPeopleWelcomeController.php

Fix "Welcome Mail" check for a message when no message existsClosedPublicActions