Page MenuHomePhabricator

Produce a more tailored checkout URI for Subversion
ClosedPublic

Authored by epriestley on Feb 28 2014, 8:54 PM.
Tags
None
Referenced Files
F19054152: D8375.id19901.diff
Fri, Nov 28, 10:54 AM
F19053795: D8375.diff
Fri, Nov 28, 9:55 AM
F18943232: D8375.id.diff
Tue, Nov 11, 1:34 PM
F18851752: D8375.id19905.diff
Fri, Oct 31, 3:50 AM
F18780646: D8375.diff
Oct 12 2025, 6:18 PM
F18630094: D8375.id.diff
Sep 16 2025, 10:45 AM
F18619175: D8375.id19905.diff
Sep 15 2025, 1:54 AM
F18616237: D8375.id19904.diff
Sep 14 2025, 4:42 PM
Subscribers

Details

Summary

For imported SVN repositories with an "Import Only" path, we produce a /path/to/root/ URI, but should produce /path/to/root/then/to/import/only/.

As it is, the URI instructs the user to check out the whole repository.

Also, don't show the "Clone As" fragment in the URI for remote repositories, and prevent it from being edited for nonhosted repositories. This is generally more consistent with user expectation.

Test Plan
  • Created a remote SVN repository with "Import Only", saw path include it.
  • Verified no "Clone As" options, no "Clone As" in URI.
  • Switched it to hosted, saw "Clone As" options appear and work properly.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

btrahan edited edge metadata.
This revision is now accepted and ready to land.Feb 28 2014, 9:04 PM
epriestley updated this revision to Unknown Object (????).Feb 28 2014, 9:04 PM
  • Add a unit test, since we have coverage on this stuff.