Extract complex condition in its own method
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F6924473: D1544.id3941.diff
	Thu, Apr 17, 10:40

	F6907942: D1544.id3940.diff
	Thu, Apr 17, 00:53

	F6907815: D1544.id3939.diff
	Thu, Apr 17, 00:49

	Unknown Object (File)
	Fri, Apr 11, 21:49

	Unknown Object (File)
	Fri, Apr 11, 18:03

	Unknown Object (File)
	Fri, Apr 11, 12:28

	Unknown Object (File)
	Fri, Apr 11, 10:30

	Unknown Object (File)
	Thu, Apr 10, 23:17

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