Replace links to /releases/<owner> with crates.io

Author: jsha

src/web/releases.rs

@@ -1403,50 +1403,6 @@ mod tests {
         });
     }
 
-    #[test]
-    fn owners_page() {
-        wrapper(|env| {
-            let web = env.frontend();
-            env.fake_release()
-                .name("some_random_crate")
-                .add_owner(CrateOwner {
-                    login: "foobar".into(),
-                    avatar: "https://example.org/foobar".into(),
-                    name: "Foo Bar".into(),
-                    email: "[email protected]".into(),
-                })
-                .create()?;
-            // Request an owner without @ sign.
-            assert_success("/releases/foobar", web)?;
-            // Request an owner with @ sign.
-            assert_success("/releases/@foobar", web)
-        })
-    }
-
-    #[test]
-    fn owners_pagination() {
-        wrapper(|env| {
-            let web = env.frontend();
-            for i in 0..RELEASES_IN_RELEASES {
-                env.fake_release()
-                    .name(&format!("some_random_crate_{}", i))
-                    .add_owner(CrateOwner {
-                        login: "foobar".into(),
-                        avatar: "https://example.org/foobar".into(),
-                        name: "Foo Bar".into(),
-                        email: "[email protected]".into(),
-                    })
-                    .create()?;
-            }
-            let page = kuchiki::parse_html().one(web.get("/releases/@foobar").send()?.text()?);
-            let button = page.select_first("a[href='/releases/@foobar/2']");
-
-            assert!(button.is_ok());
-
-            Ok(())
-        })
-    }
-
     #[test]
     fn home_page_links() {
         wrapper(|env| {

templates/crate/details.html

@@ -127,7 +127,7 @@
                         <li class="pure-menu-heading">Owners</li>
                         <li class="pure-menu-item">
                             {%- for owner in details.owners -%}
-                                <a href="/releases/{{ owner[0] }}">
+                                <a href="https://crates.io/users/{{ owner[0] }}">
                                     <img src="{{ owner[1] }}" alt="{{ owner[0] }}" class="owner">
                                 </a>
                             {%- endfor -%}

templates/releases/releases.html

@@ -58,20 +58,14 @@
 
             <div class="pagination">
                 {% block pagination %}
-                    {%- if release_type == 'owner' -%}
-                        {%- set page_link = "/releases/" ~ owner -%}
-                    {%- else -%}
-                        {%- set page_link = "/releases/" ~ release_type -%}
-                    {%- endif -%}
-
                     {%- if show_previous_page -%}
-                        <a class="pure-button pure-button-normal" href="{{ page_link | safe }}/{{ page_number - 1 }}{{ query | default(value='') }}">
+                        <a class="pure-button pure-button-normal" href="/releases/{{ release_type }}/{{ page_number - 1 }}{{ query | default(value='') }}">
                             {{ "arrow-left" | fas }} Previous Page
                         </a>
                     {%- endif -%}
 
                     {%- if show_next_page -%}
-                        <a class="pure-button pure-button-normal" href="{{ page_link | safe }}/{{ page_number + 1 }}{{ query | default(value='') }}">
+                        <a class="pure-button pure-button-normal" href="/releases/{{ release_type }}/{{ page_number + 1 }}{{ query | default(value='') }}">
                             Next Page {{ "arrow-right" | fas }}
                         </a>
                     {%- endif -%}

templates/rustdoc/topbar.html

@@ -97,7 +97,7 @@
 
                             {%- for owner in krate.owners -%}
                                 <li class="pure-menu-item">
-                                    <a href="/releases/{{ owner[0] }}" class="pure-menu-link">
+                                    <a href="https://crates.io/users/{{ owner[0] }}" class="pure-menu-link">
                                         {{ "user" | fas(fw=true) }} {{ owner[0] }}
                                     </a>
                                 </li>