Paths

Table of Contentst

Differential D20669

Simplify implementation of "SysetemAction->getSystemActionConstant()"
ClosedPublic
Actions

Authored by epriestley on Jul 19 2019, 5:27 PM.

Tags

None

Referenced Files

	F15382239: D20669.diff
	Fri, Mar 14, 11:27 AM

	F15360836: D20669.diff
	Tue, Mar 11, 10:38 AM

	F15282568: D20669.id49309.diff
	Tue, Mar 4, 12:21 AM

	F15282567: D20669.id49296.diff
	Tue, Mar 4, 12:21 AM

	F15282566: D20669.id.diff
	Tue, Mar 4, 12:21 AM

	F15282565: D20669.diff
	Tue, Mar 4, 12:21 AM

	Unknown Object (File)
	Mon, Mar 3, 5:49 PM

	Unknown Object (File)
	Tue, Feb 18, 7:45 AM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T13343: Make "Send a login link to your email address" email include why it was sent to avoid confusion

Commits

rP2ee5e71029af: Simplify implementation of "SysetemAction->getSystemActionConstant()"

Summary

Depends on D20668. Ref T13343. Just an easy cleanup/simplification while I'm here.

Test Plan

grep for getActionConstant()

Diff Detail

Repository

Branch

elogin8

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 23163
Build 31815: Run Core Tests
Build 31814: arc lint + arc unit

Event Timeline

epriestley created this revision.Jul 19 2019, 5:27 PM

Harbormaster completed remote builds in B23163: Diff 49296.Jul 19 2019, 5:29 PM

epriestley requested review of this revision.Jul 19 2019, 5:29 PM

epriestley added a child revision: D20670: Remove explicit administrative actions from the user activity log.Jul 19 2019, 6:17 PM

amckinley accepted this revision.Jul 19 2019, 9:44 PM

This revision is now accepted and ready to land.Jul 19 2019, 9:44 PM

Closed by commit rP2ee5e71029af: Simplify implementation of "SysetemAction->getSystemActionConstant()" (authored by epriestley). · Explain WhyJul 19 2019, 10:45 PM

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rP2ee5e71029af: Simplify implementation of "SysetemAction->getSystemActionConstant()".

Revision Contents
Changeset List

Path

Size

src/

applications/

auth/

action/

PhabricatorAuthChangePasswordAction.php

4 lines

PhabricatorAuthEmailLoginAction.php

4 lines

PhabricatorAuthNewFactorAction.php

4 lines

PhabricatorAuthTestSMSAction.php

4 lines

PhabricatorAuthTryEmailLoginAction.php

4 lines

PhabricatorAuthTryFactorAction.php

4 lines

PhabricatorAuthTryPasswordAction.php

4 lines

PhabricatorAuthTryPasswordWithoutCAPTCHAAction.php

4 lines

files/

action/

PhabricatorFilesOutboundRequestAction.php

4 lines

metamta/

action/

PhabricatorMetaMTAErrorMailAction.php

4 lines

phortune/

action/

PhortuneAddPaymentMethodAction.php

4 lines

settings/

action/

PhabricatorSettingsAddEmailAction.php

4 lines

system/

action/

PhabricatorSystemAction.php

5 lines

Diff 49296

src/applications/auth/action/PhabricatorAuthChangePasswordAction.php

Loading...

src/applications/auth/action/PhabricatorAuthEmailLoginAction.php

Loading...

src/applications/auth/action/PhabricatorAuthNewFactorAction.php

Loading...

src/applications/auth/action/PhabricatorAuthTestSMSAction.php

Loading...

src/applications/auth/action/PhabricatorAuthTryEmailLoginAction.php

Loading...

src/applications/auth/action/PhabricatorAuthTryFactorAction.php

Loading...

src/applications/auth/action/PhabricatorAuthTryPasswordAction.php

Loading...

src/applications/auth/action/PhabricatorAuthTryPasswordWithoutCAPTCHAAction.php

Loading...

src/applications/files/action/PhabricatorFilesOutboundRequestAction.php

Loading...

src/applications/metamta/action/PhabricatorMetaMTAErrorMailAction.php

Loading...

src/applications/phortune/action/PhortuneAddPaymentMethodAction.php

Loading...

src/applications/settings/action/PhabricatorSettingsAddEmailAction.php

Loading...

src/applications/system/action/PhabricatorSystemAction.php

Loading...