Differential D19593

Remove obsolete setup check call to Maniphest "Can Edit <X>" field checks
ClosedPublic
Actions

Authored by epriestley on Aug 16 2018, 6:09 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Feb 20, 11:35 AM

	Unknown Object (File)
	Thu, Feb 20, 8:40 AM

	Unknown Object (File)
	Wed, Feb 19, 7:21 AM

	Unknown Object (File)
	Thu, Feb 13, 6:43 PM

	Unknown Object (File)
	Sun, Feb 9, 6:20 AM

	Unknown Object (File)
	Sun, Feb 9, 6:20 AM

	Unknown Object (File)
	Sun, Feb 9, 6:20 AM

	Unknown Object (File)
	Sun, Feb 9, 2:51 AM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T13164: Plans: 2018 Week 31 - 33 Bonus Content

Commits

rPa48e6897a418: Remove obsolete setup check call to Maniphest "Can Edit <X>" field checks

Summary

Ref T13164. Missed this in D19581.

Test Plan

Forced setup checks to re-run by visiting Config → Setup Issues explicitly.
Before patch: fatal on call to nonexistent method.
After patch: setup issues.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Aug 16 2018, 6:09 PM

Harbormaster completed remote builds in B20639: Diff 46840.Aug 16 2018, 6:10 PM

epriestley requested review of this revision.Aug 16 2018, 6:10 PM

See also https://github.com/phacility/phabricator/commit/296bf046a8122c451ba0bf1a93733dccccaeb390#commitcomment-30105524.

amckinley accepted this revision.Aug 16 2018, 8:31 PM

This revision is now accepted and ready to land.Aug 16 2018, 8:31 PM

Closed by commit rPa48e6897a418: Remove obsolete setup check call to Maniphest "Can Edit <X>" field checks (authored by epriestley). · Explain WhyAug 16 2018, 8:56 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

config/

check/

PhabricatorExtraConfigSetupCheck.php

2 lines

Diff 46843

src/applications/config/check/PhabricatorExtraConfigSetupCheck.php

Loading...