diff options
| author | Kornel <kornel@geekhood.net> | 2020-02-29 19:21:36 +0000 |
|---|---|---|
| committer | Kornel <kornel@geekhood.net> | 2020-02-29 20:32:40 +0000 |
| commit | 2cac8d44702175f12de6dd9d660970e045451ecd (patch) | |
| tree | fcb9febc4c810fd9e635b95962c49562226262a2 | |
| parent | 887ddd85dab2a4081fcdf012060699345a3ac3ad (diff) | |
Rev deps link
| -rw-r--r-- | front_end/src/urler.rs | 8 | ||||
| -rw-r--r-- | front_end/templates/crate_page.rs.html | 13 |
2 files changed, 12 insertions, 9 deletions
diff --git a/front_end/src/urler.rs b/front_end/src/urler.rs index 7a54582..65f6821 100644 --- a/front_end/src/urler.rs +++ b/front_end/src/urler.rs @@ -68,12 +68,10 @@ impl Urler { } } - pub fn reverse_deps(&self, origin: &Origin) -> String { + pub fn reverse_deps(&self, origin: &Origin) -> Option<String> { match origin { - Origin::CratesIo(lowercase_name) => format!("/crates/{}/rev", encode(lowercase_name)), - Origin::GitHub { package, .. } | Origin::GitLab { package, .. } => { - format!("/crates/{}/rev", encode(package)) // FIXME: that's bogus, return None - }, + Origin::CratesIo(lowercase_name) => Some(format!("/crates/{}/rev", encode(lowercase_name))), + Origin::GitHub { package, .. } | Origin::GitLab { package, .. } => None, } } diff --git a/front_end/templates/crate_page.rs.html b/front_end/templates/crate_page.rs.html index fc4ba81..2486d8f 100644 --- a/front_end/templates/crate_page.rs.html +++ b/front_end/templates/crate_page.rs.html @@ -216,14 +216,19 @@ Used in <b>@c.format_number(deps)</b> crate@if deps != 1 {s} @if direct == 1 { @if let Some(name) = name { - (via <a href="@url.reverse_deps(&Origin::from_crates_io_name(name))">@name</a>) + @if let Some(link) = url.reverse_deps(&Origin::from_crates_io_name(name)) { + (via <a href="@link">@name</a>) + } } } else { - (<a href="@url.reverse_deps(c.ver.origin())">@c.format_number(direct) directly</a>) + @if let Some(link) = url.reverse_deps(c.ver.origin()) { + (<a href="@link">@c.format_number(direct) directly</a>) + } } - } else { - Used in <a href="@url.reverse_deps(c.ver.origin())"><b>@c.format_number(deps)</b> crate@if deps != 1 {s}</a> + @if let Some(link) = url.reverse_deps(c.ver.origin()) { + Used in <a href="@link"><b>@c.format_number(deps)</b> crate@if deps != 1 {s}</a> + } } }} } |