Differential D11374

Check policy.locked slightly sooner in PhabricatorApplication
ClosedPublic
Actions

Authored by epriestley on Jan 13 2015, 8:07 PM.

Details

Reviewers

btrahan

Commits

Restricted Diffusion Commit
rPad9af106ee08: Check policy.locked slightly sooner in PhabricatorApplication

Summary

We could still miss this if the policy had never been customized and we returned early after one of the other checks.

Test Plan

Works great on instances now.

Screen_Shot_2015-01-13_at_12.06.12_PM.png (1×1 px, 173 KB)

Diff Detail

Repository

rP Phabricator

Branch

checksooner

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 3822
Build 3834: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 27324.Jan 13 2015, 8:07 PM

epriestley retitled this revision from to Check policy.locked slightly sooner in PhabricatorApplication.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: btrahan.

Herald added a subscriber: epriestley. · View Herald TranscriptJan 13 2015, 8:07 PM

btrahan accepted this revision.Jan 13 2015, 8:33 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Jan 13 2015, 8:33 PM

Closed by commit rPad9af106ee08: Check policy.locked slightly sooner in PhabricatorApplication (authored by epriestley, committed by epriestley). · Explain WhyJan 13 2015, 8:40 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

base/

PhabricatorApplication.php

10 lines

Diff 27324

View Options

src/applications/base/PhabricatorApplication.php

Check policy.locked slightly sooner in PhabricatorApplicationClosedPublicActions