Fix Search Application Config
ClosedPublic
Actions

Authored by avivey on Jul 17 2017, 11:43 PM.

Details

Reviewers

epriestley
20after4

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T12924: Applications -> Search -> Configure is fataling

Commits

rPd1f144b214df: Fix Search Application Config

Summary

Fix T12924. Looks like this melted in D17384, and nobody noticed yet.

Test Plan

Visit page, see fancy table.

Diff Detail

Repository

rP Phabricator

Branch

master

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 17716
Build 23786: Run Core Tests
Build 23785: arc lint + arc unit

Event Timeline

avivey created this revision.Jul 17 2017, 11:43 PM

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJul 17 2017, 11:43 PM

Herald added a subscriber: Korvin. · View Herald Transcript

Harbormaster completed remote builds in B17716: Diff 43849.Jul 17 2017, 11:44 PM

looks good to me and confirmed to work by testing locally on my test instance running upstream master branch.

epriestley accepted this revision.Jul 18 2017, 9:28 AM

This revision is now accepted and ready to land.Jul 18 2017, 9:28 AM

Closed by commit rPd1f144b214df: Fix Search Application Config (authored by avivey, committed by avivey). · Explain WhyJul 18 2017, 5:45 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

search/

applicationpanel/

PhabricatorSearchApplicationStorageEnginePanel.php

47 lines

Diff 43849

View Options

src/applications/search/applicationpanel/PhabricatorSearchApplicationStorageEnginePanel.php

Fix Search Application ConfigClosedPublicActions