Page MenuHomePhabricator

Give "Autoclose Only" repository detail proper getters/setters
ClosedPublic

Authored by epriestley on Dec 10 2018, 3:50 PM.

Details

Summary

Ref T13222. See D19829. We're inconsistent about using getDetail()/setDetail() to do some ad-hoc reads. Put this stuff in proper accessor methods.

Also a couple of text fixes from D19850.

Test Plan

Set, edited, and removed autoclose branches from a repository. Got sensible persistence and rendering behavior.

Diff Detail

Repository
rP Phabricator
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.Dec 10 2018, 3:50 PM
epriestley requested review of this revision.Dec 10 2018, 3:52 PM
amckinley accepted this revision.Dec 10 2018, 6:21 PM
This revision is now accepted and ready to land.Dec 10 2018, 6:21 PM
This revision was automatically updated to reflect the committed changes.