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

	F19051057: D20669.diff
	Thu, Nov 27, 11:06 PM

	F18873778: D20669.diff
	Wed, Nov 5, 11:24 AM

	F18861666: D20669.diff
	Sun, Nov 2, 4:14 PM

	F18783700: D20669.id49309.diff
	Oct 13 2025, 8:57 AM

	F18761522: D20669.id.diff
	Oct 6 2025, 2:56 PM

	F18756383: D20669.diff
	Oct 5 2025, 11:15 AM

	F18735992: D20669.id.diff
	Oct 1 2025, 4:22 AM

	F18588722: D20669.diff
	Sep 12 2025, 1:18 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...