Page MenuHomePhabricator

Support slightly prettier repository URIs in Diffusion
ClosedPublic

Authored by epriestley on Nov 13 2016, 8:02 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 8, 1:37 AM
Unknown Object (File)
Tue, Apr 2, 5:35 AM
Unknown Object (File)
Mon, Apr 1, 12:31 AM
Unknown Object (File)
Mon, Apr 1, 12:31 AM
Unknown Object (File)
Mon, Apr 1, 12:31 AM
Unknown Object (File)
Mon, Apr 1, 12:31 AM
Unknown Object (File)
Sat, Mar 23, 6:04 PM
Unknown Object (File)
Sat, Mar 23, 6:04 PM
Subscribers

Details

Summary

Fixes T4245. When a repository has a short name, use /source/shortname/ as its primary URI.

Test Plan
  • Cloned Git repositories from shortnames via HTTP and SSH.
  • Cloned Mercurial repositories from shortnames via HTTP and SSH.
  • Cloned Subversion repositories from shortnames via SSH.
  • Browsed Git, Mercurial and Subversion repositories.
  • Added and removed short names to various repositories.

Diff Detail

Repository
rP Phabricator
Branch
pretty1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 14471
Build 18864: Run Core Tests
Build 18863: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Support slightly prettier repository URIs in Diffusion.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
chad added inline comments.
src/applications/diffusion/controller/DiffusionLastModifiedController.php
19

imgres.jpg (168×300 px, 6 KB)

This revision is now accepted and ready to land.Nov 13 2016, 8:40 PM
src/applications/diffusion/controller/DiffusionLastModifiedController.php
19

o i c

This revision was automatically updated to reflect the committed changes.

We default "short name" to a mangled version of the repository name, which is often "good enough" (looks like all repos here don't have short-name explicitly set), but we don't consider it a "short name" for this feature.

Right now, a few of my repos (The ones with the confusing names) show the /source/short-name.git, and most show /diffusion/<id>/short-name.git, which I suspect would ends up confusing for users.

We don't consider the "clone name" to be a "short name" for the purposes of this feature because it is not guaranteed to be unique (many different names mangle down to the same "clone name", and repositories are not required to have unique names in the first place).

Ahh, that makes sense.
The UI is a little confusing around this though, because it refers to both "clone name" and "repository slug" as "short name" in the manage page; I'll try to break that up a bit?

Short Name: none
Clone As: yellow-fruit (Automatic clone name)
Short Name: green-tomatoes
Clone As: green-tomatoes (Short name)

Maybe just remove "Clone As" completely? e.g., only show:

Short Name: the-short-name

...or:

Short Name: None (or just don't show anything idk)