Merge pull request 'feat(#14): Add packager to details page' (#21) from dev-0.1.4 into master

Reviewed-on: #21
Reviewed-by: nikolar <nikolar@artixlinux.org>

Cargo.lock

@@ -410,7 +410,7 @@ dependencies = [
 
 [[package]]
 name = "artix-gitea"
-version = "0.1.2"
+version = "0.1.4"
 dependencies = [
  "actix-web",
  "awc",
@@ -420,7 +420,7 @@ dependencies = [
 
 [[package]]
 name = "artix-pkglib"
-version = "0.1.2"
+version = "0.1.4"
 dependencies = [
  "alpm",
  "alpm-utils",
@@ -430,7 +430,7 @@ dependencies = [
 
 [[package]]
 name = "artixweb_packages"
-version = "0.1.2"
+version = "0.1.4"
 dependencies = [
  "actix-files",
  "actix-identity",

crates/artix-gitea/Cargo.toml

@@ -8,7 +8,7 @@ homepage = "https://packages.artixlinux.org"
 keywords = ["artix", "packages", "gitea"]
 license = "MIT OR Apache-2.0"
 edition = "2021"
-version = "0.1.2"
+version = "0.1.4"
 
 [lib]
 name = "artix_gitea"

crates/artix-pkglib/Cargo.toml

@@ -8,7 +8,7 @@ homepage = "https://packages.artixlinux.org"
 keywords = ["artix", "packages"]
 license = "MIT OR Apache-2.0"
 edition = "2021"
-version = "0.1.2"
+version = "0.1.4"
 
 [lib]
 name = "artix_pkglib"

crates/artixweb-packages/Cargo.toml

@@ -7,7 +7,7 @@ repository = "gitea.artixlinux.org/artix/artixweb_packages"
 keywords = ["artix", "packages"]
 license = "MIT OR Apache-2.0"
 edition = "2021"
-version = "0.1.2"
+version = "0.1.4"
 
 [[bin]]
 name = "artixweb_packages"
@@ -15,8 +15,8 @@ test = false
 bench = false
 
 [dependencies]
-artix-gitea = { path = "../artix-gitea", version = "=0.1.2" }
+artix-gitea = { path = "../artix-gitea", version = "=0.1.4" }
-artix-pkglib = { path = "../artix-pkglib", version = "=0.1.2" }
+artix-pkglib = { path = "../artix-pkglib", version = "=0.1.4" }
 
 actix-files = "0.6.0"
 actix-identity = "0.4"

crates/artixweb-packages/src/handlers/packages.rs

@@ -110,6 +110,7 @@ pub(crate) struct ResponseDetail {
     pub required_by: Vec<String>,
     pub contents: Vec<String>,
     pub maintainers: Vec<String>,
+    pub packaged_by: String,
     pub flagged: bool,
     pub flagged_on: i64,
     pub flagged_by: Vec<String>,
@@ -457,7 +458,7 @@ pub(crate) async fn get_packages_details_inner(
                     .iter()
                     .map(std::string::ToString::to_string)
                     .collect(),
-                gitea_url: String::new(),
+                gitea_url: format!("{}/packages/{}", SETTINGS.gitea_url, pkg.name().to_string()),
                 dependencies: get_depends_from_package(
                     &handle,
                     pkg.arch().unwrap_or("any"),
@@ -506,6 +507,7 @@ pub(crate) async fn get_packages_details_inner(
                     pkg.replaces(),
                     &DependencyKind::Misc,
                 ),
+                packaged_by: format_packager(pkg.packager().unwrap_or("").to_string()),
                 flagged: false,
                 flagged_on: 0,
                 flagged_by: Vec::new(),
@@ -517,7 +519,6 @@ pub(crate) async fn get_packages_details_inner(
                 web::block(move || get_package_metadata(&pkg_name_version, &pool)).await
             {
                 result.last_updated = metadata.last_update.timestamp();
-                result.gitea_url = metadata.gitea_url;
                 result.flagged = metadata.flagged;
                 if let Some(flagged_on) = metadata.flag_on {
                     result.flagged_on = flagged_on.timestamp_millis();
@@ -566,6 +567,14 @@ pub(crate) async fn get_packages_details_inner(
     Err(ArtixWebPackageError::NotFound)
 }
 
+// removes email address from the packager string
+fn format_packager(mut input: String) -> String {
+    if let Some(pos) = input.find('<') {
+        input.truncate(pos - 1);
+    }
+    input
+}
+
 // adds package metadata into the local postgres database
 fn add_package_metadata(metadata: &PackageMeta, pool: &web::Data<Pool>) {
     use crate::schema::packages::dsl::packages;

crates/artixweb-packages/templates/package_details.html

@@ -72,6 +72,8 @@
                 <section role="value">{{ pkg.installed_size|human_readable }} MB</section>
                 <section role="key">Build Date:</section>
                 <section role="value">{{ pkg.build_date|show_date }}</section>
+                <section role="key">Packaged By:</section>
+                <section role="value">{{ pkg.packaged_by }}</section>
                 {% if pkg.flagged && !pkg.flagged_by.is_empty() %}
                     <section role="key">Flagged By:</section>
                     <section role="value">{{ pkg.flagged_by|join(", ") }}</section>