Page MenuHomePhabricator

Add support for branch-related configuration to new Repository edit workflow
ClosedPublic

Authored by epriestley on Oct 25 2013, 5:27 PM.
Tags
None
Referenced Files
F15438434: D7399.id.diff
Wed, Mar 26, 12:40 AM
F15387394: D7399.id16682.diff
Mar 15 2025, 2:19 AM
F15387050: D7399.id.diff
Mar 15 2025, 1:44 AM
F15353670: D7399.id16682.diff
Mar 10 2025, 11:37 PM
F15347663: D7399.diff
Mar 10 2025, 2:07 PM
F15294639: D7399.id16682.diff
Mar 5 2025, 10:18 AM
Unknown Object (File)
Mar 1 2025, 9:18 PM
Unknown Object (File)
Feb 28 2025, 12:05 PM
Subscribers

Details

Reviewers
btrahan
chad
Maniphest Tasks
Restricted Maniphest Task
Commits
Restricted Diffusion Commit
rPdcb0b1b64f73: Add support for branch-related configuration to new Repository edit workflow
Summary

Ref T2231. Modernizes editing "Default Branch", "Track Only", and "Autoclose Only".

Test Plan

See screenshots.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

{F74984}

{F74985}

{F74986}

{F74987}

A whatevs inline - I think its cleaner as I suggest but the odds of needing the variables as you have them seems high so yeah.

src/applications/diffusion/controller/DiffusionRepositoryEditController.php
11–26

you could just instantiate $has_branches with this switch statement

Yeah I'm assuming I'm going to end up with a couple more of those in a bit, but maybe not. I do use $is_svn in the next diff I think, not sure how busy it will be by the time I'm done.