Fix dependencies route only showing one dependency per version (#229)
This commit is contained in:
parent
5b8612c919
commit
66becbc4cc
@ -229,9 +229,9 @@ pub async fn auth_callback(
|
|||||||
transaction.commit().await?;
|
transaction.commit().await?;
|
||||||
|
|
||||||
let redirect_url = if result.url.contains("?") {
|
let redirect_url = if result.url.contains("?") {
|
||||||
format!("{}&code={}", result.url, token.access_token)
|
format!("{}&code={}", result.url, token.access_token)
|
||||||
} else {
|
} else {
|
||||||
format!("{}?code={}", result.url, token.access_token)
|
format!("{}?code={}", result.url, token.access_token)
|
||||||
};
|
};
|
||||||
|
|
||||||
Ok(HttpResponse::TemporaryRedirect()
|
Ok(HttpResponse::TemporaryRedirect()
|
||||||
|
|||||||
@ -188,30 +188,36 @@ pub async fn dependency_list(
|
|||||||
)
|
)
|
||||||
.await?;
|
.await?;
|
||||||
|
|
||||||
let mut response: HashMap<models::projects::VersionId, DependencyInfo> = HashMap::new();
|
let mut response: HashMap<models::projects::VersionId, Vec<DependencyInfo>> =
|
||||||
|
HashMap::new();
|
||||||
|
|
||||||
for dependency in dependencies {
|
for dependency in dependencies {
|
||||||
response.insert(
|
let deps = response.get_mut(&dependency.0.into());
|
||||||
dependency.0.into(),
|
|
||||||
DependencyInfo {
|
let info = DependencyInfo {
|
||||||
project: if let Some(id) = dependency.2 {
|
project: if let Some(id) = dependency.2 {
|
||||||
projects
|
projects
|
||||||
.iter()
|
.iter()
|
||||||
.find(|x| x.inner.id == id)
|
.find(|x| x.inner.id == id)
|
||||||
.map(|x| convert_project(x.clone()))
|
.map(|x| convert_project(x.clone()))
|
||||||
} else {
|
} else {
|
||||||
None
|
None
|
||||||
},
|
|
||||||
version: if let Some(id) = dependency.1 {
|
|
||||||
versions
|
|
||||||
.iter()
|
|
||||||
.find(|x| x.id == id)
|
|
||||||
.map(|x| super::versions::convert_version(x.clone()))
|
|
||||||
} else {
|
|
||||||
None
|
|
||||||
},
|
|
||||||
},
|
},
|
||||||
);
|
version: if let Some(id) = dependency.1 {
|
||||||
|
versions
|
||||||
|
.iter()
|
||||||
|
.find(|x| x.id == id)
|
||||||
|
.map(|x| super::versions::convert_version(x.clone()))
|
||||||
|
} else {
|
||||||
|
None
|
||||||
|
},
|
||||||
|
};
|
||||||
|
|
||||||
|
if let Some(deps) = deps {
|
||||||
|
deps.push(info);
|
||||||
|
} else {
|
||||||
|
response.insert(dependency.0.into(), vec![info]);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
Ok(HttpResponse::Ok().json(response))
|
Ok(HttpResponse::Ok().json(response))
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user