Skip to content

Commit 05b7435

Browse files
GuillaumeGomezGuillaumeGomez
committed
Migrate menus URLs under /crate/ 
1 parent 529aa33 commit 05b7435

File tree

3 files changed

+48
-33
lines changed

3 files changed

+48
-33
lines changed

‎src/web/crate_details.rs‎

Lines changed: 23 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -1298,13 +1298,15 @@ mod tests {
12981298

12991299
fn run_check_links(
13001300
env: &TestEnvironment,
1301-
url: &str,
1301+
url_start: &str,
1302+
url_end: &str,
13021303
extra: &str,
13031304
should_contain_redirect: bool,
13041305
) {
13051306
run_check_links_redir(
13061307
env,
1307-
url,
1308+
url_start,
1309+
url_end,
13081310
extra,
13091311
should_contain_redirect,
13101312
should_contain_redirect,
@@ -1313,18 +1315,24 @@ mod tests {
13131315

13141316
fn run_check_links_redir(
13151317
env: &TestEnvironment,
1316-
url: &str,
1318+
url_start: &str,
1319+
url_end: &str,
13171320
extra: &str,
13181321
should_contain_redirect: bool,
13191322
ajax_should_contain_redirect: bool,
13201323
) {
1321-
let response = env.frontend().get(url).send().unwrap();
1324+
let response = env.frontend().get(&format!("{url_start}{url_end}")).send().unwrap();
13221325
assert!(response.status().is_success());
13231326
let list1 = check_links(response.text().unwrap(), false, should_contain_redirect);
13241327
// Same test with AJAX endpoint.
1328+
let (start, extra_name) = if url_start.starts_with("/crate/") {
1329+
("", "/crate")
1330+
} else {
1331+
("/crate", "")
1332+
};
13251333
let response = env
13261334
.frontend()
1327-
.get(&format!("/-/menus/platforms{url}{extra}"))
1335+
.get(&format!("{start}{url_start}/menus/platforms{extra_name}{url_end}{extra}"))
13281336
.send()
13291337
.unwrap();
13301338
assert!(response.status().is_success());
@@ -1356,17 +1364,18 @@ mod tests {
13561364

13571365
// FIXME: For some reason, there are target-redirects on non-AJAX lists on docs.rs
13581366
// crate pages other than the "default" one.
1359-
run_check_links_redir(env, "/crate/dummy/0.4.0/features", "", true, false);
1360-
run_check_links_redir(env, "/crate/dummy/0.4.0/builds", "", true, false);
1361-
run_check_links_redir(env, "/crate/dummy/0.4.0/source/", "", true, false);
1362-
run_check_links_redir(env, "/crate/dummy/0.4.0/source/README.md", "", true, false);
1363-
1364-
run_check_links(env, "/crate/dummy/0.4.0", "", false);
1365-
run_check_links(env, "/dummy/latest/dummy", "/", true);
1366-
run_check_links(env, "/dummy/0.4.0/x86_64-pc-windows-msvc/dummy", "/", true);
1367+
run_check_links_redir(env, "/crate/dummy/0.4.0", "/features", "", true, false);
1368+
run_check_links_redir(env, "/crate/dummy/0.4.0", "/builds", "", true, false);
1369+
run_check_links_redir(env, "/crate/dummy/0.4.0", "/source/", "", true, false);
1370+
run_check_links_redir(env, "/crate/dummy/0.4.0", "/source/README.md", "", true, false);
1371+
1372+
run_check_links(env, "/crate/dummy/0.4.0", "", "/", false);
1373+
run_check_links(env, "/dummy/latest", "/dummy", "/", true);
1374+
run_check_links(env, "/dummy/0.4.0", "/x86_64-pc-windows-msvc/dummy", "/", true);
13671375
run_check_links(
13681376
env,
1369-
"/dummy/0.4.0/x86_64-pc-windows-msvc/dummy/struct.A.html",
1377+
"/dummy/0.4.0",
1378+
"/x86_64-pc-windows-msvc/dummy/struct.A.html",
13701379
"/",
13711380
true,
13721381
);

‎src/web/routes.rs‎

Lines changed: 11 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -237,51 +237,47 @@ pub(super) fn build_axum_routes() -> AxumRouter {
237237
get_internal(super::source::source_browser_handler),
238238
)
239239
.route(
240-
"/-/menus/platforms/crate/:name/:version",
240+
"/crate/:name/:version/menus/platforms/crate/",
241241
get_internal(super::crate_details::get_all_platforms_root),
242242
)
243243
.route(
244-
"/-/menus/platforms/crate/:name/:version/features",
244+
"/crate/:name/:version/menus/platforms/crate/features",
245245
get_internal(super::crate_details::get_all_platforms_root),
246246
)
247247
.route(
248-
"/-/menus/platforms/crate/:name/:version/builds",
248+
"/crate/:name/:version/menus/platforms/crate/builds",
249249
get_internal(super::crate_details::get_all_platforms_root),
250250
)
251251
.route(
252-
"/-/menus/platforms/crate/:name/:version/builds/*path",
252+
"/crate/:name/:version/menus/platforms/crate/builds/*path",
253253
get_internal(super::crate_details::get_all_platforms_root),
254254
)
255255
.route(
256-
"/-/menus/platforms/crate/:name/:version/source/",
256+
"/crate/:name/:version/menus/platforms/crate/source/",
257257
get_internal(super::crate_details::get_all_platforms_root),
258258
)
259259
.route(
260-
"/-/menus/platforms/crate/:name/:version/source/*path",
260+
"/crate/:name/:version/menus/platforms/crate/source/*path",
261261
get_internal(super::crate_details::get_all_platforms_root),
262262
)
263263
.route(
264-
"/-/menus/platforms/crate/:name/:version/:target",
264+
"/crate/:name/:version/menus/platforms/:target",
265265
get_internal(super::crate_details::get_all_platforms),
266266
)
267267
.route(
268-
"/-/menus/platforms/crate/:name/:version/:target/*path",
268+
"/crate/:name/:version/menus/platforms/:target/*path",
269269
get_internal(super::crate_details::get_all_platforms),
270270
)
271271
.route(
272-
"/-/menus/platforms/:name/:version/",
272+
"/crate/:name/:version/menus/platforms/",
273273
get_internal(super::crate_details::get_all_platforms),
274274
)
275275
.route(
276-
"/-/menus/platforms/:name/:version/:target/",
276+
"/crate/:name/:version/menus/platforms/:target/",
277277
get_internal(super::crate_details::get_all_platforms),
278278
)
279279
.route(
280-
"/-/menus/platforms/:name/:version/:target/*path",
281-
get_internal(super::crate_details::get_all_platforms),
282-
)
283-
.route(
284-
"/-/menus/releases/:name",
280+
"/crate/:name/:version/menus/releases",
285281
get_internal(super::crate_details::get_all_releases),
286282
)
287283
.route(

‎static/menu.js‎

Lines changed: 14 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,14 @@ function loadAjaxMenu(menu, id, msg, path, extra) {
2020
// We're not in a documentation page, so no need to do anything.
2121
return;
2222
}
23-
const crateName = window.location.pathname.split('/')[1];
23+
const parts = window.location.pathname.split("/");
24+
let crateName = parts[1];
25+
let version = parts[2];
26+
if (crateName === "crate") {
27+
crateName = parts[2];
28+
version = parts[3];
29+
path += "/crate";
30+
}
2431
const xhttp = new XMLHttpRequest();
2532
xhttp.onreadystatechange = function() {
2633
if (xhttp.readyState !== XMLHttpRequest.DONE) {
@@ -33,7 +40,8 @@ function loadAjaxMenu(menu, id, msg, path, extra) {
3340
document.getElementById(id).innerHTML = `Failed to load ${msg}`;
3441
}
3542
};
36-
xhttp.open("GET", `/-/menus/${path}/${crateName}${extra}`, true);
43+
console.log(extra, path);
44+
xhttp.open("GET", `/crate/${crateName}/${version}/menus/${path}${extra}`, true);
3745
xhttp.send();
3846
};
3947

@@ -88,13 +96,15 @@ function loadAjaxMenu(menu, id, msg, path, extra) {
8896
if (newMenu.querySelector("#releases-list")) {
8997
loadAjaxMenu(newMenu, "releases-list", "release list", "releases", "");
9098
} else if (newMenu.querySelector("#platforms")) {
99+
const parts = window.location.pathname.split("/");
100+
const startFrom = parts[1] === "crate" ? 4 : 3;
91101
loadAjaxMenu(
92102
newMenu,
93103
"platforms",
94104
"platforms list",
95105
"platforms",
96-
// We get everything except the first crate name.
97-
"/" + window.location.pathname.split("/").slice(2).join("/")
106+
// We get everything except the first crate name and the version.
107+
"/" + parts.slice(startFrom).join("/")
98108
);
99109
}
100110
}

0 commit comments

Comments
 (0)