Extract complex condition in its own method
ClosedPublic
Actions

Authored by dereckson on Mar 29 2018, 23:47.

Tags

None

Referenced Files

	F24179310: D1544.id3941.diff
	Wed, Feb 11, 03:22

	F24175740: D1544.id3941.diff
	Wed, Feb 11, 01:44

	F24172177: D1544.id3941.diff
	Wed, Feb 11, 00:12

	F24169386: D1544.id3941.diff
	Tue, Feb 10, 23:00

	F24163341: D1544.id3941.diff
	Tue, Feb 10, 20:12

	F24163331: D1544.id3940.diff
	Tue, Feb 10, 20:12

	F24163314: D1544.id3939.diff
	Tue, Feb 10, 20:12

	F24160417: D1544.diff
	Tue, Feb 10, 18:48

View All Files

Subscribers

None

Details

Reviewers

dereckson

Commits

rNOTIFbe3382f3ca03: Extract complex condition in its own method

Test Plan

Unit test introduced in f19194a38dab.

Diff Detail

Repository

rNOTIF Notifications center

Lint

Lint Passed

Unit

Tests Passed

Branch

isVoidObjectType

Build Status

Buildable 2421
Build 2669: arc lint + arc unit

Event Timeline

dereckson requested review of this revision.Mar 29 2018, 23:47

dereckson created this revision.

Harbormaster completed remote builds in B2421: Diff 3939.Mar 29 2018, 23:47

dereckson planned changes to this revision.Mar 29 2018, 23:48

dereckson added inline comments.

app/Phabricator/PhabricatorStory.php
126	has

s/is/has

Harbormaster completed remote builds in B2422: Diff 3940.Mar 29 2018, 23:48

dereckson accepted this revision.Mar 29 2018, 23:49

This revision is now accepted and ready to land.Mar 29 2018, 23:49

Closed by commit rNOTIFbe3382f3ca03: Extract complex condition in its own method (authored by dereckson). · Explain WhyMar 29 2018, 23:49

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

app/

Phabricator/

PhabricatorStory.php

6 lines

Diff 3939

View Options

app/Phabricator/PhabricatorStory.php

Extract complex condition in its own methodClosedPublicActions