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
F14068653: D7399.diff
Tue, Nov 19, 11:26 PM
F14055775: D7399.diff
Sat, Nov 16, 3:34 PM
F14042637: D7399.diff
Tue, Nov 12, 5:02 AM
F14028815: D7399.diff
Fri, Nov 8, 4:10 PM
F13991408: D7399.id16668.diff
Tue, Oct 22, 10:39 AM
F13970284: D7399.id16682.diff
Oct 17 2024, 6:42 AM
F13961604: D7399.diff
Oct 15 2024, 5:41 AM
Unknown Object (File)
Oct 12 2024, 4:48 AM
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.