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.

Details

Reviewers

amckinley

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

rP Phabricator

Branch

uxaction10

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 20639
Build 28047: Run Core Tests
Build 28046: arc lint + arc unit

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

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 46840

View Options

src/applications/config/check/PhabricatorExtraConfigSetupCheck.php

Remove obsolete setup check call to Maniphest "Can Edit <X>" field checksClosedPublicActions