Extract complex condition in its own method
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F47616472: D1544.id3939.diff
	Tue, Sep 1, 03:48

	F47616471: D1544.id3940.diff
	Tue, Sep 1, 03:48

	F47610685: D1544.id3941.diff
	Tue, Sep 1, 03:05

	F47596462: D1544.diff
	Tue, Sep 1, 01:19

	F47568651: D1544.id3941.diff
	Mon, Aug 31, 22:13

	Unknown Object (File)
	Thu, Aug 27, 00:23

	Unknown Object (File)
	Sun, Aug 23, 13:17

	Unknown Object (File)
	Sun, Aug 23, 10:09

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 Not Applicable

Unit

Tests Not Applicable

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 3941

View Options

app/Phabricator/PhabricatorStory.php

Extract complex condition in its own methodClosedPublicActions